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 2016/05/16 14:44:47 UTC

commons-compress git commit: Document that ctor call is OK

Repository: commons-compress
Updated Branches:
  refs/heads/master f59dd65ca -> 14b3d1ad8


Document that ctor call is OK


Project: http://git-wip-us.apache.org/repos/asf/commons-compress/repo
Commit: http://git-wip-us.apache.org/repos/asf/commons-compress/commit/14b3d1ad
Tree: http://git-wip-us.apache.org/repos/asf/commons-compress/tree/14b3d1ad
Diff: http://git-wip-us.apache.org/repos/asf/commons-compress/diff/14b3d1ad

Branch: refs/heads/master
Commit: 14b3d1ad8c5efc1f21c94e6d0f72635c374b2bb6
Parents: f59dd65
Author: Sebb <se...@apache.org>
Authored: Mon May 16 15:44:32 2016 +0100
Committer: Sebb <se...@apache.org>
Committed: Mon May 16 15:44:32 2016 +0100

----------------------------------------------------------------------
 .../commons/compress/compressors/snappy/PureJavaCrc32C.java       | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/commons-compress/blob/14b3d1ad/src/main/java/org/apache/commons/compress/compressors/snappy/PureJavaCrc32C.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/commons/compress/compressors/snappy/PureJavaCrc32C.java b/src/main/java/org/apache/commons/compress/compressors/snappy/PureJavaCrc32C.java
index 611aeff..cb735d5 100644
--- a/src/main/java/org/apache/commons/compress/compressors/snappy/PureJavaCrc32C.java
+++ b/src/main/java/org/apache/commons/compress/compressors/snappy/PureJavaCrc32C.java
@@ -39,7 +39,7 @@ final class PureJavaCrc32C implements Checksum {
 
   /** Create a new PureJavaCrc32 object. */
   public PureJavaCrc32C() {
-    reset();
+    reset(); // non-private, but the class is now final
   }
 
   @Override
@@ -49,6 +49,7 @@ final class PureJavaCrc32C implements Checksum {
   }
 
   @Override
+  // called by ctor but the class is final so this is safe
   public void reset() {
     crc = 0xffffffff;
   }