bloom icon indicating copy to clipboard operation
bloom copied to clipboard

Published 20 hours ago • verified publisher icon bits-and-blooms

Remove the side effect of the `EstimateFalsePositiveRate` function

Open SimFG opened this issue 3 years ago • 0 comments

This pull request include three changes.

  • Remove the side effect of the EstimateFalsePositiveRate function. Using the Copy function will has no effect on the current object.
  • Remove the repeated TestOrAdd function. For the TestAndAdd function, when the f.b.Test(l) gets the true result, executing the f.b.Set(l) and not are same. So only getting false to execute f.b.Set(l) will decrease one step. Changing it will find the TestAndAdd function is same as the TestOrAdd.
  • Update the .gitignore file to filter the .idea directory.

SimFG avatar Jul 22 '22 11:07 SimFG