Author: vhennebert
Date: Mon Jun 18 14:06:12 2012
New Revision: 1351350

URL: http://svn.apache.org/viewvc?rev=1351350&view=rev
Log:
Fixed FindBugs issue: do not compare strings with == or !=

Modified:
    
xmlgraphics/fop/branches/Temp_TrueTypeInPostScript/src/java/org/apache/fop/fonts/truetype/TTFFontLoader.java

Modified: 
xmlgraphics/fop/branches/Temp_TrueTypeInPostScript/src/java/org/apache/fop/fonts/truetype/TTFFontLoader.java
URL: 
http://svn.apache.org/viewvc/xmlgraphics/fop/branches/Temp_TrueTypeInPostScript/src/java/org/apache/fop/fonts/truetype/TTFFontLoader.java?rev=1351350&r1=1351349&r2=1351350&view=diff
==============================================================================
--- 
xmlgraphics/fop/branches/Temp_TrueTypeInPostScript/src/java/org/apache/fop/fonts/truetype/TTFFontLoader.java
 (original)
+++ 
xmlgraphics/fop/branches/Temp_TrueTypeInPostScript/src/java/org/apache/fop/fonts/truetype/TTFFontLoader.java
 Mon Jun 18 14:06:12 2012
@@ -199,10 +199,10 @@ public class TTFFontLoader extends FontL
                     if (codePoint <= 0) {
                         int glyphIndex = segment.getGlyphStartIndex() + u - 
segment.getUnicodeStart();
                         String glyphName = ttf.getGlyphName(glyphIndex);
-                        if (glyphName == "" && ttf.getPostScriptVersion() != 
PostScriptVersion.V2) {
+                        if (glyphName.length() == 0 && 
ttf.getPostScriptVersion() != PostScriptVersion.V2) {
                             glyphName = "u" + HexEncoder.encode(u);
                         }
-                        if (glyphName != "") {
+                        if (glyphName.length() > 0) {
                             String unicode = Character.toString(u);
                             NamedCharacter nc = new NamedCharacter(glyphName, 
unicode);
                             singleFont.addUnencodedCharacter(nc, 
wx[glyphIndex]);



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to