Skip to content
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

Fix bad c-tor for WeakKeyDict. Fix #35782 #35783

Merged
merged 1 commit into from
May 8, 2020
Merged

Fix bad c-tor for WeakKeyDict. Fix #35782 #35783

merged 1 commit into from
May 8, 2020

Conversation

kshyatt
Copy link
Contributor

@kshyatt kshyatt commented May 7, 2020

And add a test.

@kshyatt kshyatt added test This change adds or pertains to unit tests bugfix This change fixes an existing bug labels May 7, 2020
@kshyatt kshyatt requested a review from JeffBezanson May 7, 2020 02:30
@JeffBezanson
Copy link
Member

This is fine, but we could also add a sizehint! method for WeakKeyDict that does this.

@kshyatt
Copy link
Contributor Author

kshyatt commented May 7, 2020

OK, added that :)

@JeffBezanson JeffBezanson merged commit ba91a98 into master May 8, 2020
@JeffBezanson JeffBezanson deleted the ksh/wkdfix branch May 8, 2020 19:07
eschnett pushed a commit to eschnett/julia that referenced this pull request May 11, 2020
KristofferC pushed a commit that referenced this pull request May 12, 2020
Roger-luo pushed a commit to Roger-luo/julia that referenced this pull request May 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix This change fixes an existing bug test This change adds or pertains to unit tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants