RFC: Make javax.jnlp package visible to javac

[Omair Majid]

Andrew John Hughes wrote:
> 2009/6/4 Omair Majid <omajid at redhat.com>:
>> Hi,
>>
>> Sun's documentation states that javac should be able to compile code that
>> uses javax.jnlp once jnlp.jar is added to the classpath [1]. IcedTea6,
>> however, has no jnlp.jar, and javax.jnlp.* classes are already in rt.jar. It
>> turns out that you have to use 'javac -XDignore.symbol.file OnlineTest.java'
>> for this to work [2].
>>
>> The attached patch adds javax.jnlp to NON_CORE_PKGS list, making it visible
>> to javac.
>>
>> ChangeLog:
>> 2009-06-04  Omair Majid  <omajid at redhat.com>
>>
>>    * patches/icedtea-webstart.patch: Make javax.jnlp classes visible.
>>
>> Ok to commit?
>>
>> Cheers,
>> Omair
>>
>>
>> [1]
>> http://java.sun.com/javase/6/docs/technotes/guides/javaws/developersguide/examples.html#Introduction
>> [2] http://andrew-haley.livejournal.com/695.html
>>
> 
> Looks good to me (though seems strange the Makefile is under docs...).
I could move the Makefile in the patch ;)

> 
> Did we ever work out if there was a good reason for keeping this
> symbol mapping stuff?

Nothing formally. If I recall correctly, Andrew Haley mentioned that it 
might make javac faster. Presumably anything not visible to javac is not 
really part of the Java API and there should be no need for programmers 
to use those classes (but this doesnt hold true for some parts of IcedTea).

Cheers,
Omair