[infinispan-dev] Static analysis (partial)

Tristan Tarrant ttarrant at redhat.com
Fri Oct 30 06:18:04 EDT 2015


Yes, I did think of that, and I like Optional a lot, but sometimes it is 
overkill.

Tristan

On 30/10/2015 10:53, Sebastian Laskawiec wrote:
> +1 from me.
>
> But since we are JDK8 based, how about using Optional for "optional"
> parameters and "nothing" for required ones?
>
> On Fri, Oct 30, 2015 at 10:08 AM, Tristan Tarrant <ttarrant at redhat.com
> <mailto:ttarrant at redhat.com>> wrote:
>
>     During the review of [1] both Pedro and Will asked about whether a field
>     could be null or not and I really wished I could use some kind of
>     @NotNull annotation to clarify that. We already have findbugs included
>     in our maven reports so what do you all think if we adopt those
>     annotations just like we already do for the jcip ones ?
>
>     Tristan
>
>
>     [1] https://github.com/infinispan/infinispan/pull/3726
>     --
>     Tristan Tarrant
>     Infinispan Lead
>     JBoss, a division of Red Hat
>     _______________________________________________
>     infinispan-dev mailing list
>     infinispan-dev at lists.jboss.org <mailto:infinispan-dev at lists.jboss.org>
>     https://lists.jboss.org/mailman/listinfo/infinispan-dev
>
>
>
>
> _______________________________________________
> infinispan-dev mailing list
> infinispan-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/infinispan-dev
>

-- 
Tristan Tarrant
Infinispan Lead
JBoss, a division of Red Hat


More information about the infinispan-dev mailing list