Fix checkstyle if-statements must use braces sal-common-impl

[controller.git] / opendaylight / md-sal / sal-common-impl / src / main / java / org / opendaylight / controller / md / sal / common / impl / AbstractDataModification.java

diff --git a/opendaylight/md-sal/sal-common-impl/src/main/java/org/opendaylight/controller/md/sal/common/impl/AbstractDataModification.java b/opendaylight/md-sal/sal-common-impl/src/main/java/org/opendaylight/controller/md/sal/common/impl/AbstractDataModification.java
index 4b6a0185ab81ecc1bdaf2799de15b1a6af33ff7b..1d3967f94d0b0883cfbaea548745dfa2a92fb01b 100644 (file)
--- a/opendaylight/md-sal/sal-common-impl/src/main/java/org/opendaylight/controller/md/sal/common/impl/AbstractDataModification.java
+++ b/opendaylight/md-sal/sal-common-impl/src/main/java/org/opendaylight/controller/md/sal/common/impl/AbstractDataModification.java
@@ -10,28 +10,28 @@ package org.opendaylight.controller.md.sal.common.impl;
 import static org.opendaylight.controller.md.sal.common.api.TransactionStatus.NEW;
 
 import java.util.Collections;
+import java.util.LinkedHashMap;
 import java.util.Map;
 import java.util.Set;
-import java.util.concurrent.ConcurrentHashMap;
-import java.util.concurrent.ConcurrentMap;
 
 import org.opendaylight.controller.md.sal.common.api.data.DataModification;
 import org.opendaylight.controller.md.sal.common.api.data.DataReader;
 import org.opendaylight.yangtools.concepts.Path;
 
+@Deprecated
 public abstract class AbstractDataModification<P extends Path<P>, D> implements DataModification<P, D> {
 
-    private final ConcurrentMap<P, D> operationalOriginal;
-    private final ConcurrentMap<P, D> configurationOriginal;
+    private final Map<P, D> operationalOriginal;
+    private final Map<P, D> configurationOriginal;
 
-    private final ConcurrentMap<P, D> operationalCreated;
-    private final ConcurrentMap<P, D> configurationCreated;
+    private final Map<P, D> operationalCreated;
+    private final Map<P, D> configurationCreated;
 
-    private final ConcurrentMap<P, D> configurationUpdate;
-    private final ConcurrentMap<P, D> operationalUpdate;
+    private final Map<P, D> configurationUpdate;
+    private final Map<P, D> operationalUpdate;
 
-    private final ConcurrentMap<P, P> configurationRemove;
-    private final ConcurrentMap<P, P> operationalRemove;
+    private final Map<P, P> configurationRemove;
+    private final Map<P, P> operationalRemove;
 
     private final Map<P, D> unmodifiable_configurationOriginal;
     private final Map<P, D> unmodifiable_operationalOriginal;
@@ -41,20 +41,20 @@ public abstract class AbstractDataModification<P extends Path<P>, D> implements
     private final Map<P, D> unmodifiable_operationalUpdate;
     private final Set<P> unmodifiable_configurationRemove;
     private final Set<P> unmodifiable_OperationalRemove;
-    private DataReader<P, D> reader;
+    private final DataReader<P, D> reader;
 
-    public AbstractDataModification(DataReader<P, D> reader) {
+    public AbstractDataModification(final DataReader<P, D> reader) {
         this.reader = reader;
-        this.configurationUpdate = new ConcurrentHashMap<>();
-        this.operationalUpdate = new ConcurrentHashMap<>();
-        this.configurationRemove = new ConcurrentHashMap<>();
-        this.operationalRemove = new ConcurrentHashMap<>();
+        this.configurationUpdate = new LinkedHashMap<>();
+        this.operationalUpdate = new LinkedHashMap<>();
+        this.configurationRemove = new LinkedHashMap<>();
+        this.operationalRemove = new LinkedHashMap<>();
 
-        this.configurationOriginal = new ConcurrentHashMap<>();
-        this.operationalOriginal = new ConcurrentHashMap<>();
+        this.configurationOriginal = new LinkedHashMap<>();
+        this.operationalOriginal = new LinkedHashMap<>();
 
-        this.configurationCreated = new ConcurrentHashMap<>();
-        this.operationalCreated = new ConcurrentHashMap<>();
+        this.configurationCreated = new LinkedHashMap<>();
+        this.operationalCreated = new LinkedHashMap<>();
 
         unmodifiable_configurationOriginal = Collections.unmodifiableMap(configurationOriginal);
         unmodifiable_operationalOriginal = Collections.unmodifiableMap(operationalOriginal);
@@ -67,7 +67,7 @@ public abstract class AbstractDataModification<P extends Path<P>, D> implements
     }
 
     @Override
-    public final void putConfigurationData(P path, D data) {
+    public final void putConfigurationData(final P path, final D data) {
         checkMutable();
         D original = null;
         if ((original = getConfigurationOriginal(path)) == null) {
@@ -78,7 +78,7 @@ public abstract class AbstractDataModification<P extends Path<P>, D> implements
     }
 
     @Override
-    public final void putOperationalData(P path, D data) {
+    public final void putOperationalData(final P path, final D data) {
         checkMutable();
         D original = null;
         if ((original = getOperationalOriginal(path)) == null) {
@@ -88,12 +88,7 @@ public abstract class AbstractDataModification<P extends Path<P>, D> implements
     }
 
     @Override
-    public final void putRuntimeData(P path, D data) {
-        putOperationalData(path, data);
-    }
-
-    @Override
-    public final void removeOperationalData(P path) {
+    public final void removeOperationalData(final P path) {
         checkMutable();
         getOperationalOriginal(path);
         operationalUpdate.remove(path);
@@ -101,12 +96,7 @@ public abstract class AbstractDataModification<P extends Path<P>, D> implements
     }
 
     @Override
-    public final void removeRuntimeData(P path) {
-        removeOperationalData(path);
-    }
-
-    @Override
-    public final void removeConfigurationData(P path) {
+    public final void removeConfigurationData(final P path) {
         checkMutable();
         getConfigurationOriginal(path);
         configurationUpdate.remove(path);
@@ -114,8 +104,9 @@ public abstract class AbstractDataModification<P extends Path<P>, D> implements
     }
 
     private final void checkMutable() {
-        if (!NEW.equals(this.getStatus()))
+        if (!NEW.equals(this.getStatus())) {
             throw new IllegalStateException("Transaction was already submitted");
+        }
     }
 
     @Override
@@ -160,46 +151,46 @@ public abstract class AbstractDataModification<P extends Path<P>, D> implements
     }
 
     @Override
-    public D readOperationalData(P path) {
+    public D readOperationalData(final P path) {
         return reader.readOperationalData(path);
     }
 
     @Override
-    public D readConfigurationData(P path) {
+    public D readConfigurationData(final P path) {
         return reader.readConfigurationData(path);
     }
 
-    private D getConfigurationOriginal(P path) {
+    private D getConfigurationOriginal(final P path) {
         D data = configurationOriginal.get(path);
         if (data != null) {
             return data;
         }
         data = reader.readConfigurationData(path);
         if (data != null) {
-            configurationOriginal.putIfAbsent(path, data);
+            configurationOriginal.put(path, data);
             return data;
         }
         return null;
     }
 
-    private D getOperationalOriginal(P path) {
+    private D getOperationalOriginal(final P path) {
         D data = operationalOriginal.get(path);
         if (data != null) {
             return data;
         }
         data = reader.readOperationalData(path);
         if (data != null) {
-            operationalOriginal.putIfAbsent(path, data);
+            operationalOriginal.put(path, data);
             return data;
         }
         return null;
     }
-    
-    protected D mergeOperationalData(P path,D stored, D modified) {
+
+    protected D mergeOperationalData(final P path,final D stored, final D modified) {
         return modified;
     }
-    
-    protected D mergeConfigurationData(P path,D stored, D modified) {
+
+    protected D mergeConfigurationData(final P path,final D stored, final D modified) {
         return modified;
     }
 }