| Summary: | Do..while loop gains a semicolon each time it is toStringed | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Jesse Ruderman <jruderman> | ||||
| Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Minor | CC: | gavin.sharp | ||||
| Priority: | P2 | Keywords: | HasReduction | ||||
| Version: | 420+ | ||||||
| Hardware: | Mac | ||||||
| OS: | OS X 10.4 | ||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 13638 | ||||||
| Attachments: |
|
||||||
Although no website would ever care about this, this totally breaks jruderman's testing tool since so many of the generated tests seem to contain do {} while
Created attachment 14416 [details]
Avoid new semicolon when toString()ing function which has do..while
EmptyStatement got created after every do..while();, because ';' was not consumed as part of the do..while.
Comment on attachment 14416 [details]
Avoid new semicolon when toString()ing function which has do..while
Nice! r=me
(In reply to comment #3) > (From update of attachment 14416 [details] [edit]) > Nice! r=me Please avoid using tabs in ChangeLog entries in the future. Thanks! Need a changelog for layouttests next time too. Commited as 21320 |
javascript:f=function(){do{}while(0)}; for(i=0;i<4;++i) { alert(f); f=eval("(" + f + ")"); }