lgoldstein commented on code in PR #446:
URL: https://github.com/apache/mina-sshd/pull/446#discussion_r1435249686


##########
sshd-core/src/main/java/org/apache/sshd/common/session/helpers/AbstractSession.java:
##########
@@ -202,13 +213,20 @@ public abstract class AbstractSession extends 
SessionHelper {
      * Rekeying
      */
     protected final AtomicLong inPacketsCount = new AtomicLong(0L);
+    protected final AtomicLong totalIncomingPacketsCount = new AtomicLong(0L);
     protected final AtomicLong outPacketsCount = new AtomicLong(0L);
+    protected final AtomicLong totalOutgingPacketsCount = new AtomicLong(0L);
     protected final AtomicLong inBytesCount = new AtomicLong(0L);
+    protected final AtomicLong totalIncomingBytesCount = new AtomicLong(0L);
     protected final AtomicLong outBytesCount = new AtomicLong(0L);
+    protected final AtomicLong totalOutgoingBytesCount = new AtomicLong(0L);
     protected final AtomicLong inBlocksCount = new AtomicLong(0L);
+    protected final AtomicLong totalIncomingBlocksCount = new AtomicLong(0L);
     protected final AtomicLong outBlocksCount = new AtomicLong(0L);
+    protected final AtomicLong totalOutgoingBlocksCount = new AtomicLong(0L);
     protected final AtomicReference<Instant> lastKeyTimeValue = new 
AtomicReference<>(Instant.now());
-    // we initialize them here in case super constructor calls some methods 
that use these values
+    // we initialize them here in case super constructor calls some methods 
that
+    // use these values

Review Comment:
   I do not format these comments this way - we have a Maven plugin (that 
someone else configured) and this is its result.



##########
sshd-core/src/main/java/org/apache/sshd/common/session/helpers/AbstractSession.java:
##########
@@ -471,7 +490,8 @@ public MacInformation getMacInformation(boolean incoming) {
     public void messageReceived(Readable buffer) throws Exception {
         synchronized (decodeLock) {
             decoderBuffer.putBuffer(buffer);
-            // One of those properties will be set by the constructor and the 
other
+            // One of those properties will be set by the constructor and the
+            // other

Review Comment:
   I do not format these comments this way - we have a Maven plugin (that 
someone else configured) and this is its result.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org
For additional commands, e-mail: dev-h...@mina.apache.org

Reply via email to