Bug 14891

Summary:

Decompilation of try block immediately following "else" fails

Product:

WebKit

Reporter:

Oliver Hunt <oliver>

Component:

JavaScriptCore

Assignee:

Nobody <webkit-unassigned>

Status:

CLOSED FIXED

Severity:

Normal

CC:

gavin.sharp, jwalden+bwo

Priority:

Keywords:

InRadar

Version:

523.x (Safari 3)

Hardware:

All

OS:

All

Bug Depends on:

Bug Blocks:

13638

Attachments:

Description	Flags
patch	oliver: review+

Description Oliver Hunt 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.

Comment 1 Sam Weinig 2007-08-06 23:05:40 PDT

Created attachment 15855 [details]
patch

Comment 2 Oliver Hunt 2007-08-06 23:09:01 PDT

Comment on attachment 15855 [details]
patch

r&#65309;&#12417;&#65281;

Comment 3 Sam Weinig 2007-08-06 23:17:26 PDT

<rdar://problem/5391038>

Comment 4 Adam Roben (:aroben) 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"

Comment 5 Sam Weinig 2007-08-06 23:26:50 PDT

Landed in r24904.