isAnnotationPresent on some android devices throws NoClassDefFoundError (fix attached)

[amiton]

isAnnotationPresent on some android devices throws NoClassDefFoundError if some unrelated annotation cannot be found so my code change suggests to allow users to override the behaviour by separating to another function (also makes code cleaner).
Please accept my code change to the master branch (diff below).

diff --git a/src/com/esotericsoftware/kryo/Kryo.java b/src/com/esotericsoftware/kryo/Kryo.java
index b04aa07..9ff2e87 100644
--- a/src/com/esotericsoftware/kryo/Kryo.java
+++ b/src/com/esotericsoftware/kryo/Kryo.java
@@ -328,10 +328,8 @@ public class Kryo {
        public Serializer getDefaultSerializer (Class type) {
                if (type == null) throw new IllegalArgumentException("type cannot be null.");

-               if (type.isAnnotationPresent(DefaultSerializer.class)) {
-                       DefaultSerializer defaultSerializerAnnotation = (DefaultSerializer)type.getAnnotation(DefaultSerializer.class);
-                       return ReflectionSerializerFactory.makeSerializer(this, defaultSerializerAnnotation.value(), type);
-               }
+               final Serializer serializerForAnnotation = getDefaultSerializerForAnnotatedType(type);
+               if (serializerForAnnotation!=null) return serializerForAnnotation;

                for (int i = 0, n = defaultSerializers.size(); i < n; i++) {
                        DefaultSerializerEntry entry = defaultSerializers.get(i);
@@ -344,6 +342,16 @@ public class Kryo {
                return newDefaultSerializer(type);
        }

+       protected Serializer getDefaultSerializerForAnnotatedType(Class type)
+       {
+               if (type.isAnnotationPresent(DefaultSerializer.class)) {
+                       DefaultSerializer defaultSerializerAnnotation = (DefaultSerializer)type.getAnnotation(DefaultSerializer.class);
+                       return ReflectionSerializerFactory.makeSerializer(this, defaultSerializerAnnotation.value(), type);
+               }
+
+               return null;    
+       }

[amiton]

Any updates on this?
What should i do to get this diff approved/committed?

Thanks,

[romix]

In principle, I don't have anything against it. Could you submit it as a PR (pull request), so that it is easier for us to merge?

[amiton]

Done. Please review my pull request and approve.
Thanks for the quick reply and great work.

[romix]

Merged. Thanks!