You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by fa...@apache.org on 2022/07/17 08:45:38 UTC
svn commit: r1902801 - in /poi/trunk: poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/ poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/helpers/ poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/ poi-scratchpad/src/main/java/org/apa...
Author: fanningpj
Date: Sun Jul 17 08:45:38 2022
New Revision: 1902801
URL: http://svn.apache.org/viewvc?rev=1902801&view=rev
Log:
[github-354] Add extra methods to StringUtil and refactor some code to use them. Thanks to XenoAmess. This closes #354
Modified:
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSimpleShape.java
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/helpers/XSSFXmlColumnPr.java
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/Chunk.java
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/MAPIProperty.java
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/PropertiesChunk.java
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/Types.java
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/dev/TypesLister.java
poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hwpf/converter/WordToHtmlUtils.java
poi/trunk/poi-scratchpad/src/test/java/org/apache/poi/hwpf/TestFieldsTables.java
poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFPalette.java
poi/trunk/poi/src/main/java/org/apache/poi/ss/formula/functions/Bin2Dec.java
poi/trunk/poi/src/main/java/org/apache/poi/util/StringUtil.java
poi/trunk/poi/src/test/java/org/apache/poi/hssf/eventusermodel/TestMissingRecordAwareHSSFListener.java
poi/trunk/poi/src/test/java/org/apache/poi/ss/formula/functions/TestIrr.java
Modified: poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSimpleShape.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSimpleShape.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSimpleShape.java (original)
+++ poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSimpleShape.java Sun Jul 17 08:45:38 2022
@@ -398,14 +398,15 @@ public class XSSFSimpleShape extends XSS
* Convert an integer to its alpha equivalent e.g. 1 = A, 2 = B, 27 = AA etc
*/
private String valueToAlpha(int value) {
- String alpha = "";
+ StringBuilder alpha = new StringBuilder();
int modulo;
while (value > 0) {
modulo = (value - 1) % 26;
- alpha = (char) (65 + modulo) + alpha;
+ alpha.append((char) (65 + modulo));
value = (value - modulo) / 26;
}
- return alpha;
+ alpha.reverse();
+ return alpha.toString();
}
private static String[] _romanChars = new String[] { "M", "CM", "D", "CD", "C", "XC", "L", "XL", "X", "IX", "V",
Modified: poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/helpers/XSSFXmlColumnPr.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/helpers/XSSFXmlColumnPr.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/helpers/XSSFXmlColumnPr.java (original)
+++ poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/helpers/XSSFXmlColumnPr.java Sun Jul 17 08:45:38 2022
@@ -78,7 +78,7 @@ public class XSSFXmlColumnPr {
String[] xPathTokens = ctXmlColumnPr.getXpath().split("/");
for (int i = numberOfCommonXPathAxis; i < xPathTokens.length; i++) {
- localXPath.append("/" + xPathTokens[i]);
+ localXPath.append("/").append(xPathTokens[i]);
}
return localXPath.toString();
}
Modified: poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/Chunk.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/Chunk.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/Chunk.java (original)
+++ poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/Chunk.java Sun Jul 17 08:45:38 2022
@@ -23,6 +23,7 @@ import java.io.OutputStream;
import java.util.Locale;
import org.apache.poi.hsmf.datatypes.Types.MAPIType;
+import org.apache.poi.util.StringUtil;
public abstract class Chunk {
public static final String DEFAULT_NAME_PREFIX = "__substg1.0_";
@@ -62,13 +63,14 @@ public abstract class Chunk {
public String getEntryName() {
String type = this.type.asFileEnding();
- String chunkId = Integer.toHexString(this.chunkId);
- while (chunkId.length() < 4) {
- chunkId = "0" + chunkId;
+ StringBuilder chunkId = new StringBuilder(Integer.toHexString(this.chunkId));
+ int need0count = 4 - chunkId.length();
+ if (need0count > 0) {
+ chunkId.insert(0, StringUtil.repeat('0', need0count));
}
return this.namePrefix
- + chunkId.toUpperCase(Locale.ROOT)
+ + chunkId.toString().toUpperCase(Locale.ROOT)
+ type.toUpperCase(Locale.ROOT);
}
Modified: poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/MAPIProperty.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/MAPIProperty.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/MAPIProperty.java (original)
+++ poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/MAPIProperty.java Sun Jul 17 08:45:38 2022
@@ -17,6 +17,15 @@
package org.apache.poi.hsmf.datatypes;
+import java.util.Collection;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.Locale;
+import java.util.Map;
+
+import org.apache.poi.hsmf.datatypes.Types.MAPIType;
+import org.apache.poi.util.StringUtil;
+
import static org.apache.poi.hsmf.datatypes.Types.ASCII_STRING;
import static org.apache.poi.hsmf.datatypes.Types.BINARY;
import static org.apache.poi.hsmf.datatypes.Types.BOOLEAN;
@@ -27,14 +36,6 @@ import static org.apache.poi.hsmf.dataty
import static org.apache.poi.hsmf.datatypes.Types.SHORT;
import static org.apache.poi.hsmf.datatypes.Types.TIME;
-import java.util.Collection;
-import java.util.Collections;
-import java.util.HashMap;
-import java.util.Locale;
-import java.util.Map;
-
-import org.apache.poi.hsmf.datatypes.Types.MAPIType;
-
/**
* Holds the list of MAPI Attributes, and allows lookup by friendly name, ID and
* MAPI Property Name.
@@ -1069,9 +1070,10 @@ public class MAPIProperty {
}
public String asFileName() {
- String str = Integer.toHexString(id).toUpperCase(Locale.ROOT);
- while (str.length() < 4) {
- str = "0" + str;
+ StringBuilder str = new StringBuilder(Integer.toHexString(id).toUpperCase(Locale.ROOT));
+ int need0count = 4 - str.length();
+ if (need0count > 0) {
+ str.insert(0, StringUtil.repeat('0', need0count));
}
return str + usualType.asFileEnding();
}
Modified: poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/PropertiesChunk.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/PropertiesChunk.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/PropertiesChunk.java (original)
+++ poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/PropertiesChunk.java Sun Jul 17 08:45:38 2022
@@ -17,8 +17,6 @@
package org.apache.poi.hsmf.datatypes;
-import static org.apache.logging.log4j.util.Unbox.box;
-
import java.io.ByteArrayInputStream;
import java.io.IOException;
import java.io.InputStream;
@@ -48,6 +46,9 @@ import org.apache.poi.poifs.filesystem.D
import org.apache.poi.util.IOUtils;
import org.apache.poi.util.LittleEndian;
import org.apache.poi.util.LittleEndian.BufferUnderrunException;
+import org.apache.poi.util.StringUtil;
+
+import static org.apache.logging.log4j.util.Unbox.box;
/**
* <p>
@@ -419,9 +420,10 @@ public abstract class PropertiesChunk ex
}
private String getFileName(MAPIProperty property, MAPIType actualType) {
- String str = Integer.toHexString(property.id).toUpperCase(Locale.ROOT);
- while (str.length() < 4) {
- str = "0" + str;
+ StringBuilder str = new StringBuilder(Integer.toHexString(property.id).toUpperCase(Locale.ROOT));
+ int need0count = 4 - str.length();
+ if (need0count > 0) {
+ str.insert(0, StringUtil.repeat('0', need0count));
}
MAPIType type = getTypeMapping(actualType);
return str + type.asFileEnding();
Modified: poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/Types.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/Types.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/Types.java (original)
+++ poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/datatypes/Types.java Sun Jul 17 08:45:38 2022
@@ -21,6 +21,8 @@ import java.util.HashMap;
import java.util.Locale;
import java.util.Map;
+import org.apache.poi.util.StringUtil;
+
/**
* The types list and details are available from
* http://msdn.microsoft.com/en-us/library/microsoft.exchange.data.contenttypes.tnef.tnefpropertytype%28v=EXCHG.140%29.aspx
@@ -148,11 +150,12 @@ public final class Types {
}
public static String asFileEnding(int type) {
- String str = Integer.toHexString(type).toUpperCase(Locale.ROOT);
- while (str.length() < 4) {
- str = "0" + str;
+ StringBuilder str = new StringBuilder(Integer.toHexString(type).toUpperCase(Locale.ROOT));
+ int need0count = 4 - str.length();
+ if (need0count > 0) {
+ str.insert(0, StringUtil.repeat('0', need0count));
}
- return str;
+ return str.toString();
}
public static String asName(int typeId) {
Modified: poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/dev/TypesLister.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/dev/TypesLister.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/dev/TypesLister.java (original)
+++ poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hsmf/dev/TypesLister.java Sun Jul 17 08:45:38 2022
@@ -22,6 +22,7 @@ import java.util.ArrayList;
import java.util.Comparator;
import org.apache.poi.hsmf.datatypes.MAPIProperty;
+import org.apache.poi.util.StringUtil;
/**
* Lists the different MAPI types
@@ -49,8 +50,11 @@ public class TypesLister {
}
private void list(ArrayList<MAPIProperty> list, PrintStream out) {
for(MAPIProperty attr : list) {
- String id = Integer.toHexString(attr.id);
- while(id.length() < 4) { id = "0"+id; }
+ StringBuilder id = new StringBuilder(Integer.toHexString(attr.id));
+ int need0count = 4 - id.length();
+ if (need0count > 0) {
+ id.insert(0, StringUtil.repeat('0', need0count));
+ }
int typeId = attr.usualType.getId();
String typeIdStr = Integer.toString(typeId);
Modified: poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hwpf/converter/WordToHtmlUtils.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hwpf/converter/WordToHtmlUtils.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hwpf/converter/WordToHtmlUtils.java (original)
+++ poi/trunk/poi-scratchpad/src/main/java/org/apache/poi/hwpf/converter/WordToHtmlUtils.java Sun Jul 17 08:45:38 2022
@@ -29,7 +29,7 @@ public class WordToHtmlUtils extends Abs
{
public static void addBold( final boolean bold, StringBuilder style )
{
- style.append( "font-weight:" + ( bold ? "bold" : "normal" ) + ";" );
+ style.append("font-weight:").append(bold ? "bold" : "normal").append(";");
}
public static void addBorder( BorderCode borderCode, String where,
@@ -71,13 +71,11 @@ public class WordToHtmlUtils extends Abs
}
if ( characterRun.getIco24() != -1 )
{
- style.append( "color:" + getColor24( characterRun.getIco24() )
- + ";" );
+ style.append("color:").append(getColor24(characterRun.getIco24())).append(";");
}
if ( characterRun.isHighlighted() )
{
- style.append( "background-color:"
- + getColor( characterRun.getHighlightedColor() ) + ";" );
+ style.append("background-color:").append(getColor(characterRun.getHighlightedColor())).append(";");
}
if ( characterRun.isStrikeThrough() )
{
@@ -85,8 +83,7 @@ public class WordToHtmlUtils extends Abs
}
if ( characterRun.isShadowed() )
{
- style.append( "text-shadow:" + characterRun.getFontSize() / 24
- + "pt;" );
+ style.append("text-shadow:").append(characterRun.getFontSize() / 24).append("pt;");
}
if ( characterRun.isSmallCaps() )
{
@@ -118,12 +115,12 @@ public class WordToHtmlUtils extends Abs
if ( isEmpty( fontFamily ) )
return;
- style.append( "font-family:" + fontFamily + ";" );
+ style.append("font-family:").append(fontFamily).append(";");
}
public static void addFontSize( final int fontSize, StringBuilder style )
{
- style.append( "font-size:" + fontSize + "pt;" );
+ style.append("font-size:").append(fontSize).append("pt;");
}
public static void addIndent( Paragraph paragraph, StringBuilder style )
@@ -143,7 +140,7 @@ public class WordToHtmlUtils extends Abs
if ( twipsValue == 0 )
return;
- style.append( cssName + ":" + ( twipsValue / TWIPS_PER_INCH ) + "in;" );
+ style.append(cssName).append(":").append(twipsValue / TWIPS_PER_INCH).append("in;");
}
public static void addJustification( Paragraph paragraph,
@@ -151,7 +148,7 @@ public class WordToHtmlUtils extends Abs
{
String justification = getJustification( paragraph.getJustification() );
if ( isNotEmpty( justification ) )
- style.append( "text-align:" + justification + ";" );
+ style.append("text-align:").append(justification).append(";");
}
public static void addParagraphProperties( Paragraph paragraph,
@@ -170,8 +167,7 @@ public class WordToHtmlUtils extends Abs
style.append( "break-before:page;" );
}
- style.append( "hyphenate:"
- + ( paragraph.isAutoHyphenated() ? "auto" : "none" ) + ";" );
+ style.append("hyphenate:").append(paragraph.isAutoHyphenated() ? "auto" : "none").append(";");
if ( paragraph.keepOnPage() )
{
@@ -188,12 +184,9 @@ public class WordToHtmlUtils extends Abs
TableCell tableCell, boolean toppest, boolean bottomest,
boolean leftest, boolean rightest, StringBuilder style )
{
- style.append( "width:" + ( tableCell.getWidth() / TWIPS_PER_INCH )
- + "in;" );
- style.append( "padding-start:"
- + ( tableRow.getGapHalf() / TWIPS_PER_INCH ) + "in;" );
- style.append( "padding-end:"
- + ( tableRow.getGapHalf() / TWIPS_PER_INCH ) + "in;" );
+ style.append("width:").append(tableCell.getWidth() / TWIPS_PER_INCH).append("in;");
+ style.append("padding-start:").append(tableRow.getGapHalf() / TWIPS_PER_INCH).append("in;");
+ style.append("padding-end:").append(tableRow.getGapHalf() / TWIPS_PER_INCH).append("in;");
BorderCode top = tableCell.getBrcTop() != null
&& tableCell.getBrcTop().getBorderType() != 0 ? tableCell
@@ -224,8 +217,7 @@ public class WordToHtmlUtils extends Abs
{
if ( tableRow.getRowHeight() > 0 )
{
- style.append( "height:"
- + ( tableRow.getRowHeight() / TWIPS_PER_INCH ) + "in;" );
+ style.append("height:").append(tableRow.getRowHeight() / TWIPS_PER_INCH).append("in;");
}
if ( !tableRow.cantSplit() )
{
Modified: poi/trunk/poi-scratchpad/src/test/java/org/apache/poi/hwpf/TestFieldsTables.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi-scratchpad/src/test/java/org/apache/poi/hwpf/TestFieldsTables.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi-scratchpad/src/test/java/org/apache/poi/hwpf/TestFieldsTables.java (original)
+++ poi/trunk/poi-scratchpad/src/test/java/org/apache/poi/hwpf/TestFieldsTables.java Sun Jul 17 08:45:38 2022
@@ -83,7 +83,7 @@ public class TestFieldsTables extends HW
private String dumpPlexes(ArrayList<PlexOfField> fieldsPlexes) {
StringBuilder dump = new StringBuilder();
for (PlexOfField flds : fieldsPlexes) {
- dump.append(flds + "\n");
+ dump.append(flds).append("\n");
}
return dump.toString();
}
Modified: poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFPalette.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFPalette.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFPalette.java (original)
+++ poi/trunk/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFPalette.java Sun Jul 17 08:45:38 2022
@@ -22,6 +22,7 @@ import java.util.Locale;
import org.apache.poi.hssf.record.PaletteRecord;
import org.apache.poi.hssf.util.HSSFColor;
import org.apache.poi.hssf.util.HSSFColor.HSSFColorPredefined;
+import org.apache.poi.util.StringUtil;
/**
* Represents a workbook color palette.
@@ -218,22 +219,22 @@ public final class HSSFPalette {
private String getGnumericPart(byte color)
{
- String s;
+ StringBuilder s;
if (color == 0)
{
- s = "0";
+ s = new StringBuilder("0");
}
else
{
int c = color & 0xff; //as unsigned
c = (c << 8) | c; //pad to 16-bit
- s = Integer.toHexString(c).toUpperCase(Locale.ROOT);
- while (s.length() < 4)
- {
- s = "0" + s;
+ s = new StringBuilder(Integer.toHexString(c).toUpperCase(Locale.ROOT));
+ int need0count = 4 - s.length();
+ if (need0count > 0) {
+ s.insert(0, StringUtil.repeat('0', need0count));
}
}
- return s;
+ return s.toString();
}
}
}
Modified: poi/trunk/poi/src/main/java/org/apache/poi/ss/formula/functions/Bin2Dec.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi/src/main/java/org/apache/poi/ss/formula/functions/Bin2Dec.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi/src/main/java/org/apache/poi/ss/formula/functions/Bin2Dec.java (original)
+++ poi/trunk/poi/src/main/java/org/apache/poi/ss/formula/functions/Bin2Dec.java Sun Jul 17 08:45:38 2022
@@ -22,6 +22,7 @@ import org.apache.poi.ss.formula.eval.Nu
import org.apache.poi.ss.formula.eval.OperandResolver;
import org.apache.poi.ss.formula.eval.RefEval;
import org.apache.poi.ss.formula.eval.ValueEval;
+import org.apache.poi.util.StringUtil;
/**
* Implementation for Excel Bin2Dec() function.
@@ -109,9 +110,12 @@ public class Bin2Dec extends Fixed1ArgFu
private static String toggleBits(String s) {
long i = Long.parseLong(s, 2);
long i2 = i ^ ((1L << s.length()) - 1);
- String s2 = Long.toBinaryString(i2);
- while (s2.length() < s.length()) s2 = '0' + s2;
- return s2;
+ StringBuilder s2 = new StringBuilder(Long.toBinaryString(i2));
+ int need0count = s.length() - s2.length();
+ if (need0count > 0) {
+ s2.insert(0, StringUtil.repeat('0', need0count));
+ }
+ return s2.toString();
}
@Override
Modified: poi/trunk/poi/src/main/java/org/apache/poi/util/StringUtil.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi/src/main/java/org/apache/poi/util/StringUtil.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi/src/main/java/org/apache/poi/util/StringUtil.java (original)
+++ poi/trunk/poi/src/main/java/org/apache/poi/util/StringUtil.java Sun Jul 17 08:45:38 2022
@@ -21,6 +21,7 @@ import static java.nio.charset.StandardC
import java.nio.charset.Charset;
import java.nio.charset.StandardCharsets;
+import java.util.Arrays;
import java.util.Locale;
/**
@@ -759,4 +760,34 @@ public final class StringUtil {
return !isBlank(cs);
}
+ /**
+ * <p>Returns padding using the specified delimiter repeated
+ * to a given length.</p>
+ *
+ * <pre>
+ * StringUtil.repeat('e', 0) = ""
+ * StringUtil.repeat('e', 3) = "eee"
+ * StringUtil.repeat('e', -2) = ""
+ * </pre>
+ *
+ * <p>Note: this method does not support padding with
+ * <a href="http://www.unicode.org/glossary/#supplementary_character">Unicode Supplementary Characters</a>
+ * as they require a pair of {@code char}s to be represented.
+ * </p>
+ *
+ * copied from commons-lang3
+ *
+ * @param ch character to repeat
+ * @param repeat number of times to repeat char, negative treated as zero
+ * @return String with repeated character
+ */
+ public static String repeat(final char ch, final int repeat) {
+ if (repeat <= 0) {
+ return "";
+ }
+ final char[] buf = new char[repeat];
+ Arrays.fill(buf, ch);
+ return new String(buf);
+ }
+
}
Modified: poi/trunk/poi/src/test/java/org/apache/poi/hssf/eventusermodel/TestMissingRecordAwareHSSFListener.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi/src/test/java/org/apache/poi/hssf/eventusermodel/TestMissingRecordAwareHSSFListener.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi/src/test/java/org/apache/poi/hssf/eventusermodel/TestMissingRecordAwareHSSFListener.java (original)
+++ poi/trunk/poi/src/test/java/org/apache/poi/hssf/eventusermodel/TestMissingRecordAwareHSSFListener.java Sun Jul 17 08:45:38 2022
@@ -212,7 +212,7 @@ final class TestMissingRecordAwareHSSFLi
dig = "ur";
}
if (lastRow != row && row > -1) {
- sb.append((lastRow > -1 ? "," : "") + row + ":");
+ sb.append(lastRow > -1 ? "," : "").append(row).append(":");
lastRow = row;
}
sb.append(dig);
Modified: poi/trunk/poi/src/test/java/org/apache/poi/ss/formula/functions/TestIrr.java
URL: http://svn.apache.org/viewvc/poi/trunk/poi/src/test/java/org/apache/poi/ss/formula/functions/TestIrr.java?rev=1902801&r1=1902800&r2=1902801&view=diff
==============================================================================
--- poi/trunk/poi/src/test/java/org/apache/poi/ss/formula/functions/TestIrr.java (original)
+++ poi/trunk/poi/src/test/java/org/apache/poi/ss/formula/functions/TestIrr.java Sun Jul 17 08:45:38 2022
@@ -106,7 +106,7 @@ final class TestIrr {
assertFormulaResult(cv, cellA);
} catch (Throwable e){
if(failures.length() > 0) failures.append('\n');
- failures.append("Row[" + (cellA.getRowIndex() + 1)+ "]: " + cellA.getCellFormula() + " ");
+ failures.append("Row[").append(cellA.getRowIndex() + 1).append("]: ").append(cellA.getCellFormula()).append(" ");
failures.append(e.getMessage());
}
@@ -116,7 +116,7 @@ final class TestIrr {
assertEquals(0, cv.getNumberValue(), 0.0001); // should agree within 0.01%
} catch (Throwable e){
if(failures.length() > 0) failures.append('\n');
- failures.append("Row[" + (cellC.getRowIndex() + 1)+ "]: " + cellC.getCellFormula() + " ");
+ failures.append("Row[").append(cellC.getRowIndex() + 1).append("]: ").append(cellC.getCellFormula()).append(" ");
failures.append(e.getMessage());
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org