[ 
https://issues.apache.org/jira/browse/COLLECTIONS-728?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16964893#comment-16964893
 ] 

Claude Warren commented on COLLECTIONS-728:
-------------------------------------------

Quick note:

I somewhat get the rationale until here:
 === begin quote ===
    True the Hasher could be limited to a StaticHasher and the Shape would not 
be needed as the Static Hahser contains the Shape it was built with, but in the 
general case a Hasher does not have a Shape.

Where it seems that the API could be improved so as to avoid having to pass 
null when the "shape" argument is unused.

=== end quote ===

The API _always_ requires both arguments.  For the static hasher the call would 
look like `filter.merge( staticHasher.getShape(), staticHasher )`

I will comment on other bits later when I have more time.


> BloomFilter contribution
> ------------------------
>
>                 Key: COLLECTIONS-728
>                 URL: https://issues.apache.org/jira/browse/COLLECTIONS-728
>             Project: Commons Collections
>          Issue Type: Task
>            Reporter: Claude Warren
>            Priority: Minor
>         Attachments: BF_Func.md, BloomFilter.java, BloomFilterI2.java, 
> Usage.md
>
>
> Contribution of BloomFilter library comprising base implementation and gated 
> collections.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to