Summary: | eliminate attributes parameter from JSObject::put for speed/clarity | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Darin Adler <darin> | ||||||||
Component: | JavaScriptCore | Assignee: | Darin Adler <darin> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Enhancement | CC: | ggaren, mjs | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | 17094 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
Darin Adler
2008-01-29 10:15:30 PST
Created attachment 18764 [details]
work in progress
Created attachment 19001 [details]
patch (passes all regression tests, but ignores attributes in put call in C API)
This is *so* close to being ready to review. The remaining issue is what we should do about the C API. Do we want to give the public put call the ability to assign attributes, and if so, how should we do this while preserving the rest of the semantics? Maybe it should do some combination of getDirect, putDirect, and put? Geoff, I'd like to land this. What do you think we should do about the attribute argument to JSObjectSetProperty? I think we discussed it but I forgot what we decided. Created attachment 19307 [details]
patch
Comment on attachment 19307 [details]
patch
r=me with the change we discussed on IRC
Committed revision 30534. |