[webkit-reviews] review denied: [Bug 77709] Rename [Deletable] IDL to [NotConfigurable] IDL : [Attachment 125249] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Feb 3 00:54:06 PST 2012


Adam Barth <abarth at webkit.org> has denied Kentaro Hara <haraken at chromium.org>'s
request for review:
Bug 77709: Rename [Deletable] IDL to [NotConfigurable] IDL
https://bugs.webkit.org/show_bug.cgi?id=77709

Attachment 125249: Patch
https://bugs.webkit.org/attachment.cgi?id=125249&action=review

------- Additional Comments from Adam Barth <abarth at webkit.org>
I'm not sure this is an improvement.  Deletable is very easy for me to
understand what it means.  If we could align exactly with WebIDL, that might
swing the balance, but given that we're not going to match WebIDL exactly, I'd
opt for the clearer "Deletable" over the somewhat-confusing "NotConfigurable".

Thoughts?


More information about the webkit-reviews mailing list