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

Bug 318517

Summary: Refactoring: Loop Fusion
Product: [Tools] PTP Reporter: Ashley Kasza <akasza2>
Component: Photran.Refactoring EngineAssignee: Jeffrey Overbey <com-eclipse-dot-org>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: akasza2, com-eclipse-dot-org
Version: 6.0   
Target Milestone: 7.0   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Attachments:
Description Flags
Loop Fusion Refactoring
g.watson: iplog+
Revised patch - renamed, messages committed none

Description Ashley Kasza CLA 2010-06-30 15:12:05 EDT
Refactoring used to fuse two compatible loops together
Comment 1 Ashley Kasza CLA 2010-06-30 15:12:43 EDT
Created attachment 173148 [details]
Loop Fusion Refactoring

Loop Fusion Refactoring
Comment 2 Jeffrey Overbey CLA 2010-06-30 15:51:56 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 3 Ashley Kasza CLA 2010-07-14 14:55:45 EDT
1. I wrote 100% of the code without incorporating content from elsewhere or
relying on the intellectual property of others,

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

3. I have included the EPL license header in all source files.
Comment 4 Jeffrey Overbey CLA 2010-09-21 11:08:38 EDT
Created attachment 179320 [details]
Revised patch - renamed, messages committed

I committed the externalized strings and plugin.xml changes to CVS, but the unit tests are not passing, so I did not commit the refactoring itself or the test suite.  Ashley, could you please replace your sources from HEAD (i.e., get rid of your local changes), then apply the REVISED patch (attached), and see why the tests are failing?  When it is fixed, please attach a revised revised patch.  Thanks.
Comment 5 Jeffrey Overbey CLA 2010-09-21 11:11:24 EDT
Oops, I attached that to the wrong bug.  That was for Unroll Loop.

Fuse Loops works fine.  It has been committed to CVS.  Thanks!