Broaden TypedArray API fix to all apps not just Bleacher Report
Created attachment 456960 [details] Patch
Comment on attachment 456960 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=456960&action=review r=me > Source/JavaScriptCore/API/JSTypedArray.cpp:369 > +inline static bool isLinkedBeforeQuirk() let's call this "isLinkedBeforeTypedArrayLengthQuirk"?
Committed r292609 (?): <https://commits.webkit.org/r292609>
<rdar://problem/91487064>
Comment on attachment 456960 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=456960&action=review >> Source/JavaScriptCore/API/JSTypedArray.cpp:369 >> +inline static bool isLinkedBeforeQuirk() > > let's call this "isLinkedBeforeTypedArrayLengthQuirk"? Typically in the past I have named such functions needsTypedArrayAPIQuirk() rather than specifying how the functions do their jobs. I think it’s easier to read at the call site.
(In reply to Darin Adler from comment #5) > Comment on attachment 456960 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=456960&action=review > > >> Source/JavaScriptCore/API/JSTypedArray.cpp:369 > >> +inline static bool isLinkedBeforeQuirk() > > > > let's call this "isLinkedBeforeTypedArrayLengthQuirk"? > > Typically in the past I have named such functions needsTypedArrayAPIQuirk() > rather than specifying how the functions do their jobs. I think it’s easier > to read at the call site. I agree this is a better name and aids readability.