Review request: 6378314: Bad warning message when agent library not found. local directory is not searched.
Tom Rodriguez
tom.rodriguez at oracle.com
Wed Aug 4 11:43:21 PDT 2010
The SunStudio compiler doesn't like this construct:
+ size_t len = strlen(msg) + strlen(name) + strlen(sub_msg) + strlen(ebuf) + 1;
+ char buf[len];
so I think you'll have to use another fixed size buffer.
tom
On Aug 4, 2010, at 11:20 AM, Chuck Rasbold wrote:
> Thanks, John.
>
> The nits have been fixed and the webrev updated.
>
> http://cr.openjdk.java.net/~rasbold/6378314/
>
> -- Chuck
>
> On Tue, Aug 3, 2010 at 4:36 PM, John Coomes <John.Coomes at oracle.com> wrote:
> Chuck Rasbold (rasbold at google.com) wrote:
> > http://cr.openjdk.java.net/~rasbold/6378314/
> >
> > Fixed: 6378314: Bad warning message when agent library not found. local
> > directory is not searched.
> > Contributed-by: jeremymanson at google.com
> >
> > Print a more detailed error message for agent library load failure.
>
> Hi Chuck & Jeremy,
>
> Looks ok to me, aside from a couple of nits: use jio_snprintf instead
> of sprintf, and strlen returns a size_t, so use that to declare len.
>
> -John
>
>
More information about the hotspot-runtime-dev
mailing list