twisted.protocols.htb.Bucket.drip() returns invalid results

[gsk]

From the api docs ( http://twistedmatrix.com/documents/current/api/twisted.protocols.htb.Bucket.html)

The method drip() of the Bucket class "Returns True if I am now empty."

The current drip() returns True only if the rate is zero (causing the bucket to drain instantaneously). This behavior does not take into account the possibility that a bucket will drain itself over time.

Attached is a patch containing a unit test for this issue and changes to the Bucket class that remedies this issue.

[gsk]

Unit test and patch to Bucket class

[jesstess]

(In [33448]) Branching to 'htb-drip-5448'

[jesstess]

(In [33449]) Apply bucket.patch by gsk.

refs #5448

[jesstess]

Thanks for the bug report and patch, gsk! I've applied your patch with a couple of cosmetic changes so that it conforms to our coding standards. (The code surrounding your changes is quite old and doesn't set a good example :) ) You can see the changes in revision r33450, and read our coding standard  here.

 Buildbot results look good.

I'll go ahead and merge.

Thanks again for the patch!

[jesstess]

(In [33452]) Merge htb-drip-5448

Author: gsk Review: jesstess Fixes: #5448

Fix twisted.protocols.htb.Bucket.drip to drip and empty properly with a non-zero drip rate.