On Tue, Dec 18, 2018 at 11:35 AM Michael Catanzaro <mcatanz...@igalia.com> wrote:
> I know I'm getting a bit far afield here, but: > > On Mon, Dec 17, 2018 at 9:26 PM, Ryosuke Niwa <rn...@webkit.org> wrote: > > But then our behavior of HashMap which doesn't accept the POD > > integral value of 0 as a key > > This behavior is really unexpected and dangerous [1], and we should > seriously consider changing it. No doubt lots of bugs caused by this > are just waiting to be uncovered. I've been working on WebKit since > 2014 and didn't know about this until last month. I tend to agree but then we'd come up with other numbers for the empty & deleted values. I've been thinking that we could use -1 and -2 but that's also somewhat arbitrary restriction. Another oddity: I recently learned that AtomicStrings are actually > interned strings. WTF. Why not call them that? I had thought for years > that they were strings safe to be shared across threads, like other > atomics. Not at all. Maybe this was dumb of me, but it could have been > avoided by better naming. > This topic has been discussed extensively in the past: https://lists.webkit.org/pipermail/webkit-dev/2013-June/024988.html - R. Niwa
_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-dev