Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 317823

Summary:

AST: ASTCaseConstructNode structure

Product:

[Tools] PTP

Reporter:

Matthew Fotzler <incongruous>

Component:

Photran.For Internal Use

Assignee:

Photran Inbox <photran-inbox>

Status:

ASSIGNED ---

QA Contact:

Severity:

normal

Priority:

CC:

com-eclipse-dot-org

Version:

unspecified

Target Milestone:

---

Hardware:

All

OS:

All

Whiteboard:

Attachments:

Description	Flags
proposed patch	com-eclipse-dot-org: iplog+

Description Matthew Fotzler

2010-06-24 10:08:29 EDT

Build Identifier: 

The case statement is not structured recursively.  It would be nice for the code inside of the case statement to be stored in the ASTCaseStmtNode, accessible by getBody() or something.

Reproducible: Always

Comment 1 Matthew Fotzler

2010-06-24 10:14:08 EDT

Also, we will need to change the Control Flow Graph code once this is fixed.

Comment 2 Matthew Fotzler

2010-10-15 15:50:48 EDT

Created attachment 181011 [details]
proposed patch

This patch alters the ASTCaseStmtNode to add a body that contains the program statements inside the case statement.  The SelectCaseRestructurer visits all case statements and modifies them to use this added body.

Comment 3 Jeffrey Overbey

2010-10-15 16:14:21 EDT

Thanks.  Can you please confirm that

1. you wrote 100% of the code without incorporating content from elsewhere or relying on the intellectual property of others,

2. you have the right to contribute the code to Eclipse, and

3. you have included the EPL license header in all source files?

Comment 4 Matthew Fotzler

2010-10-15 16:21:45 EDT

Yes, I wrote 100% of the code without incorporating content from elsewhere or relying on the intellectual property of others, I have the right to contribute the code to Eclipse, and I have included the EPL license header in all source files.