-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Strict.alterFEager
too lazy when inserting new key
#383
Labels
Comments
Judging by the commit message of 85e11c7, the strictness properties of this code already seemed dubious when it was introduced:
It's pretty disappointing that no one even bothered to record this task on the issue tracker. :( |
This was referenced Mar 20, 2022
sjakobi
added a commit
that referenced
this issue
Mar 21, 2022
sjakobi
added a commit
that referenced
this issue
Mar 21, 2022
sjakobi
added a commit
that referenced
this issue
Mar 23, 2022
sjakobi
added a commit
that referenced
this issue
Mar 23, 2022
sjakobi
added a commit
that referenced
this issue
Mar 23, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
unordered-containers/Data/HashMap/Internal/Strict.hs
Lines 384 to 422 in 7237826
The problem is in lines 404–407:
v'
is never forced.I wonder whether confusion about the strictness of
insertNewKey
may have led to this bug. Although this function doesn't have a module prefix, it is imported fromData.HashMap.Internal
. The code author may have thought that they were using a local, strict version ofinsertNewKey
.The text was updated successfully, but these errors were encountered: