Hi all, Related to https://issues.jboss.org/browse/ISPN-1661, the fix is clear, but how come didn't this blow before? The following code works fine: cache.put("hello", null); I was always under the impression that null values (as well as null keys) were not accepted, which I think is good thing btw.

I'm pretty sure this check used to be in place before, but did we somehow lost it? Anyway, I"m adding some tests in org.infinispan.api package. Cheers, -- Galder Zamarreño Sr. Software Engineer Infinispan, JBoss Cache _______________________________________________ infinispan-dev mailing list infinispan-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev

See my comment on the JIRA - my bad here. Galder, thanks for spotting and fixing. On 4 Jan 2012, at 09:54, Mircea Markus wrote: > > On 3 Jan 2012, at 20:51, Galder Zamarreño wrote: >> Hi all, >> >> Related to https://issues.jboss.org/browse/ISPN-1661, the fix is clear, but how come didn't this blow before? >> >> The following code works fine: >> >> cache.put("hello", null); >> >> I was always under the impression that null values (as well as null keys) were not accepted, which I think is good thing btw. > +1 > This is also enforced by the contract of Map.put >> >> I'm pretty sure this check used to be in place before, but did we somehow lost it? >> >> Anyway, I"m adding some tests in org.infinispan.api package. >> >> Cheers, >> -- >> Galder Zamarreño >> Sr. Software Engineer >> Infinispan, JBoss Cache >> >> >> _______________________________________________ >> infinispan-dev mailing list >> infinispan-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/infinispan-dev > > > _______________________________________________ > infinispan-dev mailing list > infinispan-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani manik(a)jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org _______________________________________________ infinispan-dev mailing list infinispan-dev(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev