You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2024/12/14 01:15:00 UTC

(commons-compress) branch master updated: Better parameter name

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-compress.git


The following commit(s) were added to refs/heads/master by this push:
     new 27385f5b0 Better parameter name
27385f5b0 is described below

commit 27385f5b08e9170ba086ee6d3a7c77510b31c79b
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Fri Dec 13 20:14:56 2024 -0500

    Better parameter name
    
    Better private instance variable name
---
 .../compress/compressors/gzip/GzipParameters.java        | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/main/java/org/apache/commons/compress/compressors/gzip/GzipParameters.java b/src/main/java/org/apache/commons/compress/compressors/gzip/GzipParameters.java
index 5e10d887b..be42e466c 100644
--- a/src/main/java/org/apache/commons/compress/compressors/gzip/GzipParameters.java
+++ b/src/main/java/org/apache/commons/compress/compressors/gzip/GzipParameters.java
@@ -290,7 +290,7 @@ public class GzipParameters {
      * means no time stamp is available.
      * </p>
      */
-    private Instant modificationTime = Instant.EPOCH;
+    private Instant modificationInstant = Instant.EPOCH;
     private ExtraField extraField;
     private String fileName;
     private Charset fileNameCharset = GzipUtils.GZIP_ENCODING;
@@ -396,7 +396,7 @@ public class GzipParameters {
      * @since 1.28.0
      */
     public Instant getModificationInstant() {
-        return modificationTime;
+        return modificationInstant;
     }
 
     /**
@@ -410,7 +410,7 @@ public class GzipParameters {
      * @return the most recent modification time in seconds since 00:00:00 GMT, Jan. 1, 1970.
      */
     public long getModificationTime() {
-        return modificationTime.getEpochSecond();
+        return modificationInstant.getEpochSecond();
     }
 
     /**
@@ -546,7 +546,7 @@ public class GzipParameters {
      * @since 1.28.0
      */
     public void setModificationInstant(final Instant modificationTime) {
-        this.modificationTime = modificationTime != null ? modificationTime : Instant.EPOCH;
+        this.modificationInstant = modificationTime != null ? modificationTime : Instant.EPOCH;
     }
 
     /**
@@ -557,10 +557,10 @@ public class GzipParameters {
      * means no time stamp is available.
      * </p>
      *
-     * @param modificationTime the modification time, in seconds.
+     * @param modificationTimeSeconds the modification time, in seconds.
      */
-    public void setModificationTime(final long modificationTime) {
-        this.modificationTime = Instant.ofEpochSecond(modificationTime);
+    public void setModificationTime(final long modificationTimeSeconds) {
+        this.modificationInstant = Instant.ofEpochSecond(modificationTimeSeconds);
     }
 
     /**
@@ -602,7 +602,7 @@ public class GzipParameters {
     @Override
     public String toString() {
         final StringBuilder builder = new StringBuilder();
-        builder.append("GzipParameters [compressionLevel=").append(compressionLevel).append(", modificationTime=").append(modificationTime)
+        builder.append("GzipParameters [compressionLevel=").append(compressionLevel).append(", modificationTime=").append(modificationInstant)
                 .append(", fileName=").append(fileName).append(", comment=").append(comment).append(", operatingSystem=").append(operatingSystem)
                 .append(", bufferSize=").append(bufferSize).append(", deflateStrategy=").append(deflateStrategy).append("]");
         return builder.toString();