[cdi-dev] [JBoss JIRA] (CDI-192) Unify coding style of the SPI interfaces

Pete Muir (JIRA) jira-events at lists.jboss.org
Wed Aug 15 14:28:14 EDT 2012

     [ https://issues.jboss.org/browse/CDI-192?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Pete Muir resolved CDI-192.

         Assignee: Pete Muir
    Fix Version/s: 1.1.EDR2
                       (was: 1.1 (Proposed))
       Resolution: Done

> Unify coding style of the SPI interfaces
> ----------------------------------------
>                 Key: CDI-192
>                 URL: https://issues.jboss.org/browse/CDI-192
>             Project: CDI Specification Issues
>          Issue Type: Bug
>          Components: Portable Extensions
>    Affects Versions: 1.1.EDR1
>            Reporter: Jozef Hartinger
>            Assignee: Pete Muir
>            Priority: Trivial
>             Fix For: 1.1.EDR2
> The [JLS|http://java.sun.com/docs/books/jls/third_edition/html/interfaces.html#9.4] says:
> {quote}
> For compatibility with older versions of the Java platform, it is permitted but discouraged, as a matter of style, to redundantly specify the abstract modifier for methods declared in interfaces.
> It is permitted, but strongly discouraged as a matter of style, to redundantly specify the public modifier for interface methods.
> {quote}
> The vast majority of CDI API interfaces specifies the public modifier while leaving out the abstract modifier, however, there are exceptions:
> {code:JAVA}
> public interface Context {
> ...
>     boolean isActive();
> }
> {code}
> {code:JAVA}
> public interface BeanAttributes<T> {
>     public abstract Set<Type> getTypes();
>     public abstract Set<Annotation> getQualifiers();
> ...
> {code}

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.jboss.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


More information about the cdi-dev mailing list