https://issues.apache.org/bugzilla/show_bug.cgi?id=45753





--- Comment #2 from Chad LaVigne <chad_lavi...@yahoo.com>  2009-03-23 14:56:31 
PST ---
Created an attachment (id=23401)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=23401)
Update to BurstFilter contribution

The attached code addresses the following issues mentioned by Curt.  

1) License headers are fixed
2) @Author tags removed
3) JavaDoc comments added to all methods
4) Filter is now level aware
5) BurstFilter & TokenBucket have been condensed into one class
6) TokenBucket.getToken() now returns false if empty 

I didn't rename the filter because I actually submitted this code about 2 years
ago under the name TokenBucketFilter and it was requested at that time that I
rename it to something like BurstFilter so that it would be implementation
agnostic.  Let me know what the consensus is, I can certainly rename it if need
be.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

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

Reply via email to