[RFC][icedtea-web]: Set plugin JVM options in itw-settings
Adam Domurad
adomurad at redhat.com
Wed Dec 5 06:26:40 PST 2012
On 12/04/2012 04:25 PM, Saad Mohammad wrote:
> Hi Adam,
>
> Thanks for all the tips and tricks around C++. Updated patch is attached!
> I'm starting to enjoy C++ as things start working more correctly now. :)
Good to hear :)
> [..snip..]
> + }
> +
> + tokenOutput = IcedTeaPluginUtilities::strSplit(output, " \n");
> +
> + if (!tokenOutput->empty() && *tokenOutput->at(0) =="null")
> + {
> + delete tokenOutput->at(0);
> + tokenOutput->erase(tokenOutput->begin());
> + }
A comment here about why you're deleting this token would be nice.
Otherwise, I have no further torture to inflict on you :) Looks good to
me now. If you have tested it to your satisfaction, OK for HEAD.
Thanks,
-Adam
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20121205/bf7382d1/attachment.html
More information about the distro-pkg-dev
mailing list