[jboss-jira] [JBoss JIRA] (LOGTOOL-46) jboss-logging-generator singletons don't play nice with Eclipse APT

David Lloyd (JIRA) jira-events at lists.jboss.org
Thu Sep 6 16:47:35 EDT 2012


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

David Lloyd resolved LOGTOOL-46.
--------------------------------

    Fix Version/s: 1.1.0.Beta1
       Resolution: Done

    
>  jboss-logging-generator singletons don't play nice with Eclipse APT
> --------------------------------------------------------------------
>
>                 Key: LOGTOOL-46
>                 URL: https://issues.jboss.org/browse/LOGTOOL-46
>             Project: Log Tool
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>    Affects Versions: 1.0.0.CR5
>         Environment: Eclipse 3.7
>            Reporter: Fred Bricon
>            Assignee: James Perkins
>             Fix For: 1.1.0.Beta1, 1.0.2.Final
>
>         Attachments: jbosslogging-ap-errors.png, test-logging.zip
>
>
> While testing [m2e-apt|https://github.com/jbosstools/m2e-apt] on [hibernate-search|https://github.com/hibernate/hibernate-search], I got these strange errors :
> !jbosslogging-ap-errors.png!
> All the log methods are marked as being duplicate. My gut feeling is this is due to [MessageIdValidator being a singleton|https://github.com/jboss-logging/jboss-logging-tools/blob/4b437dd4c7626f4c5da9fb99923fd2cac4e87a78/src/main/java/org/jboss/logging/processor/validation/MessageIdValidator.java#L44] : all AP work in eclipse reuses the same instance between each incremental builds. So validating the same file more than once lead to these errors.
> The generator being a build tool, it doesn't make much sense to use singletons at all, performance-wise.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


More information about the jboss-jira mailing list