import com.google.common.base.Preconditions;
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;
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
}
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) {
@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() {
}
@Override
- public boolean equals(final Object o) {
- return this == o || (o instanceof MemberName && name.equals(((MemberName)o).name));
+ public boolean equals(final Object obj) {
+ return this == obj || obj instanceof MemberName && name.equals(((MemberName)obj).name);
}
@Override
- public int compareTo(final MemberName o) {
- return this == o ? 0 : name.compareTo(o.name);
+ public int compareTo(final MemberName obj) {
+ return this == obj ? 0 : name.compareTo(obj.name);
}
@Override