[7u4] Review request for CR 7134730

[Kumar Srinivasan]

Test.java:
you will need a null check for args, ie. if there are no args you will 
encounter
a NPE  for args.length.


AppBundlerTask.java

there are several instances of the old type of for-loops.
lines 141, lines 455, lines 479

using  static imports for StandardCopyOption, LinkOption, will make the code
much nicer,  here is an example of some recent changes I made which 
utilizes this:
http://hg.openjdk.java.net/jdk8/tl/jdk/diff/7dbc129d8e5c/test/tools/launcher/TestHelper.java

Otherwise it looks ok to me.

Thanks
Kumar



> Thanks all for the feedback. Updated webrev below:
>
> http://cr.openjdk.java.net/~gkbrown/7134730/webrev.01/
>
> - Rename "Java" dictionary key to "JavaVM"
> - Add GPL headers as needed
> - Apply correct Oracle copyright to all files
> - Set cp_wildcard to FALSE in call to JLI_Launch()
> - Use diamond operator in collection initialization
> - Output command-line arguments in test app
>
> I also rebuilt the sample bundle with a more recent JRE and the Command-Q problem seems to have resolved itself:
>
> http://dl.dropbox.com/u/49180229/SwingSet2.dmg
>
> Greg
>
> On Feb 13, 2012, at 11:16 AM, Greg Brown wrote:
>
>> This is a request to push the initial changeset supporting the app bundler Ant task. Webrev is here:
>>
>> http://cr.openjdk.java.net/~gkbrown/7134730/webrev.00/
>>
>> Note that this code is not currently integrated into the build. An outstanding to-do is to move the compilation of main.m from build.xml to a makefile and add this to the build. These changes will be pushed in a follow-on changeset.
>>
>> Thanks,
>> Greg
>>