Bug 332187 - Refactoring [427]: Remove Assigned Goto - CQ Approved
Summary: Refactoring [427]: Remove Assigned Goto - CQ Approved
Status: RESOLVED FIXED
Alias: None
Product: PTP
Classification: Tools
Component: Photran.Refactoring Engine (show other bugs)
Version: 6.0   Edit
Hardware: All All
: P1 enhancement (vote)
Target Milestone: 7.0   Edit
Assignee: Jeffrey Overbey CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-08 23:42 EST by John Hammonds CLA
Modified: 2011-03-16 13:35 EDT (History)
6 users (show)

See Also:


Attachments
Patch file created against CVS @ 6:55PMCST 2010-12-8 (135.15 KB, text/plain)
2010-12-08 23:42 EST, John Hammonds CLA
com-eclipse-dot-org: iplog+
Details

Note You need to log in before you can comment on or make changes to this bug.
Description John Hammonds CLA 2010-12-08 23:42:12 EST
Created attachment 184839 [details]
Patch file created against CVS @ 6:55PMCST 2010-12-8

This refactoring will
•	Replace instances of ASSIGN statements with assignment statements.
•	Replace assigned GOTO statements with SELECT CASE statements with nested (normal) GOTOs.
Refactoring of a particular ASSIGN or assigned GOTO will fail if
•	The statement label used in an ASSIGN is not present in the scoping unit of the ASSIGN statement.
•	The variable name associated with the ASSIGN statement is used in an action statement.
The user will be presented with a graphical interface to select if a default CASE statement should be added to the SELECT CASE in the refactored output.
If multiple variables are specified by assign statements
•	If all of the labels are used in action statements, the refactoring will fail
•	If some of the labels are used in an action statement, the user will be prompted to decide if the refactoring should continue refactoring only the cases where the variable is not used in an action statement.

A hot key is added to launch the refactoring.  The hot key is assigned to Alt+Shift+A.
Comment 1 Jeffrey Overbey CLA 2010-12-13 04:31:53 EST
Hi all,

Thanks for the contribution!  The patch looks good; I don't believe any further changes are necessary at this time.

Before we submit this for an IP review, can the five of you *each* 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?

Thanks!
Jeff
Comment 2 John Hammonds CLA 2010-12-13 11:42:24 EST
(In reply to comment #1)

This is to confirm that we have:

1. Contributed 100% of the content.
2. We have the right to contribute the code.
3. Included the EPL header in all source files.
Comment 3 Jeffrey Overbey CLA 2010-12-13 11:48:30 EST
Thanks, John.  Andrea, Rajashekhar, Balaji, and Paramvir, I need to ask you to make similar confirmations...
Comment 4 Param CLA 2010-12-13 12:25:44 EST
(In reply to comment #1)
This is to confirm that

1. We have written 100% of the code without incorporating content from  elsewhere or relying on the intellectual property of others,
2. We have the right to contribute the code to Eclipse, and
3. We have included the EPL license header in all source files.
Comment 5 Balaji Ambresh Rajkumar CLA 2010-12-13 12:30:31 EST
This is to confirm that

1. We have written 100% of the code without incorporating content from 
elsewhere or relying on the intellectual property of others,
2. We have the right to contribute the code to Eclipse, and
3. We have included the EPL license header in all source files.
Comment 6 Andrea CLA 2010-12-13 12:40:26 EST
This is to confirm that

1. We have written 100% of the code without incorporating content from 
elsewhere or relying on the intellectual property of others,
2. We have the right to contribute the code to Eclipse, and
3. We have included the EPL license header in all source files.
Comment 7 Rajashekhar Arasanal CLA 2010-12-13 12:50:58 EST
This is to confirm that

1. We have written 100% of the code without incorporating content from 
elsewhere or relying on the intellectual property of others,
2. We have the right to contribute the code to Eclipse, and
3. We have included the EPL license header in all source files.
Comment 8 Jeffrey Overbey CLA 2010-12-13 14:12:20 EST
Thanks for the quick replies!

I'll start the IP review process soon.  If the IP team has any questions, I'll post them here.
Comment 9 Jeffrey Overbey CLA 2010-12-13 23:38:19 EST
For the IP review, I will need to describe how much of the work was done by each person involved.  Generally, for CS427 projects, the work was more-or-less evenly distributed, so I will use the following breakdown unless you inform me otherwise:

Andrea Dranberg - 20%
John Hammonds - 20%
Rajashekhar Arasanal - 20%
Balaji Ambresh Rajkumar - 20%
Paramvir Singh - 20%
Comment 10 John Hammonds CLA 2010-12-14 01:18:30 EST
(In reply to comment #9)
Given the nature of the class project, I think that this split is appropriate.
Comment 11 Jeffrey Overbey CLA 2010-12-15 11:30:58 EST
CQ 4703 Submitted - https://dev.eclipse.org/ipzilla/show_bug.cgi?id=4703
Comment 12 Jeffrey Overbey CLA 2011-01-18 14:51:12 EST
The CQ was approved.  Check in to CVS pending.
Comment 13 Jeffrey Overbey CLA 2011-03-16 13:35:25 EDT
These are committed to CVS HEAD for inclusion in Photran 7.0/Indigo (the June,
2011 release).

Thanks again for the contribution!

Please also make sure your names are all included and are spelled correctly on
the Contributors page: http://www.eclipse.org/photran/contributors.php