Happy checkstyle
Added missing javadoc.
Change-Id: I39ff73f5e876b43a9cc07c7298d21f141436b346
Signed-off-by: Philip Langer <planger@eclipsesource.com>
diff --git a/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/internal/AbstractPapyrusResourceSetHook.java b/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/internal/AbstractPapyrusResourceSetHook.java
index c99ef60..8fe2b18 100644
--- a/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/internal/AbstractPapyrusResourceSetHook.java
+++ b/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/internal/AbstractPapyrusResourceSetHook.java
@@ -30,12 +30,8 @@
/**
* File extensions registered in Papyrus.
*/
- protected final Set<String> fileExtensions;
-
- public AbstractPapyrusResourceSetHook() {
- super();
- fileExtensions = ImmutableSet.copyOf(ModelExtensionUtil.getRegisteredFileExtensions());
- }
+ protected final Set<String> fileExtensions = ImmutableSet.copyOf(ModelExtensionUtil
+ .getRegisteredFileExtensions());
/**
* Hooks in when any of the file extensions registered in Papyrus are loaded. {@inheritDoc}
diff --git a/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/internal/ElementTypeSetConfigurationRegistryInitializingHook.java b/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/internal/ElementTypeSetConfigurationRegistryInitializingHook.java
index 324053e..6470cf0 100644
--- a/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/internal/ElementTypeSetConfigurationRegistryInitializingHook.java
+++ b/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/internal/ElementTypeSetConfigurationRegistryInitializingHook.java
@@ -36,10 +36,13 @@
*/
public class ElementTypeSetConfigurationRegistryInitializingHook extends AbstractPapyrusResourceSetHook {
+ /** The id of the bundle containing the class ElementTypeSetConfigurationRegistry. */
private static final String BUNDLE_ID = "org.eclipse.papyrus.infra.elementtypesconfigurations"; //$NON-NLS-1$
+ /** The method to be invoked in ElementTypeSetConfigurationRegistry. */
private static final String GET_INSTANCE = "getInstance"; //$NON-NLS-1$
+ /** The qualified class name of ElementTypeSetConfigurationRegistry. */
private static final String ELEMENTTYPESETCONFIGREG_CLASS_NAME = "org.eclipse.papyrus.infra.elementtypesconfigurations.registries.ElementTypeSetConfigurationRegistry"; //$NON-NLS-1$
@Override
@@ -75,6 +78,12 @@
});
}
+ /**
+ * Logs the specified exception <code>e</code>.
+ *
+ * @param e
+ * The exception to be logged.
+ */
private void logException(Exception e) {
CompareDiagramIDEUIPapyrusPlugin.getDefault().getLog().log(
new Status(IStatus.WARNING, CompareDiagramIDEUIPapyrusPlugin.PLUGIN_ID,
diff --git a/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/util/ModelExtensionUtil.java b/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/util/ModelExtensionUtil.java
index 617a8b5..1457f63 100644
--- a/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/util/ModelExtensionUtil.java
+++ b/plugins/org.eclipse.emf.compare.diagram.ide.ui.papyrus/src/org/eclipse/emf/compare/diagram/ide/ui/papyrus/util/ModelExtensionUtil.java
@@ -1,5 +1,5 @@
/*******************************************************************************
- * Copyright (c) 2015 EclipseSource Muenchen GmbH and others.
+ * Copyright (c) 2015, 2016 EclipseSource Muenchen GmbH and others.
* All rights reserved. This program and the accompanying materials
* are made available under the terms of the Eclipse Public License v1.0
* which accompanies this distribution, and is available at
@@ -8,6 +8,7 @@
* Contributors:
* Stefan Dirix - initial API and implementation
* Laurent Delaigue - Log invalid configuration
+ * Philip Langer - javadoc
*******************************************************************************/
package org.eclipse.emf.compare.diagram.ide.ui.papyrus.util;
@@ -191,17 +192,36 @@
return null;
}
- // since Papyrus 2.0 we have to implement canPersist
- // we omit @Override on purpose to be backward compatible
+ /**
+ * Stub implementation, which always returns <code>false</code>.
+ * <p>
+ * Since Papyrus 2.0, {@link AbstractBaseModel} forces us to implement
+ * {@link AbstractBaseModel#canPersist(EObject)}. We omit <code>@Override</code> on purpose to be
+ * backward compatible.
+ * </p>
+ *
+ * @param eObject
+ * This parameter will be ignored.
+ * @return <code>false</code>.
+ */
@SuppressWarnings("all")
public boolean canPersist(EObject eObject) {
return false;
}
- // since Papyrus 2.0 we have to implement persist
- // we omit @Override on purpose to be backward compatible
+ /**
+ * Stub implementation, which does nothing.
+ * <p>
+ * Since Papyrus 2.0, {@link AbstractBaseModel} forces us to implement
+ * {@link AbstractBaseModel#persist(EObject)}. We omit <code>@Override</code> on purpose to be
+ * backward compatible.
+ * </p>
+ *
+ * @param eObject
+ * This parameter will be ignored.
+ */
@SuppressWarnings("all")
- public void persist(EObject object) {
+ public void persist(EObject eObject) {
// no implementation
}
}
