diff options
Diffstat (limited to 'src/com/google/doclava/MethodInfo.java')
-rw-r--r-- | src/com/google/doclava/MethodInfo.java | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/com/google/doclava/MethodInfo.java b/src/com/google/doclava/MethodInfo.java index 47b1978..33197a3 100644 --- a/src/com/google/doclava/MethodInfo.java +++ b/src/com/google/doclava/MethodInfo.java @@ -893,6 +893,8 @@ public class MethodInfo extends MemberInfo implements AbstractMethodInfo, Resolv + " to " + mInfo.scope()); } + // Changing the deprecated annotation is binary- and source-compatible, but + // we still need to log the API change. if (!isDeprecated() == mInfo.isDeprecated()) { Errors.error(Errors.CHANGED_DEPRECATED, mInfo.position(), "Method " + mInfo.prettyQualifiedSignature() + " has changed deprecation state " + isDeprecated() @@ -900,16 +902,14 @@ public class MethodInfo extends MemberInfo implements AbstractMethodInfo, Resolv consistent = false; } - // see JLS 3 13.4.20 "Adding or deleting a synchronized modifier of a method does not break " - // "compatibility with existing binaries." - /* + // Changing the synchronized modifier is binary- and source-compatible (see + // JLS 3 13.4.20), but we still need to log the API change. 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)) { |