From a57408ebfcb6e4c22b2edd868c8656d4f62ac8cc Mon Sep 17 00:00:00 2001
From: kerr <hepin@sosia.us>
Date: Fri, 19 Sep 2014 20:42:14 +0800
Subject: [PATCH] Change the operator order of TrafficCounter to calculate the
 throughput to get the correct result

Motivation:
Currently the last read/write throughput is calculated by first division,this will be 0 if the last read/write bytes < interval,change the order will get the correct result

Modifications:
Change the operator order from first do division to multiplication

Result:
Get the correct result instead of 0 when bytes are smaller than interval
---
 .../main/java/io/netty/handler/traffic/TrafficCounter.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/handler/src/main/java/io/netty/handler/traffic/TrafficCounter.java b/handler/src/main/java/io/netty/handler/traffic/TrafficCounter.java
index dedd07e020..d465c80ef5 100644
--- a/handler/src/main/java/io/netty/handler/traffic/TrafficCounter.java
+++ b/handler/src/main/java/io/netty/handler/traffic/TrafficCounter.java
@@ -233,9 +233,9 @@ public class TrafficCounter {
         }
         lastReadBytes = currentReadBytes.getAndSet(0);
         lastWrittenBytes = currentWrittenBytes.getAndSet(0);
-        lastReadThroughput = lastReadBytes / interval * 1000;
+        lastReadThroughput = lastReadBytes * 1000 / interval;
         // nb byte / checkInterval in ms * 1000 (1s)
-        lastWriteThroughput = lastWrittenBytes / interval * 1000;
+        lastWriteThroughput = lastWrittenBytes * 1000 / interval;
         // nb byte / checkInterval in ms * 1000 (1s)
         if (lastWrittenBytes > 0) {
             lastNonNullWrittenBytes = lastWrittenBytes;