[Issue 4864] ICE(statement.c) Crash on invalid 'if statement' body inside mixin

[d-bugmail]

http://d.puremagic.com/issues/show_bug.cgi?id=4864


Don  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED


-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---

[Issue 4864] ICE(statement.c) Crash on invalid 'if statement' body inside mixin

[d-bugmail]

http://d.puremagic.com/issues/show_bug.cgi?id=4864



--- Comment #4 from Walter Bright  2010-12-04 
22:57:26 PST ---
http://www.dsource.org/projects/dmd/changeset/778

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---

[Issue 4864] ICE(statement.c) Crash on invalid 'if statement' body inside mixin

[d-bugmail]

http://d.puremagic.com/issues/show_bug.cgi?id=4864


Walter Bright  changed:

   What|Removed |Added

 CC||bugzi...@digitalmars.com


--- Comment #3 from Walter Bright  2010-12-04 
22:57:08 PST ---
http://www.dsource.org/projects/dmd/changeset/778

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---

[Issue 4864] ICE(statement.c) Crash on invalid 'if statement' body inside mixin

[d-bugmail]

http://d.puremagic.com/issues/show_bug.cgi?id=4864


Don  changed:

   What|Removed |Added

   Keywords||patch
 CC||clugd...@yahoo.com.au


--- Comment #2 from Don  2010-11-17 00:13:39 PST ---
PATCH: statement.c 337.
CompileStatement::flatten()

Statements *a = new Statements();
while (p.token.value != TOKeof)
{
+   int olderrs = global.errors;
Statement *s = p.parseStatement(PSsemi | PScurlyscope);
+   if (olderrs == global.errors) // discard it if parsing failed
a->push(s);
}

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---

[Issue 4864] ICE(statement.c) Crash on invalid 'if statement' body inside mixin

[d-bugmail]

http://d.puremagic.com/issues/show_bug.cgi?id=4864



--- Comment #1 from yebblies  2010-09-15 02:13:58 PDT ---
Additional test case:

void main()
{
mixin("if (a b) ;");
}

Another case of a broken if statement.
The crash goes away if a body is added to the if statement.

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
--- You are receiving this mail because: ---