mirror of
https://github.com/eclipse-cdt/cdt
synced 2025-08-01 21:35:40 +02:00
Cosmetics.
This commit is contained in:
parent
f7cf5a154f
commit
6a1c54cc57
4 changed files with 9 additions and 9 deletions
|
@ -2452,7 +2452,7 @@ public class CompleteParser2Tests extends BaseTestCase {
|
||||||
|
|
||||||
public void test158192_declspec_on_class() throws Exception {
|
public void test158192_declspec_on_class() throws Exception {
|
||||||
if (!Platform.getOS().equals(Platform.OS_WIN32))
|
if (!Platform.getOS().equals(Platform.OS_WIN32))
|
||||||
return; // XXX: see GPPParserExtensionConfiguration.supportDeclspecSpecifiers()
|
return; // TODO: see GPPParserExtensionConfiguration.supportDeclspecSpecifiers()
|
||||||
|
|
||||||
Writer writer = new StringWriter();
|
Writer writer = new StringWriter();
|
||||||
writer.write("class __declspec(foobar) Foo1 {};\n");
|
writer.write("class __declspec(foobar) Foo1 {};\n");
|
||||||
|
@ -2479,7 +2479,7 @@ public class CompleteParser2Tests extends BaseTestCase {
|
||||||
|
|
||||||
public void test158192_declspec_on_variable() throws Exception {
|
public void test158192_declspec_on_variable() throws Exception {
|
||||||
if (!Platform.getOS().equals(Platform.OS_WIN32))
|
if (!Platform.getOS().equals(Platform.OS_WIN32))
|
||||||
return; // XXX: see GPPParserExtensionConfiguration.supportDeclspecSpecifiers()
|
return; // TODO: see GPPParserExtensionConfiguration.supportDeclspecSpecifiers()
|
||||||
|
|
||||||
Writer writer = new StringWriter();
|
Writer writer = new StringWriter();
|
||||||
writer.write("__declspec(foobar) class Foo {} bar;\n");
|
writer.write("__declspec(foobar) class Foo {} bar;\n");
|
||||||
|
@ -2500,7 +2500,7 @@ public class CompleteParser2Tests extends BaseTestCase {
|
||||||
// see https://bugs.eclipse.org/bugs/show_bug.cgi?id=158192
|
// see https://bugs.eclipse.org/bugs/show_bug.cgi?id=158192
|
||||||
public void test158192_declspec_in_declarator() throws Exception {
|
public void test158192_declspec_in_declarator() throws Exception {
|
||||||
if (!Platform.getOS().equals(Platform.OS_WIN32))
|
if (!Platform.getOS().equals(Platform.OS_WIN32))
|
||||||
return; // XXX: see GPPParserExtensionConfiguration.supportDeclspecSpecifiers()
|
return; // TODO: see GPPParserExtensionConfiguration.supportDeclspecSpecifiers()
|
||||||
|
|
||||||
Writer writer = new StringWriter();
|
Writer writer = new StringWriter();
|
||||||
|
|
||||||
|
|
|
@ -55,7 +55,7 @@ public abstract class CPPASTNameBase extends ASTNode implements ICPPASTName {
|
||||||
// exceeds MAX_RESOLUTION_DEPTH. If the resolution depth exceeds
|
// exceeds MAX_RESOLUTION_DEPTH. If the resolution depth exceeds
|
||||||
// MAX_RESOLUTION_DEPTH + 1, it means that attempting to create the
|
// MAX_RESOLUTION_DEPTH + 1, it means that attempting to create the
|
||||||
// recursion resolving binding has led us back to trying to resolve
|
// recursion resolving binding has led us back to trying to resolve
|
||||||
// the bidning for this name again, so the recursion isn't broken.
|
// the binding for this name again, so the recursion isn't broken.
|
||||||
// This can happen because the constructor of RecursionResolvingBinding
|
// This can happen because the constructor of RecursionResolvingBinding
|
||||||
// calls ProblemBinding.getMessage(), which can try to do name
|
// calls ProblemBinding.getMessage(), which can try to do name
|
||||||
// resolution to build an argument string if one wasn't provided in the
|
// resolution to build an argument string if one wasn't provided in the
|
||||||
|
|
|
@ -437,7 +437,7 @@ public class DefaultCCommentAutoEditStrategyTest extends AbstractAutoEditTest {
|
||||||
// void foo()X{}
|
// void foo()X{}
|
||||||
// void bar(int x);
|
// void bar(int x);
|
||||||
// };
|
// };
|
||||||
public void _testFollowingDeclaration4b() { // XXX - this is likely invalid anyhow
|
public void _testFollowingDeclaration4b() { // TODO - this is likely invalid anyhow
|
||||||
assertDeclarationFollowingX("void foo(){}"); // (X is just the cursor position)
|
assertDeclarationFollowingX("void foo(){}"); // (X is just the cursor position)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -449,7 +449,7 @@ public class DefaultCCommentAutoEditStrategyTest extends AbstractAutoEditTest {
|
||||||
// }
|
// }
|
||||||
// void bar(int x);
|
// void bar(int x);
|
||||||
// };
|
// };
|
||||||
public void _testFollowingDeclaration4c() { // XXX - this is likely invalid anyhow
|
public void _testFollowingDeclaration4c() { // TODO - this is likely invalid anyhow
|
||||||
assertDeclarationFollowingX("void foo()\n {\n int x;\n }\n"); // (X is just the cursor position)
|
assertDeclarationFollowingX("void foo()\n {\n int x;\n }\n"); // (X is just the cursor position)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -20,7 +20,6 @@ import junit.framework.Test;
|
||||||
import junit.framework.TestCase;
|
import junit.framework.TestCase;
|
||||||
import junit.framework.TestSuite;
|
import junit.framework.TestSuite;
|
||||||
|
|
||||||
import org.eclipse.cdt.internal.ui.text.FastCPartitionScanner;
|
|
||||||
import org.eclipse.jface.text.BadLocationException;
|
import org.eclipse.jface.text.BadLocationException;
|
||||||
import org.eclipse.jface.text.Document;
|
import org.eclipse.jface.text.Document;
|
||||||
import org.eclipse.jface.text.IDocument;
|
import org.eclipse.jface.text.IDocument;
|
||||||
|
@ -28,6 +27,8 @@ import org.eclipse.jface.text.IRegion;
|
||||||
import org.eclipse.jface.text.rules.IPartitionTokenScanner;
|
import org.eclipse.jface.text.rules.IPartitionTokenScanner;
|
||||||
import org.eclipse.jface.text.rules.IToken;
|
import org.eclipse.jface.text.rules.IToken;
|
||||||
|
|
||||||
|
import org.eclipse.cdt.internal.ui.text.FastCPartitionScanner;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Compares two <code>IParitionTokenScanner</code>s for performance.
|
* Compares two <code>IParitionTokenScanner</code>s for performance.
|
||||||
*/
|
*/
|
||||||
|
@ -141,7 +142,7 @@ public class PartitionTokenScannerTest extends TestCase {
|
||||||
|
|
||||||
int offsetIndex= offset - region.getOffset();
|
int offsetIndex= offset - region.getOffset();
|
||||||
|
|
||||||
// XXX kludge
|
// TODO kludge
|
||||||
if (offsetIndex > line.length())
|
if (offsetIndex > line.length())
|
||||||
offsetIndex= line.length();
|
offsetIndex= line.length();
|
||||||
|
|
||||||
|
@ -150,7 +151,6 @@ public class PartitionTokenScannerTest extends TestCase {
|
||||||
buffer.append("<POS>");
|
buffer.append("<POS>");
|
||||||
buffer.append(line.substring(offsetIndex));
|
buffer.append(line.substring(offsetIndex));
|
||||||
buffer.append(']');
|
buffer.append(']');
|
||||||
|
|
||||||
} catch (BadLocationException e) {
|
} catch (BadLocationException e) {
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Add table
Reference in a new issue