Cleanup comments

redis · Jan 4, 2024 · 24036dd · 24036dd
1 parent 265310a
commit 24036dd
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 7 deletions.
diff --git a/src/main/java/redis/clients/jedis/Connection.java b/src/main/java/redis/clients/jedis/Connection.java
@@ -352,12 +352,7 @@ protected Object readProtocolWithCheckingBroken() {
     }
 
     try {
-//      Protocol.readPushes(inputStream, clientSideCache);
-//      return Protocol.read(inputStream);
       return Protocol.read(inputStream, clientSideCache);
-//      Object read = Protocol.read(inputStream);
-//      System.out.println("REPLY: " + SafeEncoder.encodeObject(read));
-//      return read;
     } catch (JedisConnectionException exc) {
       broken = true;
       throw exc;

diff --git a/src/main/java/redis/clients/jedis/JedisClientSideCache.java b/src/main/java/redis/clients/jedis/JedisClientSideCache.java
@@ -33,7 +33,6 @@ private void clientTrackingOn() {
 
   @Override
   public String get(String key) {
-    //connection.readPushesWithCheckingBroken();
     String cachedValue = cache.getValue(key);
     if (cachedValue != null) return cachedValue;
 

diff --git a/src/main/java/redis/clients/jedis/util/RedisInputStream.java b/src/main/java/redis/clients/jedis/util/RedisInputStream.java
@@ -44,7 +44,7 @@ public RedisInputStream(InputStream in) {
   }
 
   public boolean peek(byte b) throws JedisConnectionException {
-    ensureFill(); // in current design we are expecting at least a reply. so not doing ensureFillSafe()
+    ensureFill(); // in current design, at least one reply is expected. so ensureFillSafe() is not necessary.
     return buf[count] == b;
   }