Invalid early error for object literal method named "__proto__"
Created attachment 407084 [details] Patch
Comment on attachment 407084 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=407084&action=review r=me, this seems like a nice cleanup. > Source/JavaScriptCore/parser/SyntaxChecker.h:107 > + ALWAYS_INLINE Property(PropertyNode::Type type, bool isUnderscoreProtoSetter = false) nit: I'm not certain that a default param value is really better than a default field value here?
Created attachment 407179 [details] Patch
Comment on attachment 407179 [details] Patch (In reply to Ross Kirsling from comment #2) > r=me, this seems like a nice cleanup. I appreciate you reviewing this, Ross! The diff is quite tough. > nit: I'm not certain that a default param value is really better than a > default field value here? I've moved the default value to field initializer (TIL `struct` can also do that), and brought back the `type` field. PropertyNode::Type is the same size as `bool`, yet it's more useful to have (unlike `name`).
Committed r266117: <https://trac.webkit.org/changeset/266117> All reviewed patches have been landed. Closing bug and clearing flags on attachment 407179 [details].
<rdar://problem/67740285>