Actually good noise ;) Could you please open a JSON-P 1.1 ticket? They must not change the behaviour from one spec release to the next! This is forbidden by the JCP rules for specs.
LieGrue, strub > Am 24.11.2016 um 11:08 schrieb Reinhard Sandtner > <[email protected]>: > > ok guys, > > i checked the javadoc from the JCP and we are perfectly fine with our > implementation ;) > > the unoffical RI does not respect it > > jsonp-1.0 - > http://download.oracle.com/otndocs/jcp/json-1_0-fr-eval-spec/index.html > <http://download.oracle.com/otndocs/jcp/json-1_0-fr-eval-spec/index.html> > jsonp-1.1 - > http://download.oracle.com/otndocs/jcp/json_p-1_1-edr-spec/index.html > <http://download.oracle.com/otndocs/jcp/json_p-1_1-edr-spec/index.html> > > sorry for the noise :) > > lg > reini > >> Am 24.11.2016 um 10:50 schrieb Romain Manni-Bucau <[email protected]>: >> >> Hmm, >> >> can we check before it is not a regression? I thought it was the first impl >> and I doubt we throw an exception there for fun (I hate to check the key >> exists before getting it to be honest). If not in jsonp 1.0 then +1 to fix >> it in both. >> >> >> Romain Manni-Bucau >> @rmannibucau <https://twitter.com/rmannibucau> | Blog >> <https://blog-rmannibucau.rhcloud.com> | Old Blog >> <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> >> | >> LinkedIn <https://www.linkedin.com/in/rmannibucau> | JavaEE Factory >> <https://javaeefactory-rmannibucau.rhcloud.com> >> >> 2016-11-24 10:47 GMT+01:00 Reinhard Sandtner <[email protected]>: >> >>> hey folks, >>> >>> while writing tests for JsonPatch in our JSONP-1.1 branch, it becomes >>> apparent that our behavior for the get*-methods in JsonObject differs with >>> the implementation in the unoffical RI. >>> >>> if a key does not exist, we throw a NPE while the spec says it should >>> return null (https://github.com/json-processing-inofficial/jsonp/ >>> blob/master/api/src/main/java/javax/json/JsonObject.java#L149 < >>> https://github.com/json-processing-inofficial/jsonp/ >>> blob/master/api/src/main/java/javax/json/JsonObject.java#L149>) >>> >>> >>> * @return the object value to which the specified name is mapped, or >>> * {@code null} if this object contains no mapping for the name >>> >>> >>> imo we should fix it - know the big question: >>> fix it only in jsonp-1.1 or also in jsonp-1.0? >>> >>> lg >>> reini >
