dgryski / go-trigram

Small trigram indexer
BSD 2-Clause "Simplified" License
34 stars 6 forks source link

we only need to check for equality #12

Closed Dieterbe closed 8 years ago

Dieterbe commented 8 years ago

the value becoming greater than len is not a concern because the code catches all cases where it's incremented until it equals.

This PR may be more about pedantery and personal preference rather than any technical merit. I looked at the asm code and the only difference is seteq vs setge instructions.

either way let me know what you think and feel free to close without merging if it doesn't match your style.