commit cd020c30a870477480261fbb1463afae4cddd42e
author Glenn Kasten <gkasten@google.com> Tue Jun 07 08:10:16 2011 -0700
committer Glenn Kasten <gkasten@google.com> Fri Jun 10 09:02:41 2011 -0700
tree 97c809c5c9643d5fa064af48fdbbe5f77301c983
parent 132afe44b3b7963e47d4c32948b2667ddc00062c

Bug 4588827 synchronized keyword add/rm compatible

This only addresses the API check part, html output still has "synchronized".

Change-Id: I4df22215480776462b6113ad802d2b46b696a41c

src/com/google/doclava/MethodInfo.java

diff --git a/src/com/google/doclava/MethodInfo.java b/src/com/google/doclava/MethodInfo.java
index c0ad628..b2d0839 100644
--- a/src/com/google/doclava/MethodInfo.java
+++ b/src/com/google/doclava/MethodInfo.java
@@ -777,12 +777,16 @@
       consistent = false;
     }
 
+    // see JLS 3 13.4.20 "Adding or deleting a synchronized modifier of a method does not break "
+    // "compatibility with existing binaries."
+    /*
     if (mIsSynchronized != mInfo.mIsSynchronized) {
       Errors.error(Errors.CHANGED_SYNCHRONIZED, mInfo.position(), "Method " + mInfo.qualifiedName()
           + " has changed 'synchronized' qualifier from " + mIsSynchronized + " to "
           + mInfo.mIsSynchronized);
       consistent = false;
     }
+    */
 
     for (ClassInfo exception : thrownExceptions()) {
       if (!mInfo.throwsException(exception)) {