Cosmetics.

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPMethod.java

@@ -10,7 +10,6 @@
  *******************************************************************************/
 package org.eclipse.cdt.core.dom.ast.cpp;
 
-
 /**
  * Base interface for methods, also used for constructors.
  * 
@@ -18,7 +17,7 @@ package org.eclipse.cdt.core.dom.ast.cpp;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPMethod extends ICPPFunction, ICPPMember {
-	public static final ICPPMethod [] EMPTY_CPPMETHOD_ARRAY = new ICPPMethod[0];
+	public static final ICPPMethod[] EMPTY_CPPMETHOD_ARRAY = {};
 	
 	/**
 	 * Returns whether this method is declared to be virtual. Does not detect whether
@@ -27,9 +26,9 @@ public interface ICPPMethod extends ICPPFunction, ICPPMember {
 	public boolean isVirtual();
 	
 	/**
-	 * is this a destructor
+	 * Is this a destructor?
 	 * 
-	 * returns true if its name starts with '~' 
+	 * Returns true if its name starts with '~' 
 	 */
 	public boolean isDestructor();
 

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPNamespace.java

@@ -21,12 +21,12 @@ import org.eclipse.cdt.core.dom.ast.IBinding;
  */
 public interface ICPPNamespace extends ICPPBinding {
 	/**
-	 * get the scope object associated with this namespace
+	 * Returns the scope object associated with this namespace
 	 */
 	public ICPPNamespaceScope getNamespaceScope();
 	
 	/**
-	 * get an array of the all the bindings declared in this namespace.
+	 * Returns an array of the all the bindings declared in this namespace.
 	 */
 	public IBinding[] getMemberBindings();
 

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPNamespaceAlias.java

@@ -17,10 +17,8 @@ import org.eclipse.cdt.core.dom.ast.IBinding;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPNamespaceAlias extends ICPPNamespace {
-
     /**
-     * get the binding of the original namespace.
+     * Returns the binding of the original namespace.
      */
     public IBinding getBinding();
-
 }

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPNamespaceScope.java

@@ -11,7 +11,6 @@
  *******************************************************************************/
 package org.eclipse.cdt.core.dom.ast.cpp;
 
-
 /**
  * A namespace scope is either a block-scope or a namespace-scope or global scope.
  * 
@@ -19,7 +18,6 @@ package org.eclipse.cdt.core.dom.ast.cpp;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPNamespaceScope extends ICPPScope {
-
 	/**
 	 * @since 5.3
 	 */

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPParameter.java

@@ -18,7 +18,6 @@ import org.eclipse.cdt.core.dom.ast.IParameter;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPParameter extends IParameter, ICPPVariable {
-	
 	/**
 	 * @since 5.2
 	 */

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPParameterPackType.java

@@ -21,7 +21,6 @@ import org.eclipse.cdt.core.dom.ast.IType;
  * @since 5.2
  */
 public interface ICPPParameterPackType extends IType {
-	
 	/**
 	 * Returns the pattern for the pack-expansion
 	 */

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPReferenceType.java

@@ -18,7 +18,6 @@ import org.eclipse.cdt.core.dom.ast.IType;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPReferenceType extends IType {
-	
 	/**
 	 * Returns the type that this is a reference of
 	 */

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPScope.java

@@ -17,5 +17,4 @@ import org.eclipse.cdt.core.dom.ast.IScope;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPScope extends IScope {
-
 }

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPTemplateDefinition.java

@@ -11,7 +11,6 @@
  *******************************************************************************/
 package org.eclipse.cdt.core.dom.ast.cpp;
 
-
 /**
  * Base interface for all template definitions including explicit (partial) specializations.
  * 
@@ -19,7 +18,6 @@ package org.eclipse.cdt.core.dom.ast.cpp;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPTemplateDefinition extends ICPPBinding {
-
 	/**
 	 * Returns an array of the template parameters. 
 	 * In the case of a specialization, the array will be empty,

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPTemplateInstance.java

@@ -9,7 +9,6 @@
  *     IBM - Initial API and implementation
  *     Markus Schorn (Wind River Systems)
  *******************************************************************************/
-
 package org.eclipse.cdt.core.dom.ast.cpp;
 
 import org.eclipse.cdt.core.dom.ast.IType;

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPTemplateParameter.java

@@ -18,7 +18,7 @@ package org.eclipse.cdt.core.dom.ast.cpp;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPTemplateParameter extends ICPPBinding {
-	public static final ICPPTemplateParameter[] EMPTY_TEMPLATE_PARAMETER_ARRAY = new ICPPTemplateParameter[0];
+	public static final ICPPTemplateParameter[] EMPTY_TEMPLATE_PARAMETER_ARRAY = {};
 
 	/**
 	 * Returns the zero-based position of this parameter within the template parameter list it belongs to. 

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPTemplateParameterMap.java

@@ -10,7 +10,6 @@
  *******************************************************************************/ 
 package org.eclipse.cdt.core.dom.ast.cpp;
 
-
 /**
  * Models the mapping of template parameters to values, or pack-expansions.
  * 
@@ -19,7 +18,6 @@ package org.eclipse.cdt.core.dom.ast.cpp;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPTemplateParameterMap {
-
 	/**
 	 * Returns the value for the template parameter with the given id.
 	 * @see ICPPTemplateParameter#getParameterID()
@@ -32,7 +30,6 @@ public interface ICPPTemplateParameterMap {
 	 */
 	public ICPPTemplateArgument getArgument(ICPPTemplateParameter param);
 
-
 	/**
 	 * Returns the values for the template parameter pack with the given id in the map, 
 	 * or <code>null</code> if the parameter is not mapped or is not a parameter pack.

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPTemplateTemplateParameter.java

@@ -20,7 +20,6 @@ import org.eclipse.cdt.core.dom.ast.IType;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPTemplateTemplateParameter extends ICPPTemplateParameter, ICPPClassTemplate {
-
 	@Override
 	public ICPPTemplateParameter[] getTemplateParameters();
 	

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPTemplateTypeParameter.java

@@ -18,17 +18,15 @@ import org.eclipse.cdt.core.dom.ast.IType;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPTemplateTypeParameter extends ICPPTemplateParameter, IType {
-
 	/**
-	 * The default type for this parameter. May be null
+	 * The default type for this parameter. May be {@code null}.
-	 * 
 	 */
 	public IType getDefault() throws DOMException;
 
 	/**
 	 * Types containing template parameters need to be compared even before it is known to which
-	 * binding the template parameter belongs to. Therefore {@link #isSameType(IType)} compares the
+	 * binding the template parameter belongs to. Therefore {@link #isSameType(IType)} compares
-	 * kind and the parameter position of the template parameter, only. The name and the owner
+	 * the kind and the parameter position of the template parameter, only. The name and the owner
 	 * is ignored.
 	 * 
 	 * @since 5.1

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPUsingDeclaration.java

@@ -23,7 +23,6 @@ import org.eclipse.cdt.core.dom.ast.IBinding;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPUsingDeclaration extends ICPPBinding {	
-	
     /**
      * Return an array of bindings that were declared by this using declaration.
      * Each of these bindings delegates to some previously declared binding to which it

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPUsingDirective.java

@@ -23,7 +23,7 @@ import org.eclipse.cdt.internal.core.dom.parser.ASTNode;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPUsingDirective {
-	ICPPUsingDirective[] EMPTY_ARRAY = new ICPPUsingDirective[0];
+	ICPPUsingDirective[] EMPTY_ARRAY = {};
 
 	/**
 	 * Returns the scope of the namespace that is nominated by this

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/core/dom/ast/cpp/ICPPVariable.java

@@ -18,7 +18,6 @@ import org.eclipse.cdt.core.dom.ast.IVariable;
  * @noimplement This interface is not intended to be implemented by clients.
  */
 public interface ICPPVariable extends IVariable, ICPPBinding {
-	
     /**
      * does this variable have the mutable storage class specifier
      */

core/org.eclipse.cdt.core/parser/org/eclipse/cdt/internal/core/dom/parser/cpp/CPPASTLiteralExpression.java

@@ -59,13 +59,9 @@ public class CPPASTLiteralExpression extends ASTNode implements ICPPASTLiteralEx
 	
 	@Override
 	public CPPASTLiteralExpression copy(CopyStyle style) {
-		CPPASTLiteralExpression copy = new CPPASTLiteralExpression(kind,
+		CPPASTLiteralExpression copy =
-				value == null ? null : value.clone());
+				new CPPASTLiteralExpression(kind, value == null ? null : value.clone());
-		copy.setOffsetAndLength(this);
+		return copy(copy, style);
-		if (style == CopyStyle.withLocations) {
-			copy.setCopyLocation(this);
-		}
-		return copy;
 	}
 
 	@Override