Added comments, clarified a previous changelog entry.

1 files changed, 4 insertions, 2 deletions

diff --git a/netx/net/sourceforge/jnlp/JNLPFile.java b/netx/net/sourceforge/jnlp/JNLPFile.java
index 2596c47..78fc60d 100644
--- a/netx/net/sourceforge/jnlp/JNLPFile.java
+++ b/netx/net/sourceforge/jnlp/JNLPFile.java
@@ -395,6 +395,8 @@ public class JNLPFile {
      * Returns the resources section of the JNLP file as viewed
      * through the default locale and the os.name and os.arch
      * properties.
+     * XXX: Before overriding this method or changing its implementation,
+     * read the comment in JNLPFile.getDownloadOptionsForJar(JARDesc).
      */
     public ResourcesDesc[] getResourcesDescs() {
         return getResourcesDescs(defaultLocale, defaultOS, defaultArch);
@@ -649,7 +651,7 @@ public class JNLPFile {
      * XXX: this method does a "==" comparison between the input JARDesc and
      * jars it finds through getResourcesDescs(). If ever the implementation
      * of that function should change to return copies of JARDescs objects,
-     * then the "jar == aJar" comparison below should change accordingly. 
+     * then the "jar == aJar" comparison below should change accordingly.
      * @param jar: the jar whose download options to get.
      * @return the download options.
      */
@@ -660,7 +662,7 @@ public class JNLPFile {
         for (ResourcesDesc desc: descs) {
             JARDesc[] jars = desc.getJARs();
             for (JARDesc aJar: jars) {
-                if (jar == aJar/*jar.getLocation().equals(aJar.getLocation())*/) {
+                if (jar == aJar) {
                     if (Boolean.valueOf(desc.getPropertiesMap().get("jnlp.packEnabled"))) {
                         usePack = true;
                     }