I would also be inclined to change
== 0
to
== NULL
This seems like another occasion to use the weird
do { ... } while(0)
trick to make the macro behave more like a statement.
I might obfuscate the macro parameters to make collisions less likely, e.g.
e => N_C_RV_e
On Wed, May 1, 2013 at 12:33 PM, Kumar Srinivasan <
[email protected]> wrote:
> Hi,
>
> Please review simple fixes for code correctness in the launcher.
>
> http://cr.openjdk.java.net/~**ksrini/8013736/webrev.0/<http://cr.openjdk.java.net/~ksrini/8013736/webrev.0/>
>
> Thanks
> Kumar
>
>
