Code Review
/
controller.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Add the ability to report known connected clients
[controller.git]
/
opendaylight
/
md-sal
/
cds-access-api
/
src
/
main
/
java
/
org
/
opendaylight
/
controller
/
cluster
/
access
/
concepts
/
MemberName.java
diff --git
a/opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/MemberName.java
b/opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/MemberName.java
index 2b1a642029f670809b167dbfce86287d64b2a3db..8a1123da520f9ff74d7a504d6839e0ec0d07be20 100644
(file)
--- a/
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/MemberName.java
+++ b/
opendaylight/md-sal/cds-access-api/src/main/java/org/opendaylight/controller/cluster/access/concepts/MemberName.java
@@
-7,11
+7,14
@@
*/
package org.opendaylight.controller.cluster.access.concepts;
*/
package org.opendaylight.controller.cluster.access.concepts;
+import static com.google.common.base.Preconditions.checkArgument;
+import static com.google.common.base.Verify.verifyNotNull;
+import static java.util.Objects.requireNonNull;
+
import com.google.common.annotations.Beta;
import com.google.common.base.MoreObjects;
import com.google.common.annotations.Beta;
import com.google.common.base.MoreObjects;
-import com.google.common.base.Preconditions;
import com.google.common.base.Strings;
import com.google.common.base.Strings;
-import
com.google.common.base.Verify
;
+import
edu.umd.cs.findbugs.annotations.SuppressFBWarnings
;
import java.io.DataInput;
import java.io.DataOutput;
import java.io.Externalizable;
import java.io.DataInput;
import java.io.DataOutput;
import java.io.Externalizable;
@@
-32,12
+35,15
@@
public final class MemberName implements Comparable<MemberName>, WritableIdentif
private static final long serialVersionUID = 1L;
private byte[] serialized;
private static final long serialVersionUID = 1L;
private byte[] serialized;
+ // checkstyle flags the public modifier as redundant however it is explicitly needed for Java serialization to
+ // be able to create instances via reflection.
+ @SuppressWarnings("checkstyle:RedundantModifier")
public Proxy() {
// For Externalizable
}
Proxy(final byte[] serialized) {
public Proxy() {
// For Externalizable
}
Proxy(final byte[] serialized) {
- this.serialized =
Preconditions.checkNot
Null(serialized);
+ this.serialized =
requireNon
Null(serialized);
}
@Override
}
@Override
@@
-60,19
+66,22
@@
public final class MemberName implements Comparable<MemberName>, WritableIdentif
private static final long serialVersionUID = 1L;
private final String name;
private static final long serialVersionUID = 1L;
private final String name;
+
+ @SuppressFBWarnings(value = "VO_VOLATILE_REFERENCE_TO_ARRAY",
+ justification = "The array elements are non-volatile but we don't access them.")
private volatile byte[] serialized;
private MemberName(final String name) {
private volatile byte[] serialized;
private MemberName(final String name) {
- this.name =
Preconditions.checkNot
Null(name);
+ this.name =
requireNon
Null(name);
}
MemberName(final String name, final byte[] serialized) {
this(name);
}
MemberName(final String name, final byte[] serialized) {
this(name);
- this.serialized =
Verify.
verifyNotNull(serialized);
+ this.serialized = verifyNotNull(serialized);
}
public static MemberName forName(final String name) {
}
public static MemberName forName(final String name) {
-
Preconditions.
checkArgument(!Strings.isNullOrEmpty(name));
+ checkArgument(!Strings.isNullOrEmpty(name));
// TODO: consider caching instances here
return new MemberName(name);
}
// TODO: consider caching instances here
return new MemberName(name);
}
@@
-85,28
+94,34
@@
public final class MemberName implements Comparable<MemberName>, WritableIdentif
@Override
public void writeTo(final DataOutput out) throws IOException {
@Override
public void writeTo(final DataOutput out) throws IOException {
- final byte[]
serialized
= getSerialized();
- out.writeInt(
serialized
.length);
- out.write(
serialized
);
+ final byte[]
local
= getSerialized();
+ out.writeInt(
local
.length);
+ out.write(
local
);
}
public String getName() {
return name;
}
}
public String getName() {
return name;
}
+ public org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.controller.cds.types.rev191024
+ .MemberName toYang() {
+ return new org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.controller.cds.types.rev191024
+ .MemberName(name);
+ }
+
@Override
public int hashCode() {
return name.hashCode();
}
@Override
@Override
public int hashCode() {
return name.hashCode();
}
@Override
- public boolean equals(final Object o) {
- return this == o
|| (o instanceof MemberName && name.equals(((MemberName)o).name)
);
+ public boolean equals(final Object o
bj
) {
+ return this == o
bj || obj instanceof MemberName && name.equals(((MemberName)obj).name
);
}
@Override
}
@Override
- public int compareTo(final MemberName o) {
- return this == o
? 0 : name.compareTo(o
.name);
+ public int compareTo(final MemberName o
bj
) {
+ return this == o
bj ? 0 : name.compareTo(obj
.name);
}
@Override
}
@Override