commit 86f61984e16cb3fd5cf4aa0892d7060096568a3b
author Sho SHIMIZU <sshimizu@us.fujitsu.com> Tue Jan 20 13:32:57 2015 -0800
committer Ray Milkey <ray@onlab.us> Tue Jan 20 23:57:56 2015 +0000
tree 7a64982adc231206a0e790c1bdb9b26426811881
parent 3cbfbc4c50b7df13b60de8e9e9ffea56b6ef0ca0

Remove TODO comment

Make the constructor of BatchWrite.Operation private

Change-Id: I496ba4502f0f4c147250909cab9123fc5c869f9e

core/api/src/main/java/org/onosproject/net/intent/BatchWrite.java

diff --git a/core/api/src/main/java/org/onosproject/net/intent/BatchWrite.java b/core/api/src/main/java/org/onosproject/net/intent/BatchWrite.java
index d528e28..e713243 100644
--- a/core/api/src/main/java/org/onosproject/net/intent/BatchWrite.java
+++ b/core/api/src/main/java/org/onosproject/net/intent/BatchWrite.java
@@ -92,7 +92,7 @@
                 .toString();
     }
 
-    public static class Operation {
+    public static final class Operation {
         final OpType type;
         final ImmutableList<Object> args;
 
@@ -100,8 +100,7 @@
             return new Operation(type, args);
         }
 
-        // TODO: consider make it private
-        public Operation(OpType type, List<Object> args) {
+        private Operation(OpType type, List<Object> args) {
             this.type = checkNotNull(type);
             this.args = ImmutableList.copyOf(args);
         }