/*
 * Copyright (C) 2007 The Guava Authors
 *
 * 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.common.eventbus;

import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;

/**
 * Wraps a single-argument 'handler' method on a specific object, and ensures
 * that only one thread may enter the method at a time.
 *
 * <p>Beyond synchronization, this class behaves identically to
 * {@link EventHandler}.
 *
 * @author Cliff Biffle
 */
class SynchronizedEventHandler extends EventHandler {
  /**
   * Creates a new SynchronizedEventHandler to wrap {@code method} on
   * {@code target}.
   *
   * @param target  object to which the method applies.
   * @param method  handler method.
   */
  public SynchronizedEventHandler(Object target, Method method) {
    super(target, method);
  }

  @Override public synchronized void handleEvent(Object event)
      throws InvocationTargetException {
    super.handleEvent(event);
  }

}

@@ -979,16 +979,22 @@
 fle%0A */%0A
+final 
 class Sy
@@ -1348,24 +1348,26 @@
 Override
+%0A 
  public 
 synchron
@@ -1362,21 +1362,8 @@
 lic 
-synchronized 
 void
@@ -1392,14 +1392,8 @@
 ent)
-%0A     
  thr
@@ -1424,16 +1424,115 @@
 ption %7B%0A
+    // https://code.google.com/p/guava-libraries/issues/detail?id=1403%0A    synchronized (this) %7B%0A  
     supe
@@ -1555,13 +1555,18 @@
 ent);%0A  
+  %7D%0A  
 %7D
-%0A
 %0A%7D%0A

Change SynchronizedEventHandler to use a synchronized block rather than just synchronizing the method. Since the method body does nothing but call the super method, it (and its synchronization) was apparently being optimized away under a certain JVM: https://code.google.com/p/guava-libraries/issues/detail?id=1403 ------------- Created by MOE: http://code.google.com/p/moe-java MOE_MIGRATED_REVID=46627764