[RFC][IcedTea-WEB]: Fix for PR765

[Deepak Bhole]

* Saad Mohammad <smohammad at redhat.com> [2011-08-08 15:04]:
> This is the patch for PR765:
>     http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=765
> 
> This bug is reproduced by having the jar (with the main class)
> marked as lazy with additional resources (if any) that are also
> marked as lazy.
> 
> ChangeLog:
> 
> 2011-08-08  Saad Mohammad <smohammad at redhat.com>
> 
>     * netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java:
>       (initializeResources): Always initializes the jar file with
> the main class
>       at start (even if the jar file is marked as lazy)
>

Hi Saad,

ChangeLog indentation is not wrong (not the overflow, which you
addressed in another email), but the spacing. Additional lines should
only be preceded with a tab, no spaces.

Also, this should have a NEWS entry for the bugfix.

Code changes look fine to me.

Please repost with fix for ChangeLog and with a NEWS item.

Cheers,
Deepak
 
> -- 
> Cheers,
> Saad Mohammad
> 

> diff -r defa7d0051bf netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java
> --- a/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java	Wed Aug 03 14:11:11 2011 -0400
> +++ b/netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java	Mon Aug 08 14:27:47 2011 -0400
> @@ -415,11 +415,14 @@
>          */
>          List<JARDesc> initialJars = new ArrayList<JARDesc>();
>  
> +        //Stores the URL of the jar file that has the main class
> +        URL mainJarURL= file.getResources().getMainJAR().getLocation();
> +
>          for (int i = 0; i < jars.length; i++) {
>  
>              available.add(jars[i]);
> 
> -            if (jars[i].isEager())
> +            if (jars[i].isEager() || jars[i].getLocation().equals(mainJarURL))
>                  initialJars.add(jars[i]); // regardless of part
>  
>              tracker.addResource(jars[i].getLocation(),