Merge pull request #42 from mschonaker/master

Fix #41: abstract could become concrete.

1 files changed, 10 insertions, 8 deletions

diff --git a/api/src/main/java/org/osjava/jardiff/Tools.java b/api/src/main/java/org/osjava/jardiff/Tools.java
index 0a8f5f1..e159541 100644
--- a/api/src/main/java/org/osjava/jardiff/Tools.java
+++ b/api/src/main/java/org/osjava/jardiff/Tools.java
@@ -66,12 +66,14 @@ public final class Tools
      * @return
      */
     public static boolean isAccessChange(int oldAccess, int newAccess) {
-        if ((oldAccess & Opcodes.ACC_FINAL) == 0 && (newAccess & Opcodes.ACC_FINAL) > 0) {
-            return true;
-        } else {
-            oldAccess = oldAccess & ~Opcodes.ACC_FINAL;
-            newAccess = newAccess & ~Opcodes.ACC_FINAL;
-        }
-        return oldAccess != newAccess;
-    }
+		if ((oldAccess & Opcodes.ACC_FINAL) == 0 && (newAccess & Opcodes.ACC_FINAL) > 0) {
+			return true;
+		} else if ((oldAccess & Opcodes.ACC_ABSTRACT) == 0 && (newAccess & Opcodes.ACC_ABSTRACT) > 0) {
+			return true;
+		} else {
+			oldAccess = oldAccess & ~Opcodes.ACC_FINAL & ~Opcodes.ACC_ABSTRACT;
+			newAccess = newAccess & ~Opcodes.ACC_FINAL & ~Opcodes.ACC_ABSTRACT;
+		}
+		return oldAccess != newAccess;
+	}
 }