andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #347 from fredericBregier/3

Browse Source 
3 Proposal to fix 1rst issue presented in #345
This commit is contained in:
Frédéric Brégier 2012-05-19 08:29:32 -07:00
commit 35d6d682df
2 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/org/jboss/netty/handler/traffic/AbstractTrafficShapingHandler.java
View File

@ -358,7 +358,7 @@ public abstract class AbstractTrafficShapingHandler extends
// Time is too short, so just lets continue
return 0;
}
return bytes * 1000 / limit - interval;
return ((bytes * 1000 / limit - interval)/10)*10;
}
@Override

7
src/main/java/org/jboss/netty/handler/traffic/TrafficCounter.java
View File

@ -247,9 +247,10 @@ public class TrafficCounter {
* @param newcheckInterval
*/
public void configure(long newcheckInterval) {
if (checkInterval.get() != newcheckInterval) {
checkInterval.set(newcheckInterval);
if (newcheckInterval <= 0) {
long newInterval = (newcheckInterval/10)*10;
if (checkInterval.get() != newInterval) {
checkInterval.set(newInterval);
if (newInterval <= 0) {
stop();
// No more active monitoring
lastTime.set(System.currentTimeMillis());