/hg/icedtea-web: Fixed a string comparison bug in PolicyEditor.main

[aazores at icedtea.classpath.org]

changeset 716fc6d1463a in /hg/icedtea-web
details: http://icedtea.classpath.org/hg/icedtea-web?cmd=changeset;node=716fc6d1463a
author: Andrew Azores <aazores at redhat.com>
date: Wed Mar 18 12:43:51 2015 -0400

	Fixed a string comparison bug in PolicyEditor.main


diffstat:

 ChangeLog                                                         |  6 ++++++
 netx/net/sourceforge/jnlp/security/policyeditor/PolicyEditor.java |  2 +-
 2 files changed, 7 insertions(+), 1 deletions(-)

diffs (25 lines):

diff -r b8689b23ce0b -r 716fc6d1463a ChangeLog
--- a/ChangeLog	Wed Mar 18 16:56:46 2015 +0100
+++ b/ChangeLog	Wed Mar 18 12:43:51 2015 -0400
@@ -1,3 +1,9 @@
+2015-03-18  Andrew Azores  <aazores at redhat.com>
+
+	* netx/net/sourceforge/jnlp/security/policyeditor/PolicyEditor.java: (main)
+	Fixed a bug where a local string variable was compared to the empty string
+	using ==.
+
 2015-03-18  Jiri Vanek  <jvanek at redhat.com>
 
 	* NEWS: mentioned documentation. -html and menu shortcuts moved to general.
diff -r b8689b23ce0b -r 716fc6d1463a netx/net/sourceforge/jnlp/security/policyeditor/PolicyEditor.java
--- a/netx/net/sourceforge/jnlp/security/policyeditor/PolicyEditor.java	Wed Mar 18 16:56:46 2015 +0100
+++ b/netx/net/sourceforge/jnlp/security/policyeditor/PolicyEditor.java	Wed Mar 18 12:43:51 2015 -0400
@@ -1627,7 +1627,7 @@
                 String filepath = optionParser.getParam(OptionsDefinitions.OPTIONS.FILE);
                 if (optionParser.getMainArgs().size() == 0) {
                     filepath = null;
-                } else if (filepath == "") {
+                } else if (filepath.isEmpty()) {
                     // maybe the user just forgot the -file flag, so try to open anyway
                     filepath = optionParser.getMainArg();
                 }