db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dyre.Tjeldv...@Sun.COM
Subject Some questions about FormatableBitSet
Date Sun, 10 Dec 2006 15:50:16 GMT
There are a couple of things in the implementation of this class that
I just don't understand:

- The doc for the method grow() says:
  "ASSUMPTIONS: that all extra bits in the last byte are zero". But in
  other methods (or() and()...) great care is taken mask out the bits
  of the last byte that isn't part of the bitset. But if it is an
  invariant of the class that these unused bits are zero, it should be
  possible to do most operations byte by byte with no special handling
  of the last byte, right?

- in or() (but not in and() or xor()) an explicit test is done on
  whether the argument of type FormatableBitSet is "instanceof
  FormatableBitSet". Will it not always be? It could have been a
  subclass I suppose, but since FormatableBitSet is final that becomes
  a bit hypothetical.

- A minor nit: This class appears to do a lot of division and
  remainder calculations with powers of 2 (frequently 8). Is it not
  generally preferably to do this with shifts and  bitwise ANDs?

- The algorithm for finding the number of bits set seems a bit
  simple (iterating over all bits, and counting those for which isSet()
  returns true). A simple google search will give a number of ways of
  calculating the number of bits in a byte quickly. Is there a reason
  not to use such an algorithm and iterate over the bytes, rather than
  the bits?


View raw message