/*
 * Copyright 2015 Google Inc.
 *
 * 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 com.google.j2cl.ast;

import com.google.auto.value.AutoValue;
import com.google.j2cl.common.Interner;
import javax.annotation.Nullable;

/**
 * Encapsulates JsInterop information.
 */
@AutoValue
public abstract class JsInfo {
  public abstract JsMemberType getJsMemberType();

  /**
   * The name specified directly on a type, method or field.
   */
  @Nullable
  public abstract String getJsName();

  /**
   * The namespace specified on a package, type, method or field.
   */
  @Nullable
  public abstract String getJsNamespace();

  public abstract boolean isJsOverlay();

  /**
   * Not a JS member.
   */
  public static final JsInfo NONE = create(JsMemberType.NONE, null, null, false);
  public static final JsInfo RAW = create(JsMemberType.METHOD, null, null, false);
  public static final JsInfo RAW_CTOR = create(JsMemberType.CONSTRUCTOR, null, null, false);
  public static final JsInfo RAW_FIELD = create(JsMemberType.PROPERTY, null, null, false);

  private static Interner<JsInfo> interner;

  /**
   * Creates an instance of an AutoValue generated JsInfo which uses Interners to share identical
   * instances of JsInfos.
   */
  public static JsInfo create(
      JsMemberType jsMemberType, String jsName, String jsNamespace, boolean isJsOverlay) {
    return getInterner()
        .intern(new AutoValue_JsInfo(jsMemberType, jsName, jsNamespace, isJsOverlay));
  }

  private static Interner<JsInfo> getInterner() {
    if (interner == null) {
      interner = new Interner<>();
    }
    return interner;
  }
}

@@ -1553,16 +1553,169 @@
 alse);%0A%0A
+  // There is no need for this interner to be ThreadLocal as it interns JsInfo instances which%0A  // are actually value types (constructed by AutoValue).%0A
   privat

Add a comment why JsInfo.interner does not need to be ThreadLocal