/*
 * Copyright (c) 2015 Brocade Communications Systems, Inc. and others.  All rights reserved.
 *
 * This program and the accompanying materials are made available under the
 * terms of the Eclipse Public License v1.0 which accompanies this distribution,
 * and is available at http://www.eclipse.org/legal/epl-v10.html
 */
package org.opendaylight.controller.cluster.datastore;

import java.util.EventListener;
import javax.annotation.concurrent.GuardedBy;
import org.opendaylight.yangtools.concepts.ListenerRegistration;

abstract class DelayedListenerRegistration<L extends EventListener, M> implements ListenerRegistration<L> {
    private final M registrationMessage;
    private volatile ListenerRegistration<L> delegate;

    @GuardedBy("this")
    private boolean closed;

    protected DelayedListenerRegistration(M registrationMessage) {
        this.registrationMessage = registrationMessage;
    }

    M getRegistrationMessage() {
        return registrationMessage;
    }

    ListenerRegistration<L> getDelegate() {
        return delegate;
    }

    synchronized <R extends ListenerRegistration<L>> void createDelegate(
            final LeaderLocalDelegateFactory<M, R> factory) {
        if (!closed) {
            this.delegate = factory.createDelegate(registrationMessage);
        }
    }

    @Override
    public L getInstance() {
        final ListenerRegistration<L> d = delegate;
        return d == null ? null : (L)d.getInstance();
    }

    @Override
    public synchronized void close() {
        if (!closed) {
            closed = true;
            if (delegate != null) {
                delegate.close();
            }
        }
    }
}

@@ -1361,103 +1361,631 @@
     
-final ListenerRegistration%3CL%3E d = delegate;%0A        return d == null ? null : (L)d.getInstance(
+// ObjectRegistration annotates this method as @Nonnull but we could return null if the delegate is not set yet.%0A        // In reality, we do not and should not ever call this method on DelayedListenerRegistration instances anyway%0A        // but, since we have to provide an implementation to satisfy the interface, we throw%0A        // UnsupportedOperationException to honor the API contract of not returning null and to avoid a FindBugs error%0A        // for possibly returning null.%0A        throw new UnsupportedOperationException(%0A                %22getInstance should not be called on this instance since it could be null%22
 );%0A 

Fix FindBugs error in DelayedListenerRegistration#getInstance