Skip to content

Minor fix.#34

Closed
lemire wants to merge 2 commits intomasterfrom
dlemire/minorfix
Closed

Minor fix.#34
lemire wants to merge 2 commits intomasterfrom
dlemire/minorfix

Conversation

@lemire
Copy link
Copy Markdown
Member

@lemire lemire commented Mar 23, 2022

In some cases, our code could produce infinite loops.

@lemire lemire requested a review from thomasmueller March 23, 2022 15:46
Comment thread fastfilter/src/main/java/org/fastfilter/xor/XorBinaryFuse8.java
Comment thread fastfilter/src/main/java/org/fastfilter/xor/XorBinaryFuse8.java Outdated
@lemire
Copy link
Copy Markdown
Member Author

lemire commented Mar 23, 2022

I do not have a test case. The reporter could not provide a test case.

Comment thread fastfilter/src/main/java/org/fastfilter/xor/XorBinaryFuse8.java
@thomasmueller
Copy link
Copy Markdown
Contributor

Ah, we have the "mainloop" label now; I assume this fixes the issue as well.

@lemire lemire closed this Jan 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants