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

[Thomas Schatzl]

Hi,

On Thu, 2013-05-02 at 11:38 -0700, John Cuthbertson wrote:

> 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.

Looks good.

 Thomas