[JSC] Add fast property enumeration mode for JSON.stringify
Created attachment 438450 [details] Patch
Created attachment 438451 [details] Patch
Created attachment 438452 [details] Patch
Created attachment 438459 [details] Patch
Created attachment 438461 [details] Patch
Created attachment 438503 [details] Patch
Created attachment 438504 [details] Patch
Comment on attachment 438504 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=438504&action=review r=me > Source/JavaScriptCore/ChangeLog:10 > + This patch adds fast object property enumeration. When we know that source object has some conditions, we can say that, I suggest "has some conditions" ==> "meets some conditions". > Source/JavaScriptCore/runtime/JSONObject.cpp:96 > + bool isFastObjectProperties() const { return m_isFastObjectProperties; } nit: let's rename these to `hasFastObjectProperties()` and `m_hasFastObjectProperties`.
Comment on attachment 438504 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=438504&action=review Thanks! >> Source/JavaScriptCore/ChangeLog:10 >> + This patch adds fast object property enumeration. When we know that source object has some conditions, we can say that, > > I suggest "has some conditions" ==> "meets some conditions". Fixed. >> Source/JavaScriptCore/runtime/JSONObject.cpp:96 >> + bool isFastObjectProperties() const { return m_isFastObjectProperties; } > > nit: let's rename these to `hasFastObjectProperties()` and `m_hasFastObjectProperties`. Changed.
Committed r282707 (241844@main): <https://commits.webkit.org/241844@main>
<rdar://problem/83259718>