[ https://issues.apache.org/jira/browse/LUCENE-3179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13054681#comment-13054681 ]
Uwe Schindler commented on LUCENE-3179: --------------------------------------- One more comment: When working on the code, the symmetry all other methods have between long and int is broken here. For consistency we should add the long method, too. I just don't like the missing consistency. Also: OpenBitSet.nextSetBit() does not use Long.numberOfTrailingZeroes() but the new prevSetBit() does. As both methods have intrinsics, why only use one of them? Yonik? Any comments? > OpenBitSet.prevSetBit() > ----------------------- > > Key: LUCENE-3179 > URL: https://issues.apache.org/jira/browse/LUCENE-3179 > Project: Lucene - Java > Issue Type: Improvement > Reporter: Paul Elschot > Assignee: Paul Elschot > Priority: Minor > Fix For: 3.3, 4.0 > > Attachments: LUCENE-3179-fix.patch, LUCENE-3179-fix.patch, > LUCENE-3179.patch, LUCENE-3179.patch, LUCENE-3179.patch, TestBitUtil.java, > TestOpenBitSet.patch > > > Find a previous set bit in an OpenBitSet. > Useful for parent testing in nested document query execution LUCENE-2454 . -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org