You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by se...@apache.org on 2014/12/29 14:46:31 UTC

svn commit: r1648344 - /commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/jpeg/iptc/IptcBlock.java

Author: sebb
Date: Mon Dec 29 13:46:30 2014
New Revision: 1648344

URL: http://svn.apache.org/r1648344
Log:
IMAGING-117 Reduce visibility of mutable array

Modified:
    commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/jpeg/iptc/IptcBlock.java

Modified: commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/jpeg/iptc/IptcBlock.java
URL: http://svn.apache.org/viewvc/commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/jpeg/iptc/IptcBlock.java?rev=1648344&r1=1648343&r2=1648344&view=diff
==============================================================================
--- commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/jpeg/iptc/IptcBlock.java (original)
+++ commons/proper/imaging/trunk/src/main/java/org/apache/commons/imaging/formats/jpeg/iptc/IptcBlock.java Mon Dec 29 13:46:30 2014
@@ -21,8 +21,9 @@ package org.apache.commons.imaging.forma
  */
 public class IptcBlock {
     public final int blockType;
-    public final byte[] blockNameBytes;
-    public final byte[] blockData;
+    //  Only currently used by classes in the package
+    final byte[] blockNameBytes; // TODO make private and provide getter?
+    final byte[] blockData; // TODO make private and provide getter?
 
     public IptcBlock(final int blockType, final byte[] blockNameBytes, final byte[] blockData) {
         this.blockData = blockData;