metafacture-javaintegration/ (main): Fix Checkstyle violations.

Author: blackwinter

metafacture-javaintegration/src/main/java/org/metafacture/javaintegration/Collector.java

@@ -13,6 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package org.metafacture.javaintegration;
 
 import java.util.Collection;
@@ -25,7 +26,7 @@
  * @author Markus Michael Geipel
  *
  */
-public interface Collector <V> {
+public interface Collector<V> {
 
     Collection<V> getCollection();
 

metafacture-javaintegration/src/main/java/org/metafacture/javaintegration/EventList.java

@@ -13,14 +13,15 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package org.metafacture.javaintegration;
 
+import org.metafacture.framework.StreamReceiver;
+
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.List;
 
-import org.metafacture.framework.StreamReceiver;
-
 /**
  * Stores the received stream events in a list.
  *
@@ -33,6 +34,9 @@ public final class EventList implements StreamReceiver {
 
     private boolean closed;
 
+    public EventList() {
+    }
+
     public List<Event> getEvents() {
         return Collections.unmodifiableList(events);
     }
@@ -131,7 +135,7 @@ public String toString() {
             final StringBuilder builder = new StringBuilder();
             builder.append(type);
             if (name != null) {
-                builder.append("(" );
+                builder.append("(");
                 builder.append(name);
                 if (value != null) {
                     builder.append("=");

metafacture-javaintegration/src/main/java/org/metafacture/javaintegration/MapToStream.java

@@ -13,9 +13,8 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package org.metafacture.javaintegration;
 
-import java.util.Map;
+package org.metafacture.javaintegration;
 
 import org.metafacture.framework.FluxCommand;
 import org.metafacture.framework.StandardEventNames;
@@ -24,6 +23,8 @@
 import org.metafacture.framework.annotations.Out;
 import org.metafacture.framework.helpers.DefaultObjectPipe;
 
+import java.util.Map;
+
 /**
  * Emits a {@link Map} as a record with a literal for each entry in the map.
  * When receiving an empty map only a <i>start-record</i> and <i>end-record</i>
@@ -59,11 +60,13 @@
 @In(Map.class)
 @Out(StreamReceiver.class)
 @FluxCommand("map-to-stream")
-public final class MapToStream extends
-        DefaultObjectPipe<Map<?, ?>, StreamReceiver> {
+public final class MapToStream extends DefaultObjectPipe<Map<?, ?>, StreamReceiver> {
 
     private Object idKey = StandardEventNames.ID;
 
+    public MapToStream() {
+    }
+
     /**
      * Sets the key of the map entry that is used for the record id.
      * <p>
@@ -88,7 +91,8 @@ public void process(final Map<?, ?> map) {
         final Object id = map.get(idKey);
         if (id == null) {
             getReceiver().startRecord("");
-        } else {
+        }
+        else {
             getReceiver().startRecord(id.toString());
         }
         for (final Map.Entry<?, ?> entry: map.entrySet()) {

metafacture-javaintegration/src/main/java/org/metafacture/javaintegration/NamedValueList.java

@@ -13,37 +13,34 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package org.metafacture.javaintegration;
 
+import org.metafacture.commons.types.NamedValue;
+import org.metafacture.framework.helpers.DefaultStreamReceiver;
+
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Iterator;
 import java.util.List;
 import java.util.ListIterator;
 
-import org.metafacture.commons.types.NamedValue;
-import org.metafacture.framework.helpers.DefaultStreamReceiver;
-
-
 /**
  * Collects the received results in a {@link List}.
  *
  * @author Markus Michael Geipel
  *
  */
-public final class NamedValueList extends DefaultStreamReceiver
-        implements List<NamedValue>, Collector<List<NamedValue>> {
+public final class NamedValueList extends DefaultStreamReceiver implements List<NamedValue>, Collector<List<NamedValue>> {
 
     private Collection<List<NamedValue>> collection;
     private List<NamedValue> list;
 
     public NamedValueList() {
-        super();
         list = new ArrayList<NamedValue>();
     }
 
     public NamedValueList(final Collection<List<NamedValue>> collection) {
-        super();
         list = new ArrayList<NamedValue>();
         this.collection = collection;
     }
@@ -83,36 +80,44 @@ public boolean add(final NamedValue element) {
         return list.add(element);
     }
 
+    @Override
+    public void add(final int index, final NamedValue element) {
+        list.add(index, element);
+    }
+
     @Override
     public boolean remove(final Object object) {
         return list.remove(object);
     }
 
     @Override
-    public boolean containsAll(final Collection<?> collection) {
-        return list.containsAll(collection);
+    public NamedValue remove(final int index) {
+        return list.remove(index);
     }
 
     @Override
-    public boolean addAll(final Collection<? extends NamedValue> collection) {
-        return list.addAll(collection);
+    public boolean containsAll(final Collection<?> currentCollection) {
+        return list.containsAll(currentCollection);
     }
 
     @Override
-    public boolean addAll(final int index, final Collection<? extends NamedValue> collection) {
-        return list.addAll(index, collection);
+    public boolean addAll(final Collection<? extends NamedValue> currentCollection) {
+        return list.addAll(currentCollection);
     }
 
     @Override
-    public boolean removeAll(final Collection<?> collection) {
-
-        return list.removeAll(collection);
+    public boolean addAll(final int index, final Collection<? extends NamedValue> currentCollection) {
+        return list.addAll(index, currentCollection);
     }
 
     @Override
-    public boolean retainAll(final Collection<?> collection) {
+    public boolean removeAll(final Collection<?> currentCollection) {
+        return list.removeAll(currentCollection);
+    }
 
-        return list.retainAll(collection);
+    @Override
+    public boolean retainAll(final Collection<?> currentCollection) {
+        return list.retainAll(currentCollection);
     }
 
     @Override
@@ -130,16 +135,6 @@ public NamedValue set(final int index, final NamedValue element) {
         return list.set(index, element);
     }
 
-    @Override
-    public void add(final int index, final NamedValue element) {
-        list.add(index, element);
-    }
-
-    @Override
-    public NamedValue remove(final int index) {
-        return list.remove(index);
-    }
-
     @Override
     public int indexOf(final Object object) {
         return list.indexOf(object);
@@ -166,7 +161,7 @@ public List<NamedValue> subList(final int fromIndex, final int toIndex) {
     }
 
     @Override
-    public void startRecord(final String identifier){
+    public void startRecord(final String identifier) {
         list.clear();
     }
 

metafacture-javaintegration/src/main/java/org/metafacture/javaintegration/NamedValueSet.java

@@ -13,31 +13,29 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package org.metafacture.javaintegration;
 
+import org.metafacture.commons.types.NamedValue;
+import org.metafacture.framework.helpers.DefaultStreamReceiver;
+
 import java.util.Collection;
 import java.util.HashSet;
 import java.util.Iterator;
 import java.util.Set;
 
-import org.metafacture.commons.types.NamedValue;
-import org.metafacture.framework.helpers.DefaultStreamReceiver;
-
-
 /**
  * Collects {@link NamedValue}s in a {@link Set}. So there will not be
  * duplicates.
  *
  * @author Markus Michael Geipel
  */
-public final class NamedValueSet extends DefaultStreamReceiver
-        implements Set<NamedValue>, Collector<Set<NamedValue>> {
+public final class NamedValueSet extends DefaultStreamReceiver implements Set<NamedValue>, Collector<Set<NamedValue>> {
 
     private Collection<Set<NamedValue>> collection;
     private Set<NamedValue> set;
 
     public NamedValueSet() {
-        super();
         set = new HashSet<>();
         this.collection = null;
 
@@ -50,7 +48,6 @@ public NamedValueSet() {
      * @param collection is filled with the received results.
      */
     public NamedValueSet(final Collection<Set<NamedValue>> collection) {
-        super();
         set = new HashSet<>();
         this.collection = collection;
     }

metafacture-javaintegration/src/main/java/org/metafacture/javaintegration/ObjectCollector.java

@@ -13,13 +13,14 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package org.metafacture.javaintegration;
 
+import org.metafacture.framework.ObjectReceiver;
+
 import java.util.LinkedList;
 import java.util.Queue;
 
-import org.metafacture.framework.ObjectReceiver;
-
 /**
  * Collects the objects emitted by an upstream module.
  *
@@ -40,7 +41,6 @@ public ObjectCollector() {
     }
 
     public ObjectCollector(final int maxCapacity) {
-        super();
         this.maxCapacity = maxCapacity;
     }
 

metafacture-javaintegration/src/main/java/org/metafacture/javaintegration/SingleValue.java

@@ -13,32 +13,29 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-package org.metafacture.javaintegration;
 
-import java.util.Collection;
+package org.metafacture.javaintegration;
 
 import org.metafacture.framework.helpers.DefaultStreamReceiver;
 
+import java.util.Collection;
 
 /**
  * just records the value of the last received literal.
  * @author Markus Michael Geipel
  *
  */
-public final class SingleValue extends DefaultStreamReceiver
-        implements Collector<String> {
+public final class SingleValue extends DefaultStreamReceiver implements Collector<String> {
 
     private boolean closed;
     private String value = "";
     private Collection<String> collection;
 
     public SingleValue() {
-        super();
         collection = null;
     }
 
     public SingleValue(final Collection<String> collection) {
-        super();
         this.collection = collection;
     }
 
@@ -72,9 +69,9 @@ public void endRecord() {
     }
 
     @Override
-    public void literal(final String name, final String value) {
+    public void literal(final String name, final String newValue) {
         assert !closed;
-        this.setValue(value);
+        setValue(newValue);
     }
 
     @Override