Claudenw commented on code in PR #492:
URL: 
https://github.com/apache/commons-collections/pull/492#discussion_r1616637135


##########
src/main/java/org/apache/commons/collections4/bloomfilter/package-info.java:
##########
@@ -56,20 +56,21 @@
  *
  *     <li>Extractor - The Extractors are {@code FunctionalInterfaces} that 
are conceptually iterators on a {@code BitMap}, an {@code Index}, or a
  *     collection of {@code Cell}s, with an early termination switch.  
Extractors have
- *     names like {@code BitMapExtractor} or {@code IndexExtractor} and  have 
a {@code processXs} methods that take a
+ *     names like {@code BitMapExtractor} or {@code IndexExtractor} and have a 
{@code processXs} methods that take a
  *     {@code Predicate<X>} argument (e.g. {@code 
processBitMaps(LongPredicate)} or {@code processIndicies(IntPredicate)}).
  *     That predicate is expected to process each of the Xs in turn and return 
{@code true} if the processing should continue
  *     or {@code false} to stop it. </li>
  * </ul>
  *
- * <p>There is an obvious association between the BitMap and the Index in that 
if bit 5 is enabled in the BitMap than the index must contain the index 5.</p>
+ * <p>There is an obvious association between the BitMap and the Index in that 
if bit 5 is enabled in the BitMap than the indices must contain the

Review Comment:
   I think I managed to clarify it.  These are in the Nomenclature section and 
are referring to the definitions just provided.  It is to provide some 
background for the naming of classes like `IndexExtractor` and 
`BitMapExtractor` and what they mean conceptually.



-- 
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