Claudenw commented on PR #501:
URL: 
https://github.com/apache/commons-collections/pull/501#issuecomment-2160622978

   We do have coverage in the tests for the bad case.
   
   If we start an integer variable `int tet = 1;` and use it on line 198 rather 
than i.  We can then do 
   ```
   if (++tet == bits) {
       tet = 0;
   }
   ```
   to ensure that tet is always in the proper range.  Conceptually this is the 
same as calling modulus bits.  This means that we only have one loop to worry 
about.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to