Summary: | [v8] implement ReturnsNew IDL hint for V8 code generator | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | anton muhin <antonm> | ||||||
Component: | WebKit Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | commit-queue | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
anton muhin
2009-10-24 11:52:09 PDT
Created attachment 41786 [details]
First take
Comment on attachment 41786 [details]
First take
This looks fine.
+ No new tests. (OOPS!)
This line will prevent the commit-queue from landing the patch, however.
Created attachment 41798 [details]
"No new tests" line removed
(In reply to comment #2) > (From update of attachment 41786 [details]) > This looks fine. > > + No new tests. (OOPS!) > > This line will prevent the commit-queue from landing the patch, however. Thanks a lot for review, Adam. "No new tests" line removed in the new patch. Comment on attachment 41798 [details] "No new tests" line removed Clearing flags on attachment: 41798 Committed r50030: <http://trac.webkit.org/changeset/50030> All reviewed patches have been landed. Closing bug. |