Author: fanningpj
Date: Thu Dec 17 21:03:27 2020
New Revision: 1884572
URL: http://svn.apache.org/viewvc?rev=1884572&view=rev
Log:
avoid string concats
Modified:
poi/trunk/src/java/org/apache/poi/POIDocument.java
poi/trunk/src/java/org/apache/poi/hpsf/CustomProperties.java
poi/trunk/src/java/org/apache/poi/hssf/dev/BiffViewer.java
poi/trunk/src/java/org/apache/poi/hssf/eventusermodel/FormatTrackingHSSFListener.java
poi/trunk/src/java/org/apache/poi/hssf/record/EmbeddedObjectRefSubRecord.java
poi/trunk/src/java/org/apache/poi/hssf/record/FeatRecord.java
poi/trunk/src/java/org/apache/poi/hssf/record/SSTDeserializer.java
poi/trunk/src/java/org/apache/poi/poifs/crypt/ChunkedCipherOutputStream.java
poi/trunk/src/java/org/apache/poi/poifs/crypt/standard/StandardEncryptor.java
poi/trunk/src/java/org/apache/poi/ss/util/DateFormatConverter.java
poi/trunk/src/java/org/apache/poi/util/Configurator.java
poi/trunk/src/java/org/apache/poi/util/IOUtils.java
poi/trunk/src/java/org/apache/poi/util/SystemOutLogger.java
poi/trunk/src/ooxml/java/org/apache/poi/ooxml/POIXMLDocumentPart.java
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/OPCPackage.java
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/PackageRelationshipCollection.java
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/ZipPackage.java
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/ZipContentTypeManager.java
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/marshallers/ZipPartMarshaller.java
poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/RelationshipTransformService.java
poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/TSPTimeStampService.java
poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFPropertiesDelegate.java
poi/trunk/src/ooxml/java/org/apache/poi/xssf/eventusermodel/XSSFSheetXMLHandler.java
poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SheetDataWriter.java
poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/chunks/ChunkFactory.java
poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/streams/ChunkStream.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/blip/PICT.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/AnimationInfo.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExEmbed.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExHyperlink.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExVideoContainer.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/Sound.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/TextRulerAtom.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFGroupShape.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFPictureShape.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShow.java
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShowImpl.java
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/datatypes/AttachmentChunks.java
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/datatypes/RecipientChunks.java
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/parsers/POIFSChunkParser.java
poi/trunk/src/scratchpad/src/org/apache/poi/hwpf/sprm/ParagraphSprmUncompressor.java
Modified: poi/trunk/src/java/org/apache/poi/POIDocument.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/POIDocument.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/POIDocument.java (original)
+++ poi/trunk/src/java/org/apache/poi/POIDocument.java Thu Dec 17 21:03:27 2020
@@ -333,9 +333,9 @@ public abstract class POIDocument implem
// Create or Update the Property Set stream in the POIFS
outFS.createOrUpdateDocument(bIn, name);
- logger.log(POILogger.INFO, "Wrote property set " + name + " of
size " + data.length);
+ logger.log(POILogger.INFO, "Wrote property set ", name, " of size
", data.length);
} catch(WritingNotSupportedException ignored) {
- logger.log( POILogger.ERROR, "Couldn't write property set with
name " + name + " as not supported by HPSF yet");
+ logger.log( POILogger.ERROR, "Couldn't write property set with
name ", name, " as not supported by HPSF yet");
}
}
Modified: poi/trunk/src/java/org/apache/poi/hpsf/CustomProperties.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/hpsf/CustomProperties.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/hpsf/CustomProperties.java (original)
+++ poi/trunk/src/java/org/apache/poi/hpsf/CustomProperties.java Thu Dec 17
21:03:27 2020
@@ -393,7 +393,7 @@ public class CustomProperties implements
try {
cps = CodePageUtil.codepageToEncoding(cp, false);
} catch (UnsupportedEncodingException e) {
- LOG.log(POILogger.ERROR, "Codepage '"+cp+"' can't be found.");
+ LOG.log(POILogger.ERROR, "Codepage '", cp, "' can't be found.");
}
if (!cps.isEmpty() &&
Charset.forName(cps).newEncoder().canEncode(value)) {
return;
Modified: poi/trunk/src/java/org/apache/poi/hssf/dev/BiffViewer.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/hssf/dev/BiffViewer.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/hssf/dev/BiffViewer.java (original)
+++ poi/trunk/src/java/org/apache/poi/hssf/dev/BiffViewer.java Thu Dec 17
21:03:27 2020
@@ -81,7 +81,7 @@ public final class BiffViewer {
try {
hasNext = recStream.hasNextRecord();
} catch (LeftoverDataException e) {
- logger.log(POILogger.ERROR, "Discarding " +
recStream.remaining() + " bytes and continuing", e);
+ logger.log(POILogger.ERROR, "Discarding ",
recStream.remaining(), " bytes and continuing", e);
recStream.readRemainder();
hasNext = recStream.hasNextRecord();
}
Modified:
poi/trunk/src/java/org/apache/poi/hssf/eventusermodel/FormatTrackingHSSFListener.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/hssf/eventusermodel/FormatTrackingHSSFListener.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/java/org/apache/poi/hssf/eventusermodel/FormatTrackingHSSFListener.java
(original)
+++
poi/trunk/src/java/org/apache/poi/hssf/eventusermodel/FormatTrackingHSSFListener.java
Thu Dec 17 21:03:27 2020
@@ -155,8 +155,8 @@ public class FormatTrackingHSSFListener
if (formatIndex >=
HSSFDataFormat.getNumberOfBuiltinBuiltinFormats()) {
FormatRecord tfr =
_customFormatRecords.get(Integer.valueOf(formatIndex));
if (tfr == null) {
- logger.log( POILogger.ERROR, "Requested format
at index " + formatIndex
- + ", but it wasn't found");
+ logger.log( POILogger.ERROR, "Requested format
at index ", formatIndex,
+ ", but it wasn't found");
} else {
format = tfr.getFormatString();
}
@@ -192,8 +192,8 @@ public class FormatTrackingHSSFListener
public int getFormatIndex(CellValueRecordInterface cell) {
ExtendedFormatRecord xfr = _xfRecords.get(cell.getXFIndex());
if (xfr == null) {
- logger.log( POILogger.ERROR, "Cell " + cell.getRow() +
"," + cell.getColumn()
- + " uses XF with index " +
cell.getXFIndex() + ", but we don't have that");
+ logger.log( POILogger.ERROR, "Cell ", cell.getRow(),
",", cell.getColumn(),
+ " uses XF with index ",
cell.getXFIndex(), ", but we don't have that");
return -1;
}
return xfr.getFormatIndex();
Modified:
poi/trunk/src/java/org/apache/poi/hssf/record/EmbeddedObjectRefSubRecord.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/hssf/record/EmbeddedObjectRefSubRecord.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/java/org/apache/poi/hssf/record/EmbeddedObjectRefSubRecord.java
(original)
+++
poi/trunk/src/java/org/apache/poi/hssf/record/EmbeddedObjectRefSubRecord.java
Thu Dec 17 21:03:27 2020
@@ -157,7 +157,7 @@ public final class EmbeddedObjectRefSubR
int nUnexpectedPadding = remaining - dataLenAfterFormula;
if (nUnexpectedPadding > 0) {
- logger.log( POILogger.ERROR, "Discarding " +
nUnexpectedPadding + " unexpected padding bytes ");
+ logger.log( POILogger.ERROR, "Discarding ",
nUnexpectedPadding, " unexpected padding bytes");
readRawData(in, nUnexpectedPadding);
remaining-=nUnexpectedPadding;
}
Modified: poi/trunk/src/java/org/apache/poi/hssf/record/FeatRecord.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/hssf/record/FeatRecord.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/hssf/record/FeatRecord.java (original)
+++ poi/trunk/src/java/org/apache/poi/hssf/record/FeatRecord.java Thu Dec 17
21:03:27 2020
@@ -108,7 +108,7 @@ public final class FeatRecord extends St
sharedFeature = new FeatSmartTag(in);
break;
default:
- logger.log( POILogger.ERROR, "Unknown Shared Feature "
+ isf_sharedFeatureType + " found!");
+ logger.log( POILogger.ERROR, "Unknown Shared Feature ",
isf_sharedFeatureType, " found!");
}
}
Modified: poi/trunk/src/java/org/apache/poi/hssf/record/SSTDeserializer.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/hssf/record/SSTDeserializer.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/hssf/record/SSTDeserializer.java
(original)
+++ poi/trunk/src/java/org/apache/poi/hssf/record/SSTDeserializer.java Thu Dec
17 21:03:27 2020
@@ -50,18 +50,18 @@ class SSTDeserializer
for (int i=0;i<stringCount;i++) {
// Extract exactly the count of strings from the SST record.
UnicodeString str;
- if(in.available() == 0 && ! in.hasNextRecord()) {
- logger.log( POILogger.ERROR, "Ran out of data before creating
all the strings! String at index " + i + "");
- str = new UnicodeString("");
- } else {
- str = new UnicodeString(in);
- }
+ if (in.available() == 0 && !in.hasNextRecord()) {
+ logger.log(POILogger.ERROR, "Ran out of data before creating all
the strings! String at index ", i);
+ str = new UnicodeString("");
+ } else {
+ str = new UnicodeString(in);
+ }
addToStringTable( strings, str );
}
}
static public void addToStringTable( IntMapper<UnicodeString> strings,
UnicodeString string )
{
- strings.add(string);
+ strings.add(string);
}
}
Modified:
poi/trunk/src/java/org/apache/poi/poifs/crypt/ChunkedCipherOutputStream.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/poifs/crypt/ChunkedCipherOutputStream.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/java/org/apache/poi/poifs/crypt/ChunkedCipherOutputStream.java
(original)
+++
poi/trunk/src/java/org/apache/poi/poifs/crypt/ChunkedCipherOutputStream.java
Thu Dec 17 21:03:27 2020
@@ -324,7 +324,7 @@ public abstract class ChunkedCipherOutpu
}
if (!fileOut.delete()) {
- LOG.log(POILogger.ERROR, "Can't delete temporary
encryption file: "+fileOut);
+ LOG.log(POILogger.ERROR, "Can't delete temporary
encryption file: ", fileOut);
}
} catch (IOException e) {
throw new EncryptedDocumentException(e);
Modified:
poi/trunk/src/java/org/apache/poi/poifs/crypt/standard/StandardEncryptor.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/poifs/crypt/standard/StandardEncryptor.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/java/org/apache/poi/poifs/crypt/standard/StandardEncryptor.java
(original)
+++
poi/trunk/src/java/org/apache/poi/poifs/crypt/standard/StandardEncryptor.java
Thu Dec 17 21:03:27 2020
@@ -193,7 +193,7 @@ public class StandardEncryptor extends E
IOUtils.copy(fis, leos);
}
if (!fileOut.delete()) {
- logger.log(POILogger.ERROR, "Can't delete temporary
encryption file: "+fileOut);
+ logger.log(POILogger.ERROR, "Can't delete temporary
encryption file: ", fileOut);
}
leos.close();
Modified: poi/trunk/src/java/org/apache/poi/ss/util/DateFormatConverter.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/ss/util/DateFormatConverter.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/ss/util/DateFormatConverter.java
(original)
+++ poi/trunk/src/java/org/apache/poi/ss/util/DateFormatConverter.java Thu Dec
17 21:03:27 2020
@@ -152,7 +152,7 @@ public final class DateFormatConverter
loc =
LocaleID.lookupByLanguageTag(languageTag.substring(0, idx));
}
if (loc == null) {
- logger.log( POILogger.ERROR, "Unable to find
prefix for Locale '" + languageTag + "' or its parent locales." );
+ logger.log( POILogger.ERROR, "Unable to find
prefix for Locale '", languageTag, "' or its parent locales." );
return "";
}
}
Modified: poi/trunk/src/java/org/apache/poi/util/Configurator.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/util/Configurator.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/util/Configurator.java (original)
+++ poi/trunk/src/java/org/apache/poi/util/Configurator.java Thu Dec 17
21:03:27 2020
@@ -29,7 +29,7 @@ public class Configurator {
try {
result = Integer.parseInt(property);
} catch (Exception e) {
- logger.log(POILogger.ERROR, "System property -D"+systemProperty +"
do not contains a valid integer " + property);
+ logger.log(POILogger.ERROR, "System property -D", systemProperty,
" does not contains a valid integer: ", property);
}
return result;
}
Modified: poi/trunk/src/java/org/apache/poi/util/IOUtils.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/util/IOUtils.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/util/IOUtils.java (original)
+++ poi/trunk/src/java/org/apache/poi/util/IOUtils.java Thu Dec 17 21:03:27 2020
@@ -394,7 +394,7 @@ public final class IOUtils {
try {
closeable.close();
} catch ( Exception exc ) {
- logger.log( POILogger.ERROR, "Unable to close resource: " + exc,
+ logger.log( POILogger.ERROR, "Unable to close resource: ", exc,
exc );
}
}
Modified: poi/trunk/src/java/org/apache/poi/util/SystemOutLogger.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/util/SystemOutLogger.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/util/SystemOutLogger.java (original)
+++ poi/trunk/src/java/org/apache/poi/util/SystemOutLogger.java Thu Dec 17
21:03:27 2020
@@ -60,7 +60,12 @@ public class SystemOutLogger implements
if (!check(level)) {
return;
}
- System.out.println("[" + _cat + "]" +
LEVEL_STRINGS_SHORT.charAt(Math.min(LEVEL_STRINGS_SHORT.length()-1, level)) + "
" + obj1);
+ System.out.print("[");
+ System.out.print(_cat);
+ System.out.print("]");
+
System.out.print(LEVEL_STRINGS_SHORT.charAt(Math.min(LEVEL_STRINGS_SHORT.length()-1,
level)));
+ System.out.print(" ");
+ System.out.println(obj1);
if (exception != null) {
exception.printStackTrace(System.out);
}
Modified: poi/trunk/src/ooxml/java/org/apache/poi/ooxml/POIXMLDocumentPart.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/ooxml/POIXMLDocumentPart.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/ooxml/POIXMLDocumentPart.java
(original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/ooxml/POIXMLDocumentPart.java Thu
Dec 17 21:03:27 2020
@@ -651,7 +651,7 @@ public class POIXMLDocumentPart {
final PackagePart p =
packagePart.getPackage().getPart(relName);
if (p == null) {
- logger.log(POILogger.ERROR, "Skipped invalid entry " +
rel.getTargetURI());
+ logger.log(POILogger.ERROR, "Skipped invalid entry ",
rel.getTargetURI());
continue;
}
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=1884572&r1=1884571&r2=1884572&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 Thu
Dec 17 21:03:27 2020
@@ -1006,9 +1006,8 @@ public abstract class OPCPackage impleme
sourcePartName =
PackagingURIHelper.createPartName(sourceURI);
} catch (InvalidFormatException e) {
logger
- .log(POILogger.ERROR, "Part
name URI '"
- + sourceURI
- + "' is not
valid ! This message is not intended to be displayed !");
+ .log(POILogger.ERROR, "Part
name URI '", sourceURI,
+ "' is not valid
! This message is not intended to be displayed !");
return;
}
if (sourcePartName.getURI().equals(
Modified:
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/PackageRelationshipCollection.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/PackageRelationshipCollection.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/PackageRelationshipCollection.java
(original)
+++
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/PackageRelationshipCollection.java
Thu Dec 17 21:03:27 2020
@@ -348,8 +348,8 @@ public final class PackageRelationshipCo
// package
target = PackagingURIHelper.toURI(value);
} catch (URISyntaxException e) {
- logger.log(POILogger.ERROR, "Cannot convert " + value
- + " in a valid relationship URI-> dummy-URI used",
e);
+ logger.log(POILogger.ERROR, "Cannot convert ", value,
+ " in a valid relationship URI-> dummy-URI used",
e);
}
addRelationship(target, targetMode, type, id);
}
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=1884572&r1=1884571&r2=1884572&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 Thu
Dec 17 21:03:27 2020
@@ -145,7 +145,7 @@ public final class ZipPackage extends OP
throw new InvalidOperationException("Can't open the specified
file: '" + file + "'", e);
}
- LOG.log(POILogger.ERROR, "Error in zip file "+file+" - falling
back to stream processing (i.e. ignoring zip central directory)");
+ LOG.log(POILogger.ERROR, "Error in zip file ", file, " - falling
back to stream processing (i.e. ignoring zip central directory)");
ze = openZipEntrySourceStream(file);
}
this.zipArchive = ze;
Modified:
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/ZipContentTypeManager.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/ZipContentTypeManager.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/ZipContentTypeManager.java
(original)
+++
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/ZipContentTypeManager.java
Thu Dec 17 21:03:27 2020
@@ -71,8 +71,8 @@ public class ZipContentTypeManager exten
zos.closeArchiveEntry();
}
} catch (IOException ioe) {
- logger.log(POILogger.ERROR, "Cannot write: " +
CONTENT_TYPES_PART_NAME
- + " in Zip !", ioe);
+ logger.log(POILogger.ERROR, "Cannot write: ",
CONTENT_TYPES_PART_NAME,
+ " in Zip !", ioe);
return false;
}
}
Modified:
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/marshallers/ZipPartMarshaller.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/marshallers/ZipPartMarshaller.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/marshallers/ZipPartMarshaller.java
(original)
+++
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/marshallers/ZipPartMarshaller.java
Thu Dec 17 21:03:27 2020
@@ -64,7 +64,7 @@ public final class ZipPartMarshaller imp
public boolean marshall(PackagePart part, OutputStream os)
throws OpenXML4JException {
if (!(os instanceof ZipArchiveOutputStream)) {
- logger.log(POILogger.ERROR,"Unexpected class " +
os.getClass().getName());
+ logger.log(POILogger.ERROR,"Unexpected class ",
os.getClass().getName());
throw new OpenXML4JException("ZipOutputStream expected
!");
// Normally should happen only in development phase, so
just throw
// exception
@@ -91,7 +91,7 @@ public final class ZipPartMarshaller imp
zos.closeArchiveEntry();
}
} catch (IOException ioe) {
- logger.log(POILogger.ERROR,"Cannot write: " +
part.getPartName() + ": in ZIP",
+ logger.log(POILogger.ERROR,"Cannot write: ",
part.getPartName(), ": in ZIP",
ioe);
return false;
}
@@ -186,7 +186,7 @@ public final class ZipPartMarshaller imp
zos.closeArchiveEntry();
}
} catch (IOException e) {
- logger.log(POILogger.ERROR,"Cannot create zip entry " +
relPartName, e);
+ logger.log(POILogger.ERROR,"Cannot create zip entry ",
relPartName, e);
return false;
}
}
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=1884572&r1=1884571&r2=1884572&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
Thu Dec 17 21:03:27 2020
@@ -138,7 +138,7 @@ public class RelationshipTransformServic
@Override
public void init(XMLStructure parent, XMLCryptoContext context) throws
InvalidAlgorithmParameterException {
LOG.log(POILogger.DEBUG, "init(parent,context)");
- LOG.log(POILogger.DEBUG, "parent java type: " +
parent.getClass().getName());
+ LOG.log(POILogger.DEBUG, "parent java type: ",
parent.getClass().getName());
DOMStructure domParent = (DOMStructure) parent;
Node parentNode = domParent.getNode();
Modified:
poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/TSPTimeStampService.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/TSPTimeStampService.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/TSPTimeStampService.java
(original)
+++
poi/trunk/src/ooxml/java/org/apache/poi/poifs/crypt/dsig/services/TSPTimeStampService.java
Thu Dec 17 21:03:27 2020
@@ -151,8 +151,8 @@ public class TSPTimeStampService impleme
int statusCode = huc.getResponseCode();
if (statusCode != 200) {
- LOG.log(POILogger.ERROR, "Error contacting TSP server ",
signatureConfig.getTspUrl() +
- ", had status code " + statusCode + "/" +
huc.getResponseMessage());
+ LOG.log(POILogger.ERROR, "Error contacting TSP server ",
signatureConfig.getTspUrl(),
+ ", had status code ", statusCode, "/",
huc.getResponseMessage());
throw new IOException("Error contacting TSP server " +
signatureConfig.getTspUrl() +
", had status code " + statusCode + "/" +
huc.getResponseMessage());
}
Modified:
poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFPropertiesDelegate.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFPropertiesDelegate.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFPropertiesDelegate.java
(original)
+++
poi/trunk/src/ooxml/java/org/apache/poi/xslf/usermodel/XSLFPropertiesDelegate.java
Thu Dec 17 21:03:27 2020
@@ -1848,7 +1848,7 @@ import org.openxmlformats.schemas.presen
} else if (props instanceof CTTextCharacterProperties) {
obj = new TextCharDelegate((CTTextCharacterProperties)props);
} else {
- LOG.log(POILogger.ERROR, props.getClass() +" is an unknown
properties type");
+ LOG.log(POILogger.ERROR, props.getClass(), " is an unknown
properties type");
return null;
}
Modified:
poi/trunk/src/ooxml/java/org/apache/poi/xssf/eventusermodel/XSSFSheetXMLHandler.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xssf/eventusermodel/XSSFSheetXMLHandler.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/ooxml/java/org/apache/poi/xssf/eventusermodel/XSSFSheetXMLHandler.java
(original)
+++
poi/trunk/src/ooxml/java/org/apache/poi/xssf/eventusermodel/XSSFSheetXMLHandler.java
Thu Dec 17 21:03:27 2020
@@ -360,7 +360,7 @@ public class XSSFSheetXMLHandler extends
thisStr = rtss.toString();
}
catch (NumberFormatException ex) {
- logger.log(POILogger.ERROR, "Failed to parse SST index
'" + sstIndex, ex);
+ logger.log(POILogger.ERROR, "Failed to parse SST index
'", sstIndex, ex);
}
break;
Modified:
poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SheetDataWriter.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SheetDataWriter.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SheetDataWriter.java
(original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SheetDataWriter.java
Thu Dec 17 21:03:27 2020
@@ -186,7 +186,7 @@ public class SheetDataWriter implements
@Override
protected void finalize() throws Throwable {
if (_fd.exists() && !_fd.delete()) {
- logger.log(POILogger.ERROR, "Can't delete temporary encryption
file: "+_fd);
+ logger.log(POILogger.ERROR, "Can't delete temporary encryption
file: ", _fd);
}
}
Modified: poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFSheet.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
(original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFSheet.java Thu
Dec 17 21:03:27 2020
@@ -557,7 +557,7 @@ public class XSSFSheet extends POIXMLDoc
break;
}
}
- logger.log(POILogger.ERROR, "Can't find drawing with id=" +
ctDrawing.getId() + " in the list of the sheet's relationships");
+ logger.log(POILogger.ERROR, "Can't find drawing with id=",
ctDrawing.getId(), " in the list of the sheet's relationships");
}
return null;
}
@@ -629,7 +629,7 @@ public class XSSFSheet extends POIXMLDoc
}
}
if(drawing == null){
- logger.log(POILogger.ERROR, "Can't find VML drawing with id="
+ id + " in the list of the sheet's relationships");
+ logger.log(POILogger.ERROR, "Can't find VML drawing with id=",
id, " in the list of the sheet's relationships");
}
}
return drawing;
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/chunks/ChunkFactory.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/chunks/ChunkFactory.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/chunks/ChunkFactory.java
(original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/chunks/ChunkFactory.java
Thu Dec 17 21:03:27 2020
@@ -165,7 +165,7 @@ public final class ChunkFactory {
data, endOfDataPos);
endOfDataPos += 8;
} else {
- logger.log(POILogger.ERROR, "Header claims a
length to " + endOfDataPos + " there's then no space for the trailer in the
data (" + data.length + ")");
+ logger.log(POILogger.ERROR, "Header claims a
length to ", endOfDataPos, " there's then no space for the trailer in the data
(", data.length, ")");
}
}
if(header.hasSeparator()) {
@@ -173,7 +173,7 @@ public final class ChunkFactory {
separator = new ChunkSeparator(
data, endOfDataPos);
} else {
- logger.log(POILogger.ERROR, "Header claims a
length to " + endOfDataPos + " there's then no space for the separator in the
data (" + data.length + ")");
+ logger.log(POILogger.ERROR, "Header claims a
length to ", endOfDataPos, " there's then no space for the separator in the
data (", data.length, ")");
}
}
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/streams/ChunkStream.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/streams/ChunkStream.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/streams/ChunkStream.java
(original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hdgf/streams/ChunkStream.java
Thu Dec 17 21:03:27 2020
@@ -73,7 +73,7 @@ public final class ChunkStream extends S
}
catch (Exception e)
{
- logger.log(POILogger.ERROR, "Failed to create chunk at
" + pos + ", ignoring rest of data." + e);
+ logger.log(POILogger.ERROR, "Failed to create chunk at
", pos, ", ignoring rest of data." + e);
}
chunks = chunksA.toArray(new Chunk[0]);
Modified: poi/trunk/src/scratchpad/src/org/apache/poi/hslf/blip/PICT.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/blip/PICT.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hslf/blip/PICT.java (original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hslf/blip/PICT.java Thu Dec 17
21:03:27 2020
@@ -85,7 +85,7 @@ public final class PICT extends Metafile
out.write(chunk, 0, lastLen);
}
// End of picture marker for PICT is 0x00 0xFF
- LOG.log(POILogger.ERROR, "PICT zip-stream is invalid, read as much
as possible. Uncompressed length of header: " + header.getWmfSize() + " / Read
bytes: " + out.size(), e);
+ LOG.log(POILogger.ERROR, "PICT zip-stream is invalid, read as much
as possible. Uncompressed length of header: ", header.getWmfSize(), " / Read
bytes: ", out.size(), e);
}
return out.toByteArray();
}
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/AnimationInfo.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/AnimationInfo.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/AnimationInfo.java
(original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/AnimationInfo.java
Thu Dec 17 21:03:27 2020
@@ -58,7 +58,7 @@ public final class AnimationInfo extends
if(_children[0] instanceof AnimationInfoAtom) {
animationAtom = (AnimationInfoAtom)_children[0];
} else {
- logger.log(POILogger.ERROR, "First child record wasn't
a AnimationInfoAtom, was of type " + _children[0].getRecordType());
+ logger.log(POILogger.ERROR, "First child record wasn't
a AnimationInfoAtom, was of type ", _children[0].getRecordType());
}
}
Modified: poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExEmbed.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExEmbed.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExEmbed.java
(original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExEmbed.java Thu
Dec 17 21:03:27 2020
@@ -105,14 +105,14 @@ public class ExEmbed extends RecordConta
if(_children[0] instanceof ExEmbedAtom) {
embedAtom = (ExEmbedAtom)_children[0];
} else {
- logger.log(POILogger.ERROR, "First child record wasn't a
ExEmbedAtom, was of type " + _children[0].getRecordType());
+ logger.log(POILogger.ERROR, "First child record wasn't a
ExEmbedAtom, was of type ", _children[0].getRecordType());
}
// Second child should be the ExOleObjAtom
if (_children[1] instanceof ExOleObjAtom) {
oleObjAtom = (ExOleObjAtom)_children[1];
} else {
- logger.log(POILogger.ERROR, "Second child record wasn't a
ExOleObjAtom, was of type " + _children[1].getRecordType());
+ logger.log(POILogger.ERROR, "Second child record wasn't a
ExOleObjAtom, was of type ", _children[1].getRecordType());
}
for (int i = 2; i < _children.length; i++) {
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExHyperlink.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExHyperlink.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExHyperlink.java
(original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExHyperlink.java
Thu Dec 17 21:03:27 2020
@@ -118,7 +118,7 @@ public class ExHyperlink extends RecordC
if(_children[0] instanceof ExHyperlinkAtom) {
linkAtom = (ExHyperlinkAtom)_children[0];
} else {
- logger.log(POILogger.ERROR, "First child record wasn't
a ExHyperlinkAtom, was of type " + _children[0].getRecordType());
+ logger.log(POILogger.ERROR, "First child record wasn't
a ExHyperlinkAtom, was of type ", _children[0].getRecordType());
}
for (int i = 1; i < _children.length; i++) {
@@ -126,7 +126,7 @@ public class ExHyperlink extends RecordC
if ( linkDetailsA == null) linkDetailsA =
(CString)_children[i];
else linkDetailsB = (CString)_children[i];
} else {
- logger.log(POILogger.ERROR, "Record after ExHyperlinkAtom
wasn't a CString, was of type " + _children[1].getRecordType());
+ logger.log(POILogger.ERROR, "Record after ExHyperlinkAtom
wasn't a CString, was of type ", _children[1].getRecordType());
}
}
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExVideoContainer.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExVideoContainer.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExVideoContainer.java
(original)
+++
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/ExVideoContainer.java
Thu Dec 17 21:03:27 2020
@@ -60,12 +60,12 @@ public final class ExVideoContainer exte
if(_children[0] instanceof ExMediaAtom) {
mediaAtom = (ExMediaAtom)_children[0];
} else {
- logger.log(POILogger.ERROR, "First child record wasn't
a ExMediaAtom, was of type " + _children[0].getRecordType());
+ logger.log(POILogger.ERROR, "First child record wasn't
a ExMediaAtom, was of type ", _children[0].getRecordType());
}
if(_children[1] instanceof CString) {
pathAtom = (CString)_children[1];
} else {
- logger.log(POILogger.ERROR, "Second child record wasn't a CString,
was of type " + _children[1].getRecordType());
+ logger.log(POILogger.ERROR, "Second child record wasn't a CString,
was of type ", _children[1].getRecordType());
}
}
Modified: poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/Sound.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/Sound.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/Sound.java
(original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/Sound.java Thu Dec
17 21:03:27 2020
@@ -68,14 +68,14 @@ public final class Sound extends RecordC
if(_children[0] instanceof CString) {
_name = (CString)_children[0];
} else {
- logger.log(POILogger.ERROR, "First child record wasn't a CString,
was of type " + _children[0].getRecordType());
+ logger.log(POILogger.ERROR, "First child record wasn't a CString,
was of type ", _children[0].getRecordType());
}
// Second child should be the ExOleObjAtom
if (_children[1] instanceof CString) {
_type = (CString)_children[1];
} else {
- logger.log(POILogger.ERROR, "Second child record wasn't a CString,
was of type " + _children[1].getRecordType());
+ logger.log(POILogger.ERROR, "Second child record wasn't a CString,
was of type ", _children[1].getRecordType());
}
for (int i = 2; i < _children.length; i++) {
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/TextRulerAtom.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/TextRulerAtom.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
--- poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/TextRulerAtom.java
(original)
+++ poi/trunk/src/scratchpad/src/org/apache/poi/hslf/record/TextRulerAtom.java
Thu Dec 17 21:03:27 2020
@@ -98,7 +98,7 @@ public final class TextRulerAtom extends
// Get the record data.
read(leis);
} catch (IOException e){
- logger.log(POILogger.ERROR, "Failed to parse TextRulerAtom: " +
e.getMessage());
+ logger.log(POILogger.ERROR, "Failed to parse TextRulerAtom: ",
e.getMessage());
}
}
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFGroupShape.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFGroupShape.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFGroupShape.java
(original)
+++
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFGroupShape.java
Thu Dec 17 21:03:27 2020
@@ -289,7 +289,7 @@ implements HSLFShapeContainer, GroupShap
} else {
// Should we do anything special with these non
// Container records?
- LOG.log(POILogger.ERROR, "Shape contained non container escher
record, was " + r.getClass().getName());
+ LOG.log(POILogger.ERROR, "Shape contained non container escher
record, was ", r.getClass().getName());
}
}
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFPictureShape.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFPictureShape.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFPictureShape.java
(original)
+++
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFPictureShape.java
Thu Dec 17 21:03:27 2020
@@ -127,7 +127,7 @@ public class HSLFPictureShape extends HS
return pd;
}
}
- LOG.log(POILogger.ERROR, "no picture found for our BSE offset " +
bse.getOffset());
+ LOG.log(POILogger.ERROR, "no picture found for our BSE offset ",
bse.getOffset());
}
return null;
}
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShow.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShow.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShow.java
(original)
+++
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShow.java
Thu Dec 17 21:03:27 2020
@@ -290,8 +290,7 @@ public final class HSLFSlideShow extends
return _mostRecentCoreRecords[coreRecordId];
}
logger.log(POILogger.ERROR,
- "We tried to look up a reference to a core
record, but there was no core ID for reference ID "
- + refID);
+ "We tried to look up a reference to a core
record, but there was no core ID for reference ID ", refID);
return null;
}
@@ -388,7 +387,7 @@ public final class HSLFSlideShow extends
// Ensure it really is a notes record
if (!(r instanceof Notes)) {
- logger.log(POILogger.ERROR, loggerLoc+", but that was actually
a " + r);
+ logger.log(POILogger.ERROR, loggerLoc, ", but that was
actually a ", r);
continue;
}
@@ -420,10 +419,10 @@ public final class HSLFSlideShow extends
// Ensure it really is a slide record
if (!(r instanceof Slide)) {
- logger.log(POILogger.ERROR, "A Slide SlideAtomSet at " + idx
- + " said its record was at refID "
- + spa.getRefID()
- + ", but that was actually a " + r);
+ logger.log(POILogger.ERROR, "A Slide SlideAtomSet at ", idx,
+ " said its record was at refID ",
+ spa.getRefID(),
+ ", but that was actually a ", r);
continue;
}
@@ -439,7 +438,7 @@ public final class HSLFSlideShow extends
if (notesPos != null && 0 <= notesPos && notesPos <
_notes.size()) {
notes = _notes.get(notesPos);
} else {
- logger.log(POILogger.ERROR, "Notes not found for noteId="
+ noteId);
+ logger.log(POILogger.ERROR, "Notes not found for noteId=",
noteId);
}
}
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShowImpl.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShowImpl.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShowImpl.java
(original)
+++
poi/trunk/src/scratchpad/src/org/apache/poi/hslf/usermodel/HSLFSlideShowImpl.java
Thu Dec 17 21:03:27 2020
@@ -330,7 +330,7 @@ public final class HSLFSlideShowImpl ext
try {
currentUser = new CurrentUserAtom(getDirectory());
} catch (IOException ie) {
- logger.log(POILogger.ERROR, "Error finding Current User Atom:\n" +
ie);
+ logger.log(POILogger.ERROR, "Error finding Current User Atom", ie);
currentUser = new CurrentUserAtom();
}
}
@@ -394,8 +394,8 @@ public final class HSLFSlideShowImpl ext
// If they type (including the bonus 0xF018) is 0, skip it
PictureType pt = PictureType.forNativeID(type - 0xF018);
if (pt == null) {
- logger.log(POILogger.ERROR, "Problem reading picture:
Invalid image type 0, on picture with length " + imgsize + ".\nYou document
will probably become corrupted if you save it!");
- logger.log(POILogger.ERROR, "" + pos);
+ logger.log(POILogger.ERROR, "Problem reading picture:
Invalid image type 0, on picture with length ", imgsize, ".\nYou document will
probably become corrupted if you save it!");
+ logger.log(POILogger.ERROR, "position: ", pos);
} else {
//The pictstream can be truncated halfway through a
picture.
//This is not a problem if the pictstream contains extra
pictures
@@ -418,7 +418,7 @@ public final class HSLFSlideShowImpl ext
pict.setIndex(_pictures.size());
_pictures.add(pict);
} catch (IllegalArgumentException e) {
- logger.log(POILogger.ERROR, "Problem reading picture:
" + e + "\nYou document will probably become corrupted if you save it!");
+ logger.log(POILogger.ERROR, "Problem reading picture:
", e, "\nYou document will probably become corrupted if you save it!");
}
}
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/datatypes/AttachmentChunks.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/datatypes/AttachmentChunks.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/datatypes/AttachmentChunks.java
(original)
+++
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/datatypes/AttachmentChunks.java
Thu Dec 17 21:03:27 2020
@@ -187,7 +187,7 @@ public class AttachmentChunks implements
} else if (chunk instanceof DirectoryChunk) {
attachmentDirectory = (DirectoryChunk) chunk;
} else {
- LOG.log(POILogger.ERROR, "Unexpected data chunk of type " +
chunk.getEntryName());
+ LOG.log(POILogger.ERROR, "Unexpected data chunk of type ",
chunk.getEntryName());
}
} else if (chunkId == ATTACH_EXTENSION.id) {
attachExtension = (StringChunk) chunk;
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/datatypes/RecipientChunks.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/datatypes/RecipientChunks.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/datatypes/RecipientChunks.java
(original)
+++
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/datatypes/RecipientChunks.java
Thu Dec 17 21:03:27 2020
@@ -88,8 +88,7 @@ public final class RecipientChunks imple
try {
recipientNumber = Integer.parseInt(number, 16);
} catch (NumberFormatException e) {
- LOG.log(POILogger.ERROR,
- "Invalid recipient number in name " + name);
+ LOG.log(POILogger.ERROR, "Invalid recipient number in name ",
name);
}
}
}
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/parsers/POIFSChunkParser.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/parsers/POIFSChunkParser.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/parsers/POIFSChunkParser.java
(original)
+++
poi/trunk/src/scratchpad/src/org/apache/poi/hsmf/parsers/POIFSChunkParser.java
Thu Dec 17 21:03:27 2020
@@ -149,7 +149,7 @@ public final class POIFSChunkParser {
try (DocumentInputStream inp = new
DocumentInputStream((DocumentNode) entry)) {
chunk.readValue(inp);
} catch (IOException e) {
- LOG.log(POILogger.ERROR, "Error reading from part " +
entry.getName(), e);
+ LOG.log(POILogger.ERROR, "Error reading from part ",
entry.getName(), e);
}
}
Modified:
poi/trunk/src/scratchpad/src/org/apache/poi/hwpf/sprm/ParagraphSprmUncompressor.java
URL:
http://svn.apache.org/viewvc/poi/trunk/src/scratchpad/src/org/apache/poi/hwpf/sprm/ParagraphSprmUncompressor.java?rev=1884572&r1=1884571&r2=1884572&view=diff
==============================================================================
---
poi/trunk/src/scratchpad/src/org/apache/poi/hwpf/sprm/ParagraphSprmUncompressor.java
(original)
+++
poi/trunk/src/scratchpad/src/org/apache/poi/hwpf/sprm/ParagraphSprmUncompressor.java
Thu Dec 17 21:03:27 2020
@@ -73,8 +73,7 @@ public final class ParagraphSprmUncompre
{
logger.log(
POILogger.ERROR,
- "Unable to apply SPRM operation '"
- + sprm.getOperation() + "': ", exc );
+ "Unable to apply SPRM operation '", sprm.getOperation(),
"': ", exc );
}
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
