mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-04-29 19:45:01 +02:00
Code Issues Wiki Activity

Added missing @Override annotations.

Browse source
This commit is contained in:
Sergey Prigogin 2011-11-01 18:32:57 -07:00
parent 2e166ecabb
commit ed8b3356ad
2 changed files with 53 additions and 47 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/IASTLiteralExpression.java
View file

@ -18,7 +18,6 @@ package org.eclipse.cdt.core.dom.ast;
* @noextend This interface is not intended to be extended by clients. * @noextend This interface is not intended to be extended by clients.
*/ */
public interface IASTLiteralExpression extends IASTExpression { public interface IASTLiteralExpression extends IASTExpression {
/** /**
* An integer literal e.g. 5 * An integer literal e.g. 5
*/ */
@ -79,6 +78,7 @@ public interface IASTLiteralExpression extends IASTExpression {
* Returns the value of the literal as string. * Returns the value of the literal as string.
* @since 5.1 * @since 5.1
*/ */
@Override
public String toString(); public String toString();
/** /**
@ -95,11 +95,13 @@ public interface IASTLiteralExpression extends IASTExpression {
/** /**
* @since 5.1 * @since 5.1
*/ */
@Override
public IASTLiteralExpression copy(); public IASTLiteralExpression copy();
/** /**
* @since 5.3 * @since 5.3
*/ */
@Override
public IASTLiteralExpression copy(CopyStyle style); public IASTLiteralExpression copy(CopyStyle style);
/** /**

68
core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/CPPASTCompoundStatement.java
View file

@ -23,17 +23,17 @@ import org.eclipse.cdt.internal.core.dom.parser.IASTAmbiguityParent;
/** /**
* @author jcamelon * @author jcamelon
*/ */
public class CPPASTCompoundStatement extends ASTNode implements public class CPPASTCompoundStatement extends ASTNode
IASTCompoundStatement, IASTAmbiguityParent { implements IASTCompoundStatement, IASTAmbiguityParent {
private IASTStatement[] statements = new IASTStatement[2];
private IASTStatement [] statements = new IASTStatement[2]; private ICPPScope scope;
private ICPPScope scope = null;
@Override
public CPPASTCompoundStatement copy() { public CPPASTCompoundStatement copy() {
return copy(CopyStyle.withoutLocations); return copy(CopyStyle.withoutLocations);
} }
@Override
public CPPASTCompoundStatement copy(CopyStyle style) { public CPPASTCompoundStatement copy(CopyStyle style) {
CPPASTCompoundStatement copy = new CPPASTCompoundStatement(); CPPASTCompoundStatement copy = new CPPASTCompoundStatement();
for (IASTStatement statement : getStatements()) for (IASTStatement statement : getStatements())
@ -45,57 +45,61 @@ public class CPPASTCompoundStatement extends ASTNode implements
return copy; return copy;
} }
@Override
public IASTStatement[] getStatements() { public IASTStatement[] getStatements() {
if( statements == null ) return IASTStatement.EMPTY_STATEMENT_ARRAY; if (statements == null)
return (IASTStatement[]) ArrayUtil.trim( IASTStatement.class, statements ); return IASTStatement.EMPTY_STATEMENT_ARRAY;
return (IASTStatement[]) ArrayUtil.trim(IASTStatement.class, statements);
} }
@Override
public void addStatement(IASTStatement statement) { public void addStatement(IASTStatement statement) {
assertNotFrozen(); assertNotFrozen();
statements = (IASTStatement[]) ArrayUtil.append( IASTStatement.class, statements, statement ); statements = (IASTStatement[]) ArrayUtil.append(IASTStatement.class, statements, statement);
if (statement != null) { if (statement != null) {
statement.setParent(this); statement.setParent(this);
statement.setPropertyInParent(NESTED_STATEMENT); statement.setPropertyInParent(NESTED_STATEMENT);
} }
} }
@Override
public IScope getScope() { public IScope getScope() {
if( scope == null ) if (scope == null)
scope = new CPPBlockScope( this ); scope = new CPPBlockScope(this);
return scope; return scope;
} }
@Override @Override
public boolean accept( ASTVisitor action ){ public boolean accept(ASTVisitor action) {
if( action.shouldVisitStatements ){ if (action.shouldVisitStatements) {
switch( action.visit( this ) ){ switch (action.visit(this)) {
case ASTVisitor.PROCESS_ABORT : return false; case ASTVisitor.PROCESS_ABORT: return false;
case ASTVisitor.PROCESS_SKIP : return true; case ASTVisitor.PROCESS_SKIP: return true;
default : break; default: break;
} }
} }
IASTStatement [] s = getStatements(); IASTStatement[] s = getStatements();
for ( int i = 0; i < s.length; i++ ) { for (int i = 0; i < s.length; i++) {
if( !s[i].accept( action ) ) return false; if (!s[i].accept(action))
return false;
} }
if( action.shouldVisitStatements ){ if (action.shouldVisitStatements) {
switch( action.leave( this ) ){ switch (action.leave(this)) {
case ASTVisitor.PROCESS_ABORT : return false; case ASTVisitor.PROCESS_ABORT: return false;
case ASTVisitor.PROCESS_SKIP : return true; case ASTVisitor.PROCESS_SKIP: return true;
default : break; default: break;
} }
} }
return true; return true;
} }
@Override
public void replace(IASTNode child, IASTNode other) { public void replace(IASTNode child, IASTNode other) {
if( statements == null ) return; if (statements == null) return;
for( int i = 0; i < statements.length; ++i ) for (int i = 0; i < statements.length; ++i) {
{ if (statements[i] == child) {
if( statements[i] == child ) other.setParent(statements[i].getParent());
{ other.setPropertyInParent(statements[i].getPropertyInParent());
other.setParent( statements[i].getParent() );
other.setPropertyInParent( statements[i].getPropertyInParent() );
statements[i] = (IASTStatement) other; statements[i] = (IASTStatement) other;
} }
} }