You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by ce...@apache.org on 2013/12/26 18:54:49 UTC
svn commit: r1553522 -
/poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/MemoryPackagePart.java
Author: centic
Date: Thu Dec 26 17:54:49 2013
New Revision: 1553522
URL: http://svn.apache.org/r1553522
Log:
Remove unnecessary separate length, an array already has one
Modified:
poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/MemoryPackagePart.java
Modified: poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/MemoryPackagePart.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/MemoryPackagePart.java?rev=1553522&r1=1553521&r2=1553522&view=diff
==============================================================================
--- poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/MemoryPackagePart.java (original)
+++ poi/trunk/src/ooxml/java/org/apache/poi/openxml4j/opc/internal/MemoryPackagePart.java Thu Dec 26 17:54:49 2013
@@ -45,11 +45,6 @@ public final class MemoryPackagePart ext
protected byte[] data;
/**
- * Size of data.
- */
- protected int length;
-
- /**
* Constructor.
*
* @param pack
@@ -104,12 +99,11 @@ public final class MemoryPackagePart ext
@Override
public long getSize() {
- return length;
+ return data == null ? 0 : data.length;
}
public void clear() {
data = null;
- length = 0;
}
@Override
@@ -129,7 +123,6 @@ public final class MemoryPackagePart ext
// Save it
data = baos.toByteArray();
- length = data.length;
// All done
return true;
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org