RFR: JDK-8222079: Don't use memset to initialize fields decode_env constructor in disassembler.cpp

[Thomas Stüfe]

Hi Roman,

Looks fine.

(surprised at "_option_buf()" btw, I always thought that member array
initialization needs curly brackets.)

Cheers, Thomas

On Thu, Jul 4, 2019 at 1:11 PM Roman Kennke <rkennke at redhat.com> wrote:

> Uh I just went through my list of open bugs only to realize that I
> totally forgot to push this one. Some things changed in the meantime, so
> here comes an updated patch:
>
> http://cr.openjdk.java.net/~rkennke/JDK-8222079/webrev.02/
>
> Can I please get another review?
>
> Thanks,
> Roman
>
>
> > Looks good.
> >
> > dl
> >
> > On 5/3/19 9:59 AM, Roman Kennke wrote:
> >> Ping?
> >>
> >> Thanks,
> >> Roman
> >>
> >>
> >>> Recent gcc (I use version 9) complains about using memset to
> >>> initialize fields of decode_env. Let's use proper field initializers
> >>> instead.
> >>>
> >>> Bug:
> >>> https://bugs.openjdk.java.net/browse/JDK-8222079
> >>> Webrev:
> >>> http://cr.openjdk.java.net/~rkennke/JDK-8222079/webrev.01/
> >>>
> >>> Can I please get a review?
> >>>
> >>> Thanks, Roman
> >
>
>