[seam-issues] [JBoss JIRA] Updated: (SOLDER-12) Add isMetadataChanged() accessor to AnnotatedTypeBuilder

Dan Allen (JIRA) jira-events at lists.jboss.org
Fri Dec 17 23:41:17 EST 2010


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

Dan Allen updated SOLDER-12:
----------------------------

             Assignee: Dan Allen
        Fix Version/s: 3.0.0.Beta2
    Affects Version/s: 3.0.0.Beta1
          Component/s: Builders


> Add isMetadataChanged() accessor to AnnotatedTypeBuilder
> --------------------------------------------------------
>
>                 Key: SOLDER-12
>                 URL: https://issues.jboss.org/browse/SOLDER-12
>             Project: Seam Solder
>          Issue Type: Feature Request
>          Components: Builders
>    Affects Versions: 3.0.0.Beta1
>            Reporter: Dan Allen
>            Assignee: Dan Allen
>            Priority: Minor
>             Fix For: 3.0.0.Beta2
>
>
> As an optimization, it makes sense to only create the AnnotatedType and assign it to ProcessAnnotatedType if the metadata has changed in anyway. Otherwise, you are wrapping the type just for the sake of wrapping it.
> Therefore, I propose that any change to the state of the annotated type be tracked by a changed flag that is accessible by calling the AnnotatedTypeBuilder#isMetadataChanged() (or similarly named) method. It would be used as follows:
> AnnotatedTypeBuilder b = AnnotatedTypeBuilder.readFromType(event.getAnnotatedType(), true);
> // do extension logic
> if (b.isMetadataChanged())
> {
>    event.setAnnotatedType(b.create());
> }

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        


More information about the seam-issues mailing list