RFR (S): 8013574: PrintMalloc conflicts with the command line parsing

[John Cuthbertson]

Hi Bengt,

Changes look good to me. Good find.

JohnC

On 5/1/2013 1:52 PM, Bengt Rutisson wrote:
>
> Hi all,
>
> Could I have a couple of reviews for this small fix?
>
> http://cr.openjdk.java.net/~brutisso/8013574/webrev.00/
>
> The problem is that Arguments::add_string() updates the count ( 
> _num_jvm_args) before it adds the actual entry to the bldarray ( 
> _jvm_args_array). This is a problem since the allocation might 
> eventually lead to a call to print_jvm_args_on(), which relies on 
> _num_jvm_args and _jvm_args_array being in sync.
>
> The proposed fix updates the count after the corresponding entry has 
> been allocated.
>
> Thanks,
> Bengt

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/attachments/20130502/36b7acb0/attachment.html