Merge pull request dsldevkit#837 from joaodinissf/mrqa

Remove empty inheritDoc's + format all files

com.avaloq.tools.ddk.check.core.test/src/com/avaloq/tools/ddk/check/core/test/AbstractCheckTestCase.java

@@ -20,7 +20,7 @@
 import java.util.Set;
 
 import org.apache.logging.log4j.Logger;
-import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.LogManager;
 import org.eclipse.core.resources.IFile;
 import org.eclipse.core.resources.IProject;
 import org.eclipse.core.resources.IWorkspaceRoot;

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/CheckConstants.java

@@ -23,5 +23,6 @@ public final class CheckConstants {
   public static final String IT = "it"; //$NON-NLS-1$
   public static final QualifiedName QNIT = QualifiedName.create(IT);
 
-  private CheckConstants() {}
+  private CheckConstants() {
+  }
 }

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/CheckStandaloneSetup.java

@@ -21,4 +21,3 @@ public static void doSetup() {
     new CheckStandaloneSetup().createInjectorAndDoEMFRegistration();
   }
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/check/impl/CategoryImplCustom.java

@@ -30,4 +30,3 @@ public String getName() {
     }
   }
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/check/impl/DocumentedImplCustom.java

@@ -20,7 +20,6 @@
  */
 public class DocumentedImplCustom extends DocumentedImpl {
 
-  /** {@inheritDoc} */
   @Override
   public String getDescription() {
     if (!this.eIsProxy() && this.eResource() != null) {
@@ -36,4 +35,3 @@ public String getDescription() {
     return null;
   }
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/generator/CheckCompiler.java

@@ -193,7 +193,6 @@ protected void internalToConvertedExpression(final XExpression obj, final ITreeA
     }
   }
 
-  /** {@inheritDoc} */
   // CHECKSTYLE:OFF
   protected void _toJavaStatement(final XGuardExpression expr, final ITreeAppendable b, final boolean isReferenced) {
     // CHECKSTYLE:ON
@@ -225,14 +224,12 @@ protected void appendImmutableCollectionExpression(final XCollectionLiteral lite
     super.appendImmutableCollectionExpression(literal, b, collectionsMethod, guavaHelper, guavaHelperMethod);
   }
 
-  /** {@inheritDoc} */
   // CHECKSTYLE:OFF
   protected void _toJavaExpression(final XGuardExpression expr, final ITreeAppendable b) {
     // CHECKSTYLE:ON
     b.append(getVarName(expr, b));
   }
 
-  /** {@inheritDoc} */
   // CHECKSTYLE:OFF
   protected void _toJavaStatement(final XIssueExpression expr, final ITreeAppendable b, final boolean isReferenced) { // NOPMD NPath complexity
     // CHECKSTYLE:ON
@@ -364,7 +361,6 @@ private String getLoggerString(final XIssueExpression expr) {
         + " for \" + " + getContextImplicitVariableName(expr) + ".eResource().getURI() + \" at \" + stackTraceElements[1].toString()";
   }
 
-  /** {@inheritDoc} */
   // CHECKSTYLE:OFF
   @SuppressWarnings("restriction")
   protected JvmType findEPackageInterfaceType(final ENamedElement element, final EObject context) {
@@ -374,7 +370,6 @@ protected JvmType findEPackageInterfaceType(final ENamedElement element, final E
     return getTypeComputationServices().getTypeReferences().findDeclaredType(qualifiedPackageInterfaceName, context);
   }
 
-  /** {@inheritDoc} */
   // CHECKSTYLE:OFF
   protected void _toJavaExpression(final XIssueExpression expr, final ITreeAppendable b) {
     // CHECKSTYLE:ON

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/generator/CheckGeneratorConstants.java

@@ -23,4 +23,3 @@ private CheckGeneratorConstants() {
   }
 
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/naming/CheckDeclarativeQualifiedNameProvider.java

@@ -54,4 +54,3 @@ QualifiedName qualifiedName(final Check check) {
   }
 
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/resource/CheckLocationInFileProvider.java

@@ -47,4 +47,3 @@ public EStructuralFeature getIdentifierFeature(final EObject object) {
   }
 
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/resource/CheckResourceDescriptionStrategy.java

@@ -36,4 +36,3 @@ public boolean createEObjectDescriptions(final EObject eObject, final IAcceptor<
   }
 
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/scoping/CheckLinkingService.java

@@ -46,4 +46,3 @@ public List<EObject> getLinkedObjects(final EObject context, final EReference re
   }
 
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/serializer/CheckSemanticSequencer.java

@@ -8,10 +8,11 @@
  * Contributors:
  *     Avaloq Group AG - initial API and implementation
  *******************************************************************************/
-package  com.avaloq.tools.ddk.check.serializer;
+package com.avaloq.tools.ddk.check.serializer;
+
 /**
  * Semantic sequencer for Check DSL.
  */
-public class CheckSemanticSequencer  extends AbstractCheckSemanticSequencer {
+public class CheckSemanticSequencer extends AbstractCheckSemanticSequencer {
 
 }

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/serializer/CheckSyntacticSequencer.java

@@ -8,7 +8,7 @@
  * Contributors:
  *     Avaloq Group AG - initial API and implementation
  *******************************************************************************/
-package  com.avaloq.tools.ddk.check.serializer;
+package com.avaloq.tools.ddk.check.serializer;
 
 /**
  * Syntactic sequencer for Check DSL.

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/typing/CheckExpressionHelper.java

@@ -31,4 +31,3 @@ public boolean hasSideEffects(final XExpression expr) {
   }
 
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/util/GrammarAccessHelper.java

@@ -37,4 +37,3 @@ public String getBundleSymbolicName() {
   }
 
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/util/GrammarHelper.java

@@ -135,4 +135,3 @@ public List<String> getMetamodelPackages() {
   }
 
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/validation/ClasspathBasedChecks.java

@@ -74,7 +74,8 @@ public void checkFileNamingConventions(final CheckCatalog catalog) {
     }
     String catalogName = catalog.getName();
     if (catalogName != null && !equal(resourceURI.trimFileExtension().lastSegment(), catalogName)) {
-      error("The catalog '" + (packageName != null ? notNull(packageName) + DOT : "") + catalogName + "' must be defined in its own file", catalog, CheckPackage.Literals.CHECK_CATALOG__NAME, ValidationMessageAcceptor.INSIGNIFICANT_INDEX, IssueCodes.WRONG_FILE);
+      error("The catalog '" + (packageName != null ? notNull(packageName) + DOT : "") + catalogName
+          + "' must be defined in its own file", catalog, CheckPackage.Literals.CHECK_CATALOG__NAME, ValidationMessageAcceptor.INSIGNIFICANT_INDEX, IssueCodes.WRONG_FILE);
     }
   }
 
@@ -90,9 +91,11 @@ public void checkFileNamingConventions(final CheckCatalog catalog) {
    */
   protected void reportInvalidPackage(final CheckCatalog catalog, final String givenPackageName, final String expectedPackageName) {
     if (expectedPackageName != null) {
-      error("The declared package '" + notNull(givenPackageName) + "' does not match the expected package", catalog, CheckPackage.Literals.CHECK_CATALOG__PACKAGE_NAME, ValidationMessageAcceptor.INSIGNIFICANT_INDEX, IssueCodes.WRONG_PACKAGE, expectedPackageName);
+      error("The declared package '" + notNull(givenPackageName)
+          + "' does not match the expected package", catalog, CheckPackage.Literals.CHECK_CATALOG__PACKAGE_NAME, ValidationMessageAcceptor.INSIGNIFICANT_INDEX, IssueCodes.WRONG_PACKAGE, expectedPackageName);
     } else {
-      error("The declared package '" + notNull(givenPackageName) + "' does not match the expected package", catalog, CheckPackage.Literals.CHECK_CATALOG__PACKAGE_NAME, ValidationMessageAcceptor.INSIGNIFICANT_INDEX, IssueCodes.WRONG_PACKAGE);
+      error("The declared package '" + notNull(givenPackageName)
+          + "' does not match the expected package", catalog, CheckPackage.Literals.CHECK_CATALOG__PACKAGE_NAME, ValidationMessageAcceptor.INSIGNIFICANT_INDEX, IssueCodes.WRONG_PACKAGE);
     }
   }
 }

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/validation/FormalParameterCheckBase.java

@@ -60,4 +60,3 @@ protected void checkRightHandSideHasOnlyIntegralNumbers(final XExpression value,
   }
 
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/validation/FormalParameterChecks.java

@@ -32,4 +32,3 @@ public void checkFormalParameterNumbersAreIntegers(final FormalParameter paramet
   }
 
 }
-

com.avaloq.tools.ddk.check.core/src/com/avaloq/tools/ddk/check/validation/UniquenessValidationHelper.java

@@ -90,4 +90,3 @@ public Function<T, QualifiedName> getNameFunction() {
   }
 
 }
-

com.avaloq.tools.ddk.check.ide/src/com/avaloq/tools/ddk/check/ide/CheckIdeModule.java

@@ -3,7 +3,6 @@
  */
 package com.avaloq.tools.ddk.check.ide;
 
-
 /**
  * Use this class to register ide components.
  */

com.avaloq.tools.ddk.check.lib/src/com/avaloq/tools/ddk/check/lib/internal/Index.java

@@ -45,7 +45,6 @@ protected Index() {
     // Prevent explicit instantiations. To be used via injection.
   }
 
-  /** {@inheritDoc} */
   @Override
   public IIndex.Query newQuery(final EClass type, final String namePattern) {
     if (type == null || namePattern == null || namePattern.length() == 0) {
@@ -54,7 +53,6 @@ public IIndex.Query newQuery(final EClass type, final String namePattern) {
     return new Query(domainMapper, nameConverter, type).withName(namePattern);
   }
 
-  /** {@inheritDoc} */
   @Override
   public IIndex.Query newQuery(final EClass type, final QualifiedName namePattern) {
     if (type == null || namePattern == null || namePattern.isEmpty()) {
@@ -114,7 +112,6 @@ private Query withName(final QualifiedName namePattern) {
       return this;
     }
 
-    /** {@inheritDoc} */
     @Override
     public Query withData(final String key, final String value) {
       if (key == null || value == null) {
@@ -124,7 +121,6 @@ public Query withData(final String key, final String value) {
       return this;
     }
 
-    /** {@inheritDoc} */
     @Override
     public Iterable<IIndex.Entry> run(final EObject context) {
       if (context == null) {
@@ -165,25 +161,21 @@ protected Entry(final EObject context, final IQualifiedNameConverter nameConvert
       this.delegate = internalDescription;
     }
 
-    /** {@inheritDoc} */
     @Override
     public String getName() {
       return nameConverter.toString(getQualifiedName());
     }
 
-    /** {@inheritDoc} */
     @Override
     public QualifiedName getQualifiedName() {
       return delegate.getQualifiedName();
     }
 
-    /** {@inheritDoc} */
     @Override
     public EClass getType() {
       return delegate.getEClass();
     }
 
-    /** {@inheritDoc} */
     @Override
     public EObject getModelObject() {
       EObject result = delegate.getEObjectOrProxy();
@@ -193,7 +185,6 @@ public EObject getModelObject() {
       return result;
     }
 
-    /** {@inheritDoc} */
     @Override
     public String[] getDataKeys() {
       String[] result = delegate.getUserDataKeys();
@@ -203,7 +194,6 @@ public String[] getDataKeys() {
       return result;
     }
 
-    /** {@inheritDoc} */
     @Override
     public String getData(final String key) {
       if (key == null) {
@@ -212,7 +202,6 @@ public String getData(final String key) {
       return delegate.getUserData(key);
     }
 
-    /** {@inheritDoc} */
     @Override
     public String getSourceName() {
       final URI uri = delegate.getEObjectURI().trimFragment();

com.avaloq.tools.ddk.check.lib/src/com/avaloq/tools/ddk/check/lib/internal/Messages.java

@@ -27,6 +27,6 @@ public class Messages extends NLS {
     NLS.initializeMessages(BUNDLE_NAME, Messages.class);
   }
 
-  private Messages() {}
+  private Messages() {
+  }
 }
-

com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/context/CheckContextTest.java

@@ -43,4 +43,3 @@ public void testIssuesEnabledDisabled() {
 
   }
 }
-

com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/context/DummyCheckContext.java

@@ -56,4 +56,3 @@ public boolean enabledForCodes(final EObject context) {
   }
 
 }
-

com.avaloq.tools.ddk.check.runtime.core.test/src/com/avaloq/tools/ddk/check/runtime/core/validation/CheckValidatorTest.java

@@ -50,4 +50,3 @@ public void testAtLeastOneValidatorFound() {
     assertEquals("Dummy validator found", "DummyValidator", dummyValidator.getClass().getSimpleName());
   }
 }
-

com.avaloq.tools.ddk.check.runtime.core/src/com/avaloq/tools/ddk/check/runtime/CheckRuntimeConstants.java

@@ -22,5 +22,6 @@ public final class CheckRuntimeConstants {
   /** Suffix used in issue codes class names. */
   public static final String ISSUE_CODES_CLASS_NAME_SUFFIX = "IssueCodes"; //$NON-NLS-1$
 
-  private CheckRuntimeConstants() {}
+  private CheckRuntimeConstants() {
+  }
 }

com.avaloq.tools.ddk.check.runtime.core/src/com/avaloq/tools/ddk/check/runtime/configuration/AbstractModelLocation.java

@@ -17,7 +17,7 @@
 import java.net.URL;
 
 import org.apache.logging.log4j.Logger;
-import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.LogManager;
 import org.eclipse.osgi.util.NLS;
 
 
@@ -37,7 +37,6 @@ public AbstractModelLocation(final URL catalogUrl) {
     this.catalogUrl = catalogUrl;
   }
 
-  /** {@inheritDoc} */
   @Override
   public InputStream getCatalogStream() {
     InputStream stream = null;
@@ -49,19 +48,16 @@ public InputStream getCatalogStream() {
     return stream;
   }
 
-  /** {@inheritDoc} */
   @Override
   public String getCatalogPath() {
     return catalogUrl.getPath();
   }
 
-  /** {@inheritDoc} */
   @Override
   public URL getCatalogUrl() {
     return catalogUrl;
   }
 
-  /** {@inheritDoc} */
   @Override
   public URI getCatalogUri() {
     try {

com.avaloq.tools.ddk.check.runtime.core/src/com/avaloq/tools/ddk/check/runtime/configuration/ICheckConfigurationStoreService.java

@@ -38,4 +38,3 @@ public interface ICheckConfigurationStoreService {
   ICheckConfigurationStore getCheckConfigurationStore(Object context);
 
 }
-

com.avaloq.tools.ddk.check.runtime.core/src/com/avaloq/tools/ddk/check/runtime/configuration/ModelLocation.java

@@ -31,4 +31,3 @@ public String getCatalogPath() {
   }
 
 }
-

com.avaloq.tools.ddk.check.runtime.core/src/com/avaloq/tools/ddk/check/runtime/context/AbstractCheckContext.java

@@ -16,7 +16,7 @@
 import java.util.Set;
 
 import org.apache.logging.log4j.Logger;
-import org.apache.logging.log4j.LogManager;
+import org.apache.logging.log4j.LogManager;
 import org.eclipse.emf.ecore.EObject;
 
 import com.google.common.collect.HashMultimap;
@@ -42,7 +42,6 @@ public class AbstractCheckContext implements ICheckContext {
   /** Maps issue codes to the predicate methods that contain that issue code in their annotation. */
   private Multimap<String, Method> predicatesForIssueCode;
 
-  /** {@inheritDoc} */
   @Override
   public boolean isCheckValid(final EObject context, final String issueCode) {
     if (predicatesForIssueCode == null) {

com.avaloq.tools.ddk.check.runtime.core/src/com/avaloq/tools/ddk/check/runtime/context/CheckContext.java

@@ -31,4 +31,3 @@
   String[] issueCodes();
 
 }
-

com.avaloq.tools.ddk.check.runtime.core/src/com/avaloq/tools/ddk/check/runtime/context/DefaultCheckContext.java

@@ -28,4 +28,3 @@ public boolean isCheckValid(final EObject context, final String issueCode) {
   }
 
 }
-

com.avaloq.tools.ddk.check.runtime.core/src/com/avaloq/tools/ddk/check/runtime/context/ICheckContext.java

@@ -46,4 +46,3 @@ public boolean isCheckValid(final EObject context, final String issueCode) {
 
   }
 }
-

com.avaloq.tools.ddk.check.runtime.core/src/com/avaloq/tools/ddk/check/runtime/internal/AbstractCheckRegistryReader.java

@@ -69,7 +69,6 @@ abstract class AbstractCheckRegistryReader extends RegistryReader {
    */
   protected abstract Logger getLogger();
 
-  /** {@inheritDoc} */
   @Override
   protected void logError(final IConfigurationElement element, final String text) {
     IExtension extension = element.getDeclaringExtension();
@@ -104,7 +103,6 @@ protected String getKeyAttribute() {
    */
   abstract String getAttribute();
 
-  /** {@inheritDoc} */
   @Override
   protected boolean readElement(final IConfigurationElement element, final boolean add) {
     if (element.getName().equals(topLevelElementName())) {