Out of date comment?

Hey,
Thanks for publishing this! Currently reading through the code, and snagged on this comment:

Lines 142 to 146 in 633a6c1

    
           // Add it to the map. Since this is the first time, they can't possibly be 
        
           // rate limited yet, so return true. 
        
           s.data[key] = b 
        
           s.dataLock.Unlock() 
        
           return b.take()

While b.take() will return true, the comment is a bit confusing. I suspect it was written before that call was put in, I guess to avoid having an extra parameter to newBucket with "initial tokens"?

Oh yea, that's outdated. The original implementation didn't include "buckets".

This issue has been automatically locked since there has not been any
recent activity after it was closed. Please open a new issue for
related bugs.

	// Add it to the map. Since this is the first time, they can't possibly be
	// rate limited yet, so return true.
	s.data[key] = b
	s.dataLock.Unlock()
	return b.take()