RFR: 8170832: Add a new launcher environment variable JAVA_OPTIONS

[Mandy Chung]

> On Jan 24, 2017, at 10:20 AM, Henry Jen <henry.jen at oracle.com> wrote:
> 
> Hi,
> 
> Please review the webrev[1] that add support for JAVA_OPTIONS environment variable. The bug[2] describes how JAVA_OPTIONS works.
> 
> [1] http://cr.openjdk.java.net/~henryjen/jdk9/8170832/4/webrev/

This looks quite good.  A couple of minor comments:

 503         // Must be after expansion so we can caught if main class specified @argfile

typo: s/caught/catch.  It’d be clear to simply say:
 // Check if Main class specified after argument checked.
 // This check must be done after expansion.

  42 #define JAVA_OPTIONS “JAVA_OPTIONS"

I think java.h is the appropriate file to declare this instead of jli_util.h.

emessages.h
  Would it be clearer to rename ARG_INFO to ARG_INFO_ENVVAR?

thanks
Mandy