Committed by
Ray Milkey
Remove TODO comment
Make the constructor of BatchWrite.Operation private Change-Id: I496ba4502f0f4c147250909cab9123fc5c869f9e
Showing
1 changed file
with
2 additions
and
3 deletions
... | @@ -92,7 +92,7 @@ public final class BatchWrite { | ... | @@ -92,7 +92,7 @@ public final class BatchWrite { |
92 | .toString(); | 92 | .toString(); |
93 | } | 93 | } |
94 | 94 | ||
95 | - public static class Operation { | 95 | + public static final class Operation { |
96 | final OpType type; | 96 | final OpType type; |
97 | final ImmutableList<Object> args; | 97 | final ImmutableList<Object> args; |
98 | 98 | ||
... | @@ -100,8 +100,7 @@ public final class BatchWrite { | ... | @@ -100,8 +100,7 @@ public final class BatchWrite { |
100 | return new Operation(type, args); | 100 | return new Operation(type, args); |
101 | } | 101 | } |
102 | 102 | ||
103 | - // TODO: consider make it private | 103 | + private Operation(OpType type, List<Object> args) { |
104 | - public Operation(OpType type, List<Object> args) { | ||
105 | this.type = checkNotNull(type); | 104 | this.type = checkNotNull(type); |
106 | this.args = ImmutableList.copyOf(args); | 105 | this.args = ImmutableList.copyOf(args); |
107 | } | 106 | } | ... | ... |
-
Please register or login to post a comment