Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 310646 - AST Nodes that not implemnet IActionStmt and it sholud.
Summary: AST Nodes that not implemnet IActionStmt and it sholud.
Status: RESOLVED FIXED
Alias: None
Product: PTP
Classification: Tools
Component: Photran.For Internal Use (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P2 minor (vote)
Target Milestone: 6.0   Edit
Assignee: Jeffrey Overbey CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-04-27 11:03 EDT by Mariano Mendez CLA
Modified: 2011-05-14 06:43 EDT (History)
1 user (show)

See Also:

Attachments
The file to correct it. (15.34 KB, patch)
2010-04-27 11:04 EDT, Mariano Mendez CLA 		g.watson: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mariano Mendez CLA 2010-04-27 11:03:17 EDT 
Build Identifier:  20090920-1017

Making a refactoring called Remove Unreferenced Labels , I noticed that there were some statments that allowed a label but they did not implement the IActionStmt , for example ASTIfThenStmtNode , or ASTFormatNode , etc.

Reproducible: Always
Comment 1 Mariano Mendez CLA 2010-04-27 11:04:45 EDT 
Created attachment 166206 [details]
The file to correct it.
Comment 2 Mariano Mendez CLA 2010-05-10 15:45:57 EDT 
With the lasta changes this patch doesnt work anymore ! :( , I'll send a new one in the sort run.
Comment 3 Jeffrey Overbey CLA 2010-05-11 19:35:50 EDT 
According to Mariano's e-mail and the attached patch, the affected classes are:

ASTIfThenStmtNode
ASTFormatStmtNode
ASTElseStmtNode
ASTSubroutineStmtNode
ASTSelectCaseStmtNode
ASTDataStmtNode
ASTElseIfStmtNode
ASTFunctionStmtNode
ASTCaseStmtNode
ASTForallConstructStmtNode
Comment 4 Jeffrey Overbey CLA 2010-05-11 20:53:49 EDT 
I just committed a fix for this to CVS.  If it did not completely take care of the problem, please re-open this bug.  Thanks.
Comment 5 Jeffrey Overbey CLA 2010-05-11 20:55:08 EDT 
Forgot to mark this resolved-fixed