EDIT: Removed unnecessary syntax.

Author: tsaglam

src/main/java/jce/codemanipulation/ecore/EcoreImportManipulator.java

@@ -65,7 +65,7 @@ private void addQualifiedNamesToTypeReferences(ICompilationUnit unit, String typ
      * imports.
      */
     private void applyRetentionVisitor(ICompilationUnit unit, IImportDeclaration[] imports) throws JavaModelException {
-        ASTVisitor visitor = new TypeRetentionVisitor(unit, imports, properties);
+        ASTVisitor visitor = new TypeRetentionVisitor(unit, imports);
         ASTUtil.applyVisitorModifications(unit, visitor, monitor);
     }
 

src/main/java/jce/codemanipulation/ecore/FactoryInitMethodCorrectionVisitor.xtend

@@ -9,13 +9,13 @@ import org.eclipse.jdt.core.dom.ReturnStatement
  * @author Heiko Klare
  */
 class FactoryInitMethodCorrectionVisitor extends ASTVisitor {
-	private static final String FACTORY_INIT_METHOD_NAME = "init";
+	static final String FACTORY_INIT_METHOD_NAME = "init";
 
 	/**
 	 * Removes the namespace-URI-based factory determination logic from the original Ecore factories, 
 	 * because they are are only used internally and are not registered globally. 
 	 */
-	public override boolean visit(MethodDeclaration node) {
+	override boolean visit(MethodDeclaration node) {
 		if (node.name.toString == FACTORY_INIT_METHOD_NAME) {
 			node.body.statements.removeIf[!(it instanceof ReturnStatement)]
 		}

src/main/java/jce/codemanipulation/ecore/PackageImplFactoryCorrectionVisitor.xtend

@@ -23,15 +23,13 @@ import jce.util.PathHelper
 class PackageImplFactoryCorrectionVisitor extends ASTVisitor {
 	public static val PACKAGE_IMPL_SUFFIX = "PackageImpl";
 	public static val PACKAGE_SUFFIX = "Package";
-	private static val FACTORY_SUFFIX = "Factory";
-	private val String factoryName;
-	private val ICompilationUnit compilationUnit;
-	private val EcorificationProperties properties;
-	private val PathHelper pathHelper;
+	static val FACTORY_SUFFIX = "Factory";
+	val String factoryName;
+	val EcorificationProperties properties;
+	val PathHelper pathHelper;
 
 	new(ICompilationUnit compilationUnit, EcorificationProperties properties) {
 		this.properties = properties;
-		this.compilationUnit = compilationUnit;
 		this.pathHelper = new PathHelper(".")
 		val compilationUnitName = pathHelper.cutLastSegment(compilationUnit.elementName);
 		this.factoryName = if (compilationUnit.elementName.contains(PACKAGE_IMPL_SUFFIX)) {

src/main/java/jce/codemanipulation/ecore/PackageImplFactoryCorrector.xtend

@@ -18,8 +18,8 @@ import static extension jce.util.EPackageUtil.*;
  * @author Heiko Klare
  */
 class PackageImplFactoryCorrector extends AbstractCodeManipulator {
-	private val extension PathHelper pathHelper;
-	private val GeneratedEcoreMetamodel metamodel;
+	val extension PathHelper pathHelper;
+	val GeneratedEcoreMetamodel metamodel;
 
 	new(GeneratedEcoreMetamodel metamodel, EcorificationProperties properties) {
 		super(properties.get(TextProperty.ECORE_PACKAGE), properties)

src/main/java/jce/codemanipulation/ecore/TypeRetentionVisitor.xtend

@@ -31,7 +31,6 @@ class TypeRetentionVisitor extends ASTVisitor {
 	final String currentPackage
 	final IImportDeclaration[] imports
 	final PathHelper pathHelper
-	final EcorificationProperties properties
 
 	/** 
 	 * Basic constructor. Creates the visitor.
@@ -40,9 +39,8 @@ class TypeRetentionVisitor extends ASTVisitor {
 	 * @param properties are the {@link EcorificationProperties}.
 	 * @throws JavaModelException if there is a problem with the {@link ICompilationUnit}.
 	 */
-	new(ICompilationUnit compilationUnit, IImportDeclaration[] imports, EcorificationProperties properties) throws JavaModelException {
+	new(ICompilationUnit compilationUnit, IImportDeclaration[] imports) throws JavaModelException {
 		super()
-		this.properties = properties
 		this.imports = imports
 		currentPackage = compilationUnit.parent.elementName
 		pathHelper = new PathHelper('.')

src/main/java/jce/generators/ClassGenerator.xtend

@@ -32,7 +32,7 @@ class ClassGenerator {
 	/**
 	 * Creates an IFile from a project relative path, a file name and creates the file content.
 	 */
-	def public void createClass(String path, String name, String content, IProject project) {
+	def void createClass(String path, String name, String content, IProject project) {
 		var folder = project.getFolder(append(SOURCE_FOLDER.get, path))
 		var file = folder.getFile(name)
 		if(file.exists) {
@@ -46,7 +46,7 @@ class ClassGenerator {
 	/**
 	 * Creates an {@link IFolder} in the project with a project relative path.
 	 */
-	def public void createFolder(String path, IProject project) {
+	def void createFolder(String path, IProject project) {
 		var IFolder folder = project.getFolder(path)
 		if(!folder.exists) {
 			folder.create(false, true, monitor)

src/main/java/jce/generators/ConstructorRepresentation.xtend

@@ -17,7 +17,6 @@ import org.eclipse.xtend.lib.annotations.Accessors
 @Accessors(PUBLIC_GETTER)
 class ConstructorRepresentation {
 	@Accessors(NONE) static final Logger logger = LogManager.getLogger(ConstructorRepresentation.getName)
-	@Accessors(NONE) MethodDeclaration declaration
 	@Accessors(NONE) List<SingleVariableDeclaration> parameters
 	List<String> imports
 	String content
@@ -27,7 +26,6 @@ class ConstructorRepresentation {
 	 * correlating compilation unit. 
 	 */
 	new(MethodDeclaration declaration, ICompilationUnit unit) {
-		this.declaration = declaration
 		parameters = new LinkedList
 		for (parameter : declaration.parameters) {
 			addParameter(parameter)

src/main/java/jce/generators/EFactoryGenerator.xtend

@@ -24,7 +24,7 @@ class EFactoryGenerator extends ClassGenerator {
 	/**
 	 * Creates a Ecore Factory in a package path with a specific name. 
 	 */
-	def public void create(String path, List<String> packageTypes, IProject project) {
+	def void create(String path, List<String> packageTypes, IProject project) {
 		val currentPackage = path.replace(File.separatorChar, '.') // path to package declaration
 		val packageName = currentPackage.getLastSegment.toFirstUpper
 		val content = createFactoryContent(currentPackage, packageName, packageTypes)

src/main/java/jce/generators/EFactoryImplementationGenerator.xtend

@@ -24,7 +24,7 @@ class EFactoryImplementationGenerator extends ClassGenerator {
 	/**
 	 * Creates a Ecore Factory in a package path with a specific name. 
 	 */
-	def public void create(String path, List<String> packageTypes, IProject project) {
+	def void create(String path, List<String> packageTypes, IProject project) {
 		val currentPackage = path.replace(File.separatorChar, '.') // path to package declaration
 		val interfacePackage = currentPackage.cutLastSegment
 		val packageName = interfacePackage.getLastSegment.toFirstUpper

src/main/java/jce/generators/EcoreFactoryGenerator.xtend

@@ -5,11 +5,9 @@ import java.io.File
 import jce.properties.EcorificationProperties
 import jce.util.PathHelper
 import jce.util.ResourceRefresher
-import jce.util.logging.MonitorFactory
 import org.apache.log4j.LogManager
 import org.apache.log4j.Logger
 import org.eclipse.core.resources.IProject
-import org.eclipse.core.runtime.IProgressMonitor
 import org.eclipse.emf.ecore.EPackage
 
 import static jce.properties.TextProperty.ECORE_PACKAGE
@@ -26,7 +24,6 @@ final class EcoreFactoryGenerator {
 	extension final EcorificationProperties properties
 
 	static final Logger logger = LogManager.getLogger(EcoreFactoryGenerator.getName)
-	final IProgressMonitor monitor
 	final EFactoryGenerator factoryGenerator
 	final EFactoryImplementationGenerator factoryImplementationGenerator
 
@@ -37,7 +34,6 @@ final class EcoreFactoryGenerator {
 		this.properties = properties
 		factoryGenerator = new EFactoryGenerator(properties)
 		factoryImplementationGenerator = new EFactoryImplementationGenerator(properties)
-		monitor = MonitorFactory.createProgressMonitor(logger, properties)
 		pathUtil = new PathHelper(File.separatorChar)
 	}