You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by on...@apache.org on 2017/10/18 08:41:43 UTC
svn commit: r1812476 - in /poi/trunk/src: java/org/apache/poi/hpsf/
java/org/apache/poi/poifs/crypt/ java/org/apache/poi/sl/draw/
java/org/apache/poi/ss/formula/functions/ java/org/apache/poi/ss/usermodel/
java/org/apache/poi/util/ ooxml/java/org/apach...
Author: onealj
Date: Wed Oct 18 08:41:42 2017
New Revision: 1812476
URL: http://svn.apache.org/viewvc?rev=1812476&view=rev
Log:
replace "".equals(string) with string.isEmpty() to avoid null strings silently evaluating to false; most code should probably check string for null if it is not already
Modified:
poi/trunk/src/java/org/apache/poi/hpsf/Variant.java
poi/trunk/src/java/org/apache/poi/poifs/crypt/CryptoFunctions.java
poi/trunk/src/java/org/apache/poi/poifs/crypt/DataSpaceMapUtils.java
poi/trunk/src/java/org/apache/poi/sl/draw/DrawTextParagraph.java
poi/trunk/src/java/org/apache/poi/ss/formula/functions/Countblank.java
poi/trunk/src/java/org/apache/poi/ss/usermodel/FractionFormat.java
poi/trunk/src/java/org/apache/poi/util/StringUtil.java
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/PackagePartName.java
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/ZipPackage.java
poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/OOXMLURIDereferencer.java
poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/RelationshipTransformService.java
poi/trunk/src/ooxml/java/org/apache/poi/ss/extractor/EmbeddedExtractor.java
poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFHyperlink.java
poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFTextRun.java
poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFPivotTable.java
poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/helpers/XSSFPasswordHelper.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFFontInfo.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSimpleShape.java
Modified: poi/trunk/src/java/org/apache/poi/hpsf/Variant.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/hpsf/Variant.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/hpsf/Variant.java (original)
+++ poi/trunk/src/java/org/apache/poi/hpsf/Variant.java Wed Oct 18 08:41:42 2017
@@ -420,7 +420,7 @@ public class Variant
}
name += numberToName.get(vt);
- return !"".equals(name) ? name : "unknown variant type";
+ return !name.isEmpty() ? name : "unknown variant type";
}
/**
Modified: poi/trunk/src/java/org/apache/poi/poifs/crypt/CryptoFunctions.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/poifs/crypt/CryptoFunctions.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/poifs/crypt/CryptoFunctions.java (original)
+++ poi/trunk/src/java/org/apache/poi/poifs/crypt/CryptoFunctions.java Wed Oct 18 08:41:42 2017
@@ -379,7 +379,7 @@ public class CryptoFunctions {
// SET Verifier TO 0x0000
short verifier = 0;
- if (!"".equals(password)) {
+ if (!password.isEmpty()) {
// FOR EACH PasswordByte IN PasswordArray IN REVERSE ORDER
for (int i = arrByteChars.length-1; i >= 0; i--) {
// SET Verifier TO Intermediate3 BITWISE XOR PasswordByte
@@ -418,7 +418,7 @@ public class CryptoFunctions {
//Maximum length of the password is 15 chars.
final int maxPasswordLength = 15;
- if (!"".equals(password)) {
+ if (!password.isEmpty()) {
// Truncate the password to 15 characters
password = password.substring(0, Math.min(password.length(), maxPasswordLength));
Modified: poi/trunk/src/java/org/apache/poi/poifs/crypt/DataSpaceMapUtils.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/poifs/crypt/DataSpaceMapUtils.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/poifs/crypt/DataSpaceMapUtils.java (original)
+++ poi/trunk/src/java/org/apache/poi/poifs/crypt/DataSpaceMapUtils.java Wed Oct 18 08:41:42 2017
@@ -356,7 +356,7 @@ public class DataSpaceMapUtils {
}
public static void writeUtf8LPP4(LittleEndianOutput os, String str) {
- if (str == null || "".equals(str)) {
+ if (str == null || str.isEmpty()) {
os.writeInt(str == null ? 0 : 4);
os.writeInt(0);
} else {
Modified: poi/trunk/src/java/org/apache/poi/sl/draw/DrawTextParagraph.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/sl/draw/DrawTextParagraph.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/sl/draw/DrawTextParagraph.java (original)
+++ poi/trunk/src/java/org/apache/poi/sl/draw/DrawTextParagraph.java Wed Oct 18 08:41:42 2017
@@ -257,7 +257,7 @@ public class DrawTextParagraph implement
fact.fixFonts(graphics);
StringBuilder text = new StringBuilder();
AttributedString at = getAttributedString(graphics, text);
- boolean emptyParagraph = ("".equals(text.toString().trim()));
+ boolean emptyParagraph = text.toString().trim().isEmpty();
AttributedCharacterIterator it = at.getIterator();
LineBreakMeasurer measurer = new LineBreakMeasurer(it, graphics.getFontRenderContext());
Modified: poi/trunk/src/java/org/apache/poi/ss/formula/functions/Countblank.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/ss/formula/functions/Countblank.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/ss/formula/functions/Countblank.java (original)
+++ poi/trunk/src/java/org/apache/poi/ss/formula/functions/Countblank.java Wed Oct 18 08:41:42 2017
@@ -54,7 +54,7 @@ public final class Countblank extends Fi
return valueEval == BlankEval.instance ||
// see https://support.office.com/en-us/article/COUNTBLANK-function-6a92d772-675c-4bee-b346-24af6bd3ac22
// "Cells with formulas that return "" (empty text) are also counted."
- (valueEval instanceof StringEval && "".equals(((StringEval)valueEval).getStringValue()));
+ (valueEval instanceof StringEval && ((StringEval)valueEval).getStringValue().isEmpty());
}
};
}
Modified: poi/trunk/src/java/org/apache/poi/ss/usermodel/FractionFormat.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/ss/usermodel/FractionFormat.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/ss/usermodel/FractionFormat.java (original)
+++ poi/trunk/src/java/org/apache/poi/ss/usermodel/FractionFormat.java Wed Oct 18 08:41:42 2017
@@ -150,7 +150,7 @@ public class FractionFormat extends Form
}
//if whole part has to go into the numerator
- if ("".equals(wholePartFormatString)){
+ if (wholePartFormatString.isEmpty()){
int trueNum = (fract.getDenominator()*(int)wholePart)+fract.getNumerator();
sb.append(trueNum).append("/").append(fract.getDenominator());
return sb.toString();
Modified: poi/trunk/src/java/org/apache/poi/util/StringUtil.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/util/StringUtil.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/util/StringUtil.java (original)
+++ poi/trunk/src/java/org/apache/poi/util/StringUtil.java Wed Oct 18 08:41:42 2017
@@ -348,7 +348,7 @@ public class StringUtil {
* @see <a href="http://www.alanwood.net/demos/symbol.html">Symbol font - Unicode alternatives for Greek and special characters in HTML</a>
*/
public static String mapMsCodepointString(String string) {
- if (string == null || "".equals(string)) return string;
+ if (string == null || string.isEmpty()) return string;
initMsCodepointMap();
StringBuilder sb = new StringBuilder();
Modified: poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java Wed Oct 18 08:41:42 2017
@@ -242,7 +242,7 @@ public abstract class OPCPackage impleme
*/
public static OPCPackage open(String path, PackageAccess access)
throws InvalidFormatException, InvalidOperationException {
- if (path == null || "".equals(path.trim())) {
+ if (path == null || path.trim().isEmpty()) {
throw new IllegalArgumentException("'path' must be given");
}
@@ -459,7 +459,7 @@ public abstract class OPCPackage impleme
try {
l.writeLock().lock();
if (this.originalPackagePath != null
- && !"".equals(this.originalPackagePath.trim())) {
+ && !this.originalPackagePath.trim().isEmpty()) {
File targetFile = new File(this.originalPackagePath);
if (!targetFile.exists()
|| !(this.originalPackagePath
Modified: poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/PackagePartName.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/PackagePartName.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/PackagePartName.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/PackagePartName.java Wed Oct 18 08:41:42 2017
@@ -260,7 +260,7 @@ public final class PackagePartName imple
+ partUri.getPath());
}
- if ("".equals(seg.replaceAll("\\\\.", ""))) {
+ if (seg.replaceAll("\\\\.", "").isEmpty()) {
// Normally will never been invoked with the previous
// implementation rule [M1.9]
throw new InvalidFormatException(
Modified: poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/ZipPackage.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/ZipPackage.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/ZipPackage.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/ZipPackage.java Wed Oct 18 08:41:42 2017
@@ -430,7 +430,7 @@ public final class ZipPackage extends OP
// Flush the package
flush();
- if (this.originalPackagePath == null || "".equals(this.originalPackagePath)) {
+ if (this.originalPackagePath == null || this.originalPackagePath.isEmpty()) {
return;
}
Modified: poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/OOXMLURIDereferencer.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/OOXMLURIDereferencer.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/OOXMLURIDereferencer.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/OOXMLURIDereferencer.java Wed Oct 18 08:41:42 2017
@@ -112,7 +112,7 @@ public class OOXMLURIDereferencer implem
LOG.log(POILogger.DEBUG, "dereference", uri);
String path = uri.getPath();
- if (path == null || "".equals(path)) {
+ if (path == null || path.isEmpty()) {
LOG.log(POILogger.DEBUG, "illegal part name (expected)", uri);
return null;
}
Modified: poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/RelationshipTransformService.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/RelationshipTransformService.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/RelationshipTransformService.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/RelationshipTransformService.java Wed Oct 18 08:41:42 2017
@@ -205,7 +205,7 @@ public class RelationshipTransformServic
String id = el.getAttribute("Id");
if (sourceIds.contains(id)) {
String targetMode = el.getAttribute("TargetMode");
- if ("".equals(targetMode)) {
+ if (targetMode.isEmpty()) {
el.setAttribute("TargetMode", "Internal");
}
rsList.put(id, el);
Modified: poi/trunk/src/ooxml/java/org/apache/poi/ss/extractor/EmbeddedExtractor.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/ss/extractor/EmbeddedExtractor.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/ss/extractor/EmbeddedExtractor.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/ss/extractor/EmbeddedExtractor.java Wed Oct 18 08:41:42 2017
@@ -138,14 +138,14 @@ public class EmbeddedExtractor implement
String extension = (filename == null || filename.lastIndexOf('.') == -1) ? ".bin" : filename.substring(filename.lastIndexOf('.'));
// try to find an alternative name
- if (filename == null || "".equals(filename) || filename.startsWith("MBD") || filename.startsWith("Root Entry")) {
+ if (filename == null || filename.isEmpty() || filename.startsWith("MBD") || filename.startsWith("Root Entry")) {
filename = shape.getShapeName();
if (filename != null) {
filename += extension;
}
}
// default to dummy name
- if (filename == null || "".equals(filename)) {
+ if (filename == null || filename.isEmpty()) {
filename = "picture_" + embeddings.size() + extension;
}
filename = filename.trim();
Modified: poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFHyperlink.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFHyperlink.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFHyperlink.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFHyperlink.java Wed Oct 18 08:41:42 2017
@@ -51,7 +51,7 @@ public class XSLFHyperlink implements Hy
@Override
public String getAddress() {
String id = _link.getId();
- if (id == null || "".equals(id)) {
+ if (id == null || id.isEmpty()) {
return _link.getAction();
}
@@ -163,4 +163,4 @@ public class XSLFHyperlink implements Hy
_link.setId("");
_link.setAction("ppaction://hlinkshowjump?jump="+jump);
}
-}
\ No newline at end of file
+}
Modified: poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFTextRun.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFTextRun.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFTextRun.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFTextRun.java Wed Oct 18 08:41:42 2017
@@ -845,7 +845,7 @@ public class XSLFTextRun implements Text
}
// SYMBOL is missing
- if (font == null || !font.isSetTypeface() || "".equals(font.getTypeface())) {
+ if (font == null || !font.isSetTypeface() || font.getTypeface().isEmpty()) {
font = coll.getLatin();
}
}
Modified: poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFPivotTable.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFPivotTable.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFPivotTable.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFPivotTable.java Wed Oct 18 08:41:42 2017
@@ -312,7 +312,7 @@ public class XSSFPivotTable extends POIX
pivotField.setAxis(STAxis.AXIS_COL);
pivotField.setShowAll(false);
- if (valueFormat != null && !"".equals(valueFormat.trim())) {
+ if (valueFormat != null && !valueFormat.trim().isEmpty()) {
DataFormat df = parentSheet.getWorkbook().createDataFormat();
pivotField.setNumFmtId(df.getFormat(valueFormat));
}
@@ -440,7 +440,7 @@ public class XSSFPivotTable extends POIX
cell.setCellType(CellType.STRING);
dataField.setName(valueFieldName);
dataField.setFld(columnIndex);
- if (valueFormat != null && !"".equals(valueFormat.trim())) {
+ if (valueFormat != null && !valueFormat.trim().isEmpty()) {
DataFormat df = parentSheet.getWorkbook().createDataFormat();
dataField.setNumFmtId(df.getFormat(valueFormat));
}
Modified: poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/helpers/XSSFPasswordHelper.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/helpers/XSSFPasswordHelper.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/helpers/XSSFPasswordHelper.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/helpers/XSSFPasswordHelper.java Wed Oct 18 08:41:42 2017
@@ -127,7 +127,7 @@ public final class XSSFPasswordHelper {
private static QName getAttrName(String prefix, String name) {
- if (prefix == null || "".equals(prefix)) {
+ if (prefix == null || prefix.isEmpty()) {
return new QName(name);
} else {
return new QName(prefix+Character.toUpperCase(name.charAt(0))+name.substring(1));
Modified: poi/trunk/src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java (original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hslf/extractor/PowerPointExtractor.java Wed Oct 18 08:41:42 2017
@@ -230,7 +230,7 @@ public final class PowerPointExtractor e
if(sh instanceof HSLFTextShape){
HSLFTextShape hsh = (HSLFTextShape)sh;
final String text = hsh.getText();
- if (text == null || "".equals(text) || "*".equals(text)) {
+ if (text == null || text.isEmpty() || "*".equals(text)) {
continue;
}
Modified: poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFFontInfo.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFFontInfo.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFFontInfo.java (original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFFontInfo.java Wed Oct 18 08:41:42 2017
@@ -125,7 +125,7 @@ public class HSLFFontInfo implements Fon
@Override
public void setTypeface(String typeface){
- if (typeface == null || "".equals(typeface)) {
+ if (typeface == null || typeface.isEmpty()) {
throw new IllegalArgumentException("typeface can't be null nor empty");
}
this.typeface = typeface;
Modified: poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSimpleShape.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSimpleShape.java?rev=1812476&r1=1812475&r2=1812476&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSimpleShape.java (original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSimpleShape.java Wed Oct 18 08:41:42 2017
@@ -323,7 +323,7 @@ public abstract class HSLFSimpleShape ex
}
name = name.replace("adj", "");
- if ("".equals(name)) {
+ if (name.isEmpty()) {
name = "1";
}
@@ -727,4 +727,4 @@ public abstract class HSLFSimpleShape ex
protected void setHyperlink(HSLFHyperlink link) {
_hyperlink = link;
}
-}
\ No newline at end of file
+}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org