(tomcat) branch 10.1.x updated: Avoid useless add operation

[remm]

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

remm pushed a commit to branch 10.1.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/10.1.x by this push:
     new 9aedadc2ab Avoid useless add operation
9aedadc2ab is described below

commit 9aedadc2ab50ebe9e954ff8ee34e5bb8e0ac1112
Author: remm <r...@apache.org>
AuthorDate: Tue Feb 13 09:49:20 2024 +0100

    Avoid useless add operation
---
 .../apache/coyote/http11/upgrade/UpgradeGroupInfo.java   | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/java/org/apache/coyote/http11/upgrade/UpgradeGroupInfo.java 
b/java/org/apache/coyote/http11/upgrade/UpgradeGroupInfo.java
index dc44062461..638d0d23d2 100644
--- a/java/org/apache/coyote/http11/upgrade/UpgradeGroupInfo.java
+++ b/java/org/apache/coyote/http11/upgrade/UpgradeGroupInfo.java
@@ -61,7 +61,9 @@ public class UpgradeGroupInfo extends BaseModelMBean {
     }
     public void setBytesReceived(long bytesReceived) {
         deadBytesReceived.reset();
-        deadBytesReceived.add(bytesReceived);
+        if (bytesReceived != 0) {
+            deadBytesReceived.add(bytesReceived);
+        }
         for (UpgradeInfo ui : upgradeInfos) {
             ui.setBytesReceived(bytesReceived);
         }
@@ -77,7 +79,9 @@ public class UpgradeGroupInfo extends BaseModelMBean {
     }
     public void setBytesSent(long bytesSent) {
         deadBytesSent.reset();
-        deadBytesSent.add(bytesSent);
+        if (bytesSent != 0) {
+            deadBytesSent.add(bytesSent);
+        }
         for (UpgradeInfo ui : upgradeInfos) {
             ui.setBytesSent(bytesSent);
         }
@@ -93,7 +97,9 @@ public class UpgradeGroupInfo extends BaseModelMBean {
     }
     public void setMsgsReceived(long msgsReceived) {
         deadMsgsReceived.reset();
-        deadMsgsReceived.add(msgsReceived);
+        if (msgsReceived != 0) {
+            deadMsgsReceived.add(msgsReceived);
+        }
         for (UpgradeInfo ui : upgradeInfos) {
             ui.setMsgsReceived(msgsReceived);
         }
@@ -109,7 +115,9 @@ public class UpgradeGroupInfo extends BaseModelMBean {
     }
     public void setMsgsSent(long msgsSent) {
         deadMsgsSent.reset();
-        deadMsgsSent.add(msgsSent);
+        if (msgsSent != 0) {
+            deadMsgsSent.add(msgsSent);
+        }
         for (UpgradeInfo ui : upgradeInfos) {
             ui.setMsgsSent(msgsSent);
         }


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