mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-04-22 14:12:10 +02:00
Code Issues Wiki Activity

Bug 486036 - Toggle Function Definition does not work when comment is involved

Browse source 
Change-Id: I77a2c2fe0607ce098da4a4e7679fc3999ea7ebbf
Signed-off-by: Marc-Andre Laperle <malaperle@gmail.com>
This commit is contained in:
Marc-Andre Laperle 2018-09-22 00:08:58 -04:00 • committed by Doug Schaefer
parent dac83cdde6
commit d537c6030c
2 changed files with 22 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
core/org.eclipse.cdt.ui.tests/ui/org/eclipse/cdt/ui/tests/refactoring/togglefunction/ToggleRefactoringTest.java
View file

@ -3648,4 +3648,22 @@ public class ToggleRefactoringTest extends RefactoringTestBase {
public void testToggleWithLambdaExpression_518271() throws Exception {
assertRefactoringSuccess();
}
//Test.cpp
//class MyClass {
// void MyMethod();
//};
//
//void MyClass::/*$*/MyMethod/*$$*/() {
//// A comment
//}
//====================
//class MyClass {
// void MyMethod() {
// // A comment
// }
//};
public void testToggleWithCommentsInSameFile_486036() throws Exception {
assertRefactoringSuccess();
}
}

4
core/org.eclipse.cdt.ui/src/org/eclipse/cdt/internal/ui/refactoring/togglefunction/ToggleFromImplementationToHeaderOrClassStrategy.java
View file

@ -146,6 +146,10 @@ public class ToggleFromImplementationToHeaderOrClassStrategy implements IToggleR
}
private void copyAllCommentsToNewLocation(IASTNode node, final ASTRewrite oldRw, final ASTRewrite newRw) {
//Bug 486036: In case oldRw == newRw (same file), there's no need to copy anything, the comments will follow along.
if (oldRw == newRw) {
return;
}
node.accept(new CPPASTAllVisitor() {
@Override
public int visitAll(IASTNode node){