Remove unused ARMv6-specific #if branches in WebCore::canHyphenate
Created attachment 237438 [details] Patch
http://trac.webkit.org/changeset/173151
Comment on attachment 237438 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=237438&action=review > Source/WebCore/ChangeLog:10 > + Use kCFAllocatorDefault instead of 0. Do we do that? I thought that 0 was better in nearly every respect - shorter and faster.
(In reply to comment #3) > (From update of attachment 237438 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=237438&action=review > > > Source/WebCore/ChangeLog:10 > > + Use kCFAllocatorDefault instead of 0. > > Do we do that? I thought that 0 was better in nearly every respect - shorter and faster. Perhaps, but also completely mysterious when reading the code.
We do need a certain level of C++ knowledge and platform savviness from contributors. I don't think that we have any who actually find this mysterious. I don't have any links, but I'm pretty sure that this was discussed before, and kCFAllocatorDefault was agreed unnecessary.