I did not but all the other _parse* methods seemed to check for the null
token so I was just adding it in case. If it's handled at a higher level
then it's not a problem. I'll just remove it.  I've also added the @since
2.4 javadoc.  Pull request submitted.


On Fri, Jan 10, 2014 at 9:39 AM, Tatu Saloranta <notifications@github.com>wrote:

> In
> src/main/java/com/fasterxml/jackson/databind/deser/std/NumberDeserializers.java:
>
> >                  }
> > +                return value;
> > +            } else if (t == JsonToken.VALUE_NULL && !_valueClass.isPrimitive()) {
>
> Null handling should actually occur at higher level, and deserializers
> never called with null token. Did you find a case where this is not true?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/FasterXML/jackson-databind/pull/384/files#r8795548>
> .
>