diff options
| author | Adam Domurad <[email protected]> | 2012-05-17 14:14:03 -0400 |
|---|---|---|
| committer | Adam Domurad <[email protected]> | 2012-05-17 14:14:03 -0400 |
| commit | b42384fe344b0af2a54eb00735f7d59b6ba1ab6e (patch) | |
| tree | 1dc30db08c963ecdd9fbe5c7186a007f110b8057 /netx/net/sourceforge/jnlp/JNLPFile.java | |
| parent | 0c598ca6848b417c79245a971f26570d05144132 (diff) | |
Went through the source of IcedTeaWeb with FindBugs and went over all reported cases of == being used to compare String's. Some usage cases were valid (eg, .equals eventually called, magic String value). I noted one such usage case. The others were changed to .equals calls.
Diffstat (limited to 'netx/net/sourceforge/jnlp/JNLPFile.java')
| -rw-r--r-- | netx/net/sourceforge/jnlp/JNLPFile.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/netx/net/sourceforge/jnlp/JNLPFile.java b/netx/net/sourceforge/jnlp/JNLPFile.java index 714ff7e..5ddb989 100644 --- a/netx/net/sourceforge/jnlp/JNLPFile.java +++ b/netx/net/sourceforge/jnlp/JNLPFile.java @@ -208,7 +208,7 @@ public class JNLPFile { //(i.e. If the jnlp file being launched exist locally, but it //originated from a website, then download the one from the website //into the cache). - if (sourceLocation != null && location.getProtocol() == "file") { + if (sourceLocation != null && "file".equals(location.getProtocol())) { openURL(sourceLocation, version, policy); } |