Page MenuHomePhabricator

Implement P0513R0 - "Poisoning the Hash"

Authored by EricWF on Jan 19 2017, 10:21 PM.



Exactly what the title says.

This patch also adds a std::hash<nullptr_t> specialization in C++17, but it was not added by this paper and I can't find the actual paper that adds it.

See for more info.

If there are no comments in the next couple of days I'll commit this

Diff Detail

Event Timeline

EricWF created this revision.Jan 19 2017, 10:21 PM
EricWF updated this revision to Diff 85172.Jan 20 2017, 12:13 PM
  • Fix bitset test that was overflowing the stack.
  • Add tests that basic_string_view hashes to the same value as an equivalent basic_string.
EricWF accepted this revision.Jan 20 2017, 4:09 PM

I feel comfortable with this in its current state.

This revision is now accepted and ready to land.Jan 20 2017, 4:09 PM
EricWF closed this revision.Jan 20 2017, 4:13 PM