/**
 * Copyright (C) FuseSource, Inc.
 * http://fusesource.com
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *    http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package org.fusesource.fabric.api.scr;

import org.fusesource.fabric.api.jcip.ThreadSafe;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;


/**
 * An abstract base class for validatable components.
 *
 * @author Thomas.Diesler@jboss.com
 * @since 13-Sep-2013
 */
@ThreadSafe
public abstract class AbstractComponent implements Validatable {

    private static final transient Logger LOG = LoggerFactory.getLogger(AbstractComponent.class);

    /* This uses volatile to make sure that every thread sees the last written value
     *
     * - The use of AtomicBoolean would be wrong because it does not guarantee that
     *   prior written state is also seen by other threads
     *
     * - Synchronizing all methods in here would also work, but would effectively cause
     *   a lock acquisition on every public method in every component
     */
    private ValidationSupport active = new ValidationSupport();

    public void activateComponent() {
        active.setValid();
        LOG.debug("activateComponent: " + this);
    }

    public void deactivateComponent() {
        LOG.debug("deactivateComponent: " + this);
        active.setInvalid();
    }

    @Override
    public boolean isValid() {
        return active.isValid();
    }

    @Override
    public void assertValid() {
        active.assertValid();
    }
}
@@ -1155,323 +1155,8 @@
 lue%0A
-     *%0A     * - The use of AtomicBoolean would be wrong because it does not guarantee that%0A     *   prior written state is also seen by other threads%0A     *%0A     * - Synchronizing all methods in here would also work, but would effectively cause%0A     *   a lock acquisition on every public method in every component%0A
     

Fix wrong comment in AbstractComponent