Merge pull request msgpack#75 from august782/master

oza · oza · commit 9fce02943cf1 · 2014-04-27T08:01:15.000+09:00
Fixed to check for interface modifier before checking for abstract modifier.

An interface also has the abstract modifier. We should check for interface modifier before checking for abstract modifier no to confuse users by error messages.
diff --git a/src/main/java/org/msgpack/template/builder/AbstractTemplateBuilder.java b/src/main/java/org/msgpack/template/builder/AbstractTemplateBuilder.java
@@ -72,14 +72,14 @@ public <T> Template<T> buildTemplate(final Class<T> targetClass, final FieldList
     protected abstract <T> Template<T> buildTemplate(Class<T> targetClass, FieldEntry[] entries);
 
     protected void checkClassValidation(final Class<?> targetClass) {
-        if (Modifier.isAbstract(targetClass.getModifiers())) {
-            throw new TemplateBuildException(
-                    "Cannot build template for abstract class: " + targetClass.getName());
-        }
         if (targetClass.isInterface()) {
             throw new TemplateBuildException(
                     "Cannot build template for interface: " + targetClass.getName());
         }
+        if (Modifier.isAbstract(targetClass.getModifiers())) {
+            throw new TemplateBuildException(
+                    "Cannot build template for abstract class: " + targetClass.getName());
+        }
         if (targetClass.isArray()) {
             throw new TemplateBuildException(
                     "Cannot build template for array class: " + targetClass.getName());
