Bug 14891

Summary: Decompilation of try block immediately following "else" fails
Product: WebKit Reporter: Oliver Hunt <oliver>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: CLOSED FIXED    
Severity: Normal CC: gavin.sharp, jwalden+bwo
Priority: P2 Keywords: InRadar
Version: 523.x (Safari 3)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 13638    
Attachments:
Description Flags
patch oliver: review+

Oliver Hunt
Reported 2007-08-06 14:48:26 PDT
Decompilation of function(){if (y) {} else try {} finally {}} incorrectly deompiles to if (y) { } elsetry { } finally { } Notice the absence of whitespace between else and try.
Attachments
patch (3.42 KB, patch)
2007-08-06 23:05 PDT, Sam Weinig 		oliver: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Sam Weinig
Comment 1 2007-08-06 23:05:40 PDT
Created attachment 15855 [details] patch
Oliver Hunt
Comment 2 2007-08-06 23:09:01 PDT
Comment on attachment 15855 [details] patch r&#65309;&#12417;&#65281;
Sam Weinig
Comment 3 2007-08-06 23:17:26 PDT
<rdar://problem/5391038>
Adam Roben (:aroben)
Comment 4 2007-08-06 23:24:12 PDT
Comment on attachment 15855 [details] patch + (TryNode::streamTo): Add newline before else. Looks to me like you added a newline before "try"
Sam Weinig
Comment 5 2007-08-06 23:26:50 PDT
Landed in r24904.
Note You need to log in before you can comment on or make changes to this bug.