Mark Dickinson <dicki...@gmail.com> added the comment:

One other note:  I find the bit numbering in find_last_set_bit peculiar:  isn't 
the least significant bit usually bit 0?  (Well, okay some people number the 
msb 0, but that's just weird. :)  I know the ffs and fls functions also start 
their bit numbering at one, but this seems very unconventional to me.

Perhaps rename find_last_set_bit to bit_length?  i.e., it's the *size* of the 
small bitfield that can contain the given value, rather than the index of the 
highest bit in that bitfield.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue8692>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to