[icedtea-web] RFC: launch application that have a non-public main class

[Omair Majid]

On 10/26/2010 06:23 PM, Dr Andrew John Hughes wrote:
> On 16:04 Tue 26 Oct     , Omair Majid wrote:
>> Hi,
>>
>> The attached patch bypasses checks imposed by the reflection API to
>> ensure that netx is allowed to access the class it is trying to access.
>> Generally, this makes complete sense. However, there are some
>> applications (such as the Present/Future Value Calculator [1]) that
>> declare their main method in a non public class. Netx currently fails
>> with an IllegalAccessException when it encounters this.
>>
>> ChangeLog
>> 2010-10-26  Omair Majid<omajid at redhat.com>
>>
>>       * netx/net/sourceforge/jnlp/Launcher.java
>>       (launchApplication): Mark main method as accessible before invoking
>>       it.
>>
>> Thanks,
>> Omair
>>
>> [1] http://www.arbores.ca/PresentFuture.jnlp
>
> Looks good to me.
>

Thanks for the review. I have pushed the change.

Cheers,
Omair