[cdi-dev] [JBoss JIRA] (CDI-584) Clarification needed on Set mutability returned by SPI

Martin Kouba (JIRA) issues at jboss.org
Fri Feb 19 09:46:00 EST 2016


    [ https://issues.jboss.org/browse/CDI-584?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13165806#comment-13165806 ] 

Martin Kouba edited comment on CDI-584 at 2/19/16 9:45 AM:
-----------------------------------------------------------

I see, that's a good example. So something like {{ProcessBean#setBeanInjectionPoints()}} might be useful. By the way, the example wouldn't work because {{ips}} is not a reference to the original set of injection points (it only contains all the elements from the original set ;-).


was (Author: mkouba):
I see, that's a good example. So something like {{ProcessBean#setBeanInjectionPoints()}} might be useful.

> Clarification needed on Set mutability returned by SPI
> ------------------------------------------------------
>
>                 Key: CDI-584
>                 URL: https://issues.jboss.org/browse/CDI-584
>             Project: CDI Specification Issues
>          Issue Type: Clarification
>          Components: Portable Extensions
>            Reporter: Antoine Sabot-Durand
>
> {{ProcessBean}} lifecycle event could be the occasion to add new Injection point to an existing bean, but when you call {{ProcessBean#getBean()#getInjectionPoints()}} the returned set is mutable in OWB and immutable in Weld.
> As the spec doesn't precise if the Set should be mutable or not, each impl made its own choice.
> The result is a lack of consistency between both impl and an interesting feature harder to do with Weld (work with AnnotatedType or create a custom bean to add injection point)
> More generally we should clarify mutability of each Set returned in the SPI. And check if it wouldn't be better to add method to life cycle event to provide a "temporary" mutable set (i.e. {{ProcessBean#getInjectionPoints()}}



--
This message was sent by Atlassian JIRA
(v6.4.11#64026)


More information about the cdi-dev mailing list