This was a small nitpick, agreed.  I just think a comment should be more
than restating the obvious, e.g., "Ensure we capture subsecond intervals"
or some such, that's all.

Adam


On Tue, Jan 21, 2014 at 12:12 PM, CBWhiz <notifications@github.com> wrote:

> In fsq/ratelimit.py:
>
> > +       """
> > +
> > +    def __init__(self, limit_qty, limit_period, iterator, clock_func=None, sleep_func=None, allow_negative_sleep=False):
> > +        if not clock_func:
> > +            clock_func = time.time
> > +        if not sleep_func:
> > +            sleep_func = time.sleep
> > +
> > +        self.limit_qty = limit_qty
> > +        self.limit_period = limit_period
> > +        self.iterator = iterator
> > +        self.clock_func = clock_func
> > +        self.sleep_func = sleep_func
> > +        self.allow_negative_sleep = allow_negative_sleep
> > +
> > +        self.interval = limit_period / float(limit_qty) #Float division
>
> The reason the comment is here is to point out that we're making sure we
> don't do integer division.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/axialmarket/fsq/pull/6/files#r9044711>
> .
>