X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=binding%2Fmdsal-binding-dom-codec%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fmdsal%2Fbinding%2Fdom%2Fcodec%2Fimpl%2FCodecDataObject.java;h=65e40086155a3d804c3262dbc1d05ef2105a6643;hb=9f23891d96e635e1cd30c699f9b72b9336fc9d06;hp=78d8bc7cb89143819a6184966a245bca6f484e3b;hpb=259957ffe47414cc197f4a819361e8a6dcf50b19;p=mdsal.git diff --git a/binding/mdsal-binding-dom-codec/src/main/java/org/opendaylight/mdsal/binding/dom/codec/impl/CodecDataObject.java b/binding/mdsal-binding-dom-codec/src/main/java/org/opendaylight/mdsal/binding/dom/codec/impl/CodecDataObject.java index 78d8bc7cb8..65e4008615 100644 --- a/binding/mdsal-binding-dom-codec/src/main/java/org/opendaylight/mdsal/binding/dom/codec/impl/CodecDataObject.java +++ b/binding/mdsal-binding-dom-codec/src/main/java/org/opendaylight/mdsal/binding/dom/codec/impl/CodecDataObject.java @@ -9,13 +9,15 @@ package org.opendaylight.mdsal.binding.dom.codec.impl; import static java.util.Objects.requireNonNull; -import com.google.common.base.MoreObjects; -import com.google.common.base.MoreObjects.ToStringHelper; -import java.util.concurrent.atomic.AtomicReferenceFieldUpdater; +import com.google.common.base.VerifyException; +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings; +import java.lang.invoke.MethodHandles; +import java.lang.invoke.VarHandle; import org.eclipse.jdt.annotation.NonNull; import org.eclipse.jdt.annotation.Nullable; import org.opendaylight.yangtools.yang.binding.DataObject; -import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNodeContainer; +import org.opendaylight.yangtools.yang.data.api.schema.DataContainerNode; +import org.opendaylight.yangtools.yang.data.api.schema.MapEntryNode; /** * A base class for {@link DataObject}s backed by {@link DataObjectCodecContext}. While this class is public, it not @@ -25,92 +27,125 @@ import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNodeContainer; * @param DataObject type */ public abstract class CodecDataObject implements DataObject { + // An object representing a null value in a member field. private static final @NonNull Object NULL_VALUE = new Object(); - @SuppressWarnings("rawtypes") - private final @NonNull NormalizedNodeContainer data; - final @NonNull DataObjectCodecContext context; + private static final VarHandle CACHED_HASH_CODE; - private volatile Integer cachedHashcode = null; + static { + try { + CACHED_HASH_CODE = MethodHandles.lookup().findVarHandle(CodecDataObject.class, "cachedHashcode", + Integer.class); + } catch (NoSuchFieldException | IllegalAccessException e) { + throw new ExceptionInInitializerError(e); + } + } + + private final @NonNull AbstractDataObjectCodecContext context; + private final @NonNull DataContainerNode data; - public CodecDataObject(final DataObjectCodecContext ctx, final NormalizedNodeContainer data) { - this.context = requireNonNull(ctx, "Context must not be null"); + // Accessed via a VarHandle + @SuppressWarnings("unused") + // FIXME: consider using a primitive int-based cache (with 0 being uninit) + @SuppressFBWarnings(value = "UUF_UNUSED_FIELD", justification = "https://github.com/spotbugs/spotbugs/issues/2749") + private volatile Integer cachedHashcode; + + protected CodecDataObject(final AbstractDataObjectCodecContext context, final DataContainerNode data) { this.data = requireNonNull(data, "Data must not be null"); + this.context = requireNonNull(context, "Context must not be null"); } @Override public final int hashCode() { - final Integer cached = cachedHashcode; - if (cached != null) { - return cached; - } - - final int result = codecAugmentedHashCode(); - cachedHashcode = result; - return result; + final var cached = (Integer) CACHED_HASH_CODE.getAcquire(this); + return cached != null ? cached : loadHashCode(); } @Override public final boolean equals(final Object obj) { - if (obj == this) { - return true; - } - final Class iface = implementedInterface(); - if (!iface.isInstance(obj)) { - return false; - } - @SuppressWarnings("unchecked") - final T other = (T) iface.cast(obj); - if (other instanceof CodecDataObject) { - return data.equals(((CodecDataObject) obj).data); - } - return codecAugmentedEquals(other); + // Indirection to keep checkstyle happy + return codecEquals(obj); } @Override - public final String toString() { - return codecAugmentedFillToString(MoreObjects.toStringHelper(implementedInterface()).omitNullValues()) - .toString(); + public abstract String toString(); + + protected final Object codecMember(final VarHandle handle, final String localName) { + final Object cached = handle.getAcquire(this); + return cached != null ? unmaskNull(cached) : loadMember(handle, context.getLeafChild(localName)); + } + + protected final Object codecMember(final VarHandle handle, final Class bindingClass) { + final Object cached = handle.getAcquire(this); + return cached != null ? unmaskNull(cached) : loadMember(handle, context.getStreamChild(bindingClass)); } - // TODO: consider switching to VarHandles for Java 9+ - protected final Object codecMember(final AtomicReferenceFieldUpdater, Object> updater, - final int offset) { - final Object cached = updater.get(this); - return cached != null ? unmaskNull(cached) : loadMember(updater, offset); + protected final Object codecMember(final VarHandle handle, final CodecContextSupplier supplier) { + final Object cached = handle.getAcquire(this); + return cached != null ? unmaskNull(cached) : loadMember(handle, supplier.get()); + } + + protected final @NonNull Object codecMemberOrEmpty(final @Nullable Object value, + final @NonNull Class bindingClass) { + return value != null ? value : emptyObject(bindingClass); + } + + private @NonNull Object emptyObject(final @NonNull Class bindingClass) { + final var childContext = context.getStreamChild(bindingClass); + if (childContext instanceof StructuralContainerCodecContext structural) { + return structural.emptyObject(); + } + throw new VerifyException("Unexpected context " + childContext); + } + + protected final @NonNull Object codecKey(final VarHandle handle) { + final Object cached = handle.getAcquire(this); + return cached != null ? cached : loadKey(handle); } protected abstract int codecHashCode(); - protected abstract boolean codecEquals(T other); + protected abstract boolean codecEquals(Object obj); - protected abstract ToStringHelper codecFillToString(ToStringHelper helper); + final @NonNull AbstractDataObjectCodecContext codecContext() { + return context; + } - @SuppressWarnings("rawtypes") - final @NonNull NormalizedNodeContainer codecData() { + final @NonNull DataContainerNode codecData() { return data; } - // Non-final to allow specialization in AugmentableCodecDataObject - int codecAugmentedHashCode() { - return codecHashCode(); - } + // Helper split out of codecMember to aid its inlining + private Object loadMember(final VarHandle handle, final CodecContext childCtx) { + final var child = data.childByArg(childCtx.getDomPathArgument()); - // Non-final to allow specialization in AugmentableCodecDataObject - boolean codecAugmentedEquals(final T other) { - return codecEquals(other); + // We do not want to use Optional.map() here because we do not want to invoke defaultObject() when we have + // normal value because defaultObject() may end up throwing an exception intentionally. + final Object obj = child != null ? childCtx.deserializeObject(child) : childCtx.defaultObject(); + final Object witness = handle.compareAndExchangeRelease(this, null, maskNull(obj)); + return witness == null ? obj : unmaskNull(witness); } - // Non-final to allow specialization in AugmentableCodecDataObject - ToStringHelper codecAugmentedFillToString(final ToStringHelper helper) { - return codecFillToString(helper); + // Helper split out of codecKey to aid its inlining + private @NonNull Object loadKey(final VarHandle handle) { + if (!(data instanceof MapEntryNode mapEntry)) { + throw new VerifyException("Unsupported value " + data); + } + if (!(context instanceof MapCodecContext listContext)) { + throw new VerifyException("Unexpected context " + context); + } + + final Object obj = listContext.deserialize(mapEntry.name()); + // key is known to be non-null, no need to mask it + final Object witness = handle.compareAndExchangeRelease(this, null, obj); + return witness == null ? obj : witness; } - // Helpers split out of codecMember to aid its inlining - private Object loadMember(final AtomicReferenceFieldUpdater, Object> updater, - final int offset) { - final Object decoded = context.getBindingChildValue(data, offset); - return updater.compareAndSet(this, null, maskNull(decoded)) ? decoded : unmaskNull(updater.get(this)); + // Helper split out of hashCode() to aid its inlining + private int loadHashCode() { + final int result = codecHashCode(); + final Object witness = CACHED_HASH_CODE.compareAndExchangeRelease(this, null, result); + return witness == null ? result : (Integer) witness; } private static @NonNull Object maskNull(final @Nullable Object unmasked) {