what is the 2nd issue? The one I mean is that the scanned classes are processed one after the other (as specified). But the sample code relies on ProcessInjectionPoint of ALL classes gets scanned before the first ProcessBeanAttributes get fired.
LieGrue, strub > Am 01.06.2015 um 09:58 schrieb Romain Manni-Bucau <[email protected]>: > > @Mark: for which of 2 issue? > > Agree spec is not 100% clean > Le 1 juin 2015 00:38, "Mark Struberg" <[email protected]> a écrit : > >> Can you plz create a JIRA issue? >> >> LieGrue, >> strub >> >> >>> Am 01.06.2015 um 03:51 schrieb Romain Manni-Bucau <[email protected] >>> : >>> >>> PS: alternative is to fix the lifecycle between ProcessBeanAttributes and >>> ProcessInjectionPoint >>> >>> >>> Romain Manni-Bucau >>> @rmannibucau <https://twitter.com/rmannibucau> | Blog >>> <http://rmannibucau.wordpress.com> | Github < >> https://github.com/rmannibucau> | >>> LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber >>> <http://www.tomitribe.com> >>> >>> 2015-06-01 3:42 GMT+02:00 Romain Manni-Bucau <[email protected]>: >>> >>>> Hi guys, >>>> >>>> >> org.apache.webbeans.component.BeanAttributesImpl#BeanAttributesImpl(java.util.Set<java.lang.reflect.Type>, >>>> java.util.Set<java.lang.annotation.Annotation>, java.lang.Class<? >> extends >>>> java.lang.annotation.Annotation>, java.lang.String, boolean, >>>> java.util.Set<java.lang.Class<? extends >> java.lang.annotation.Annotation>>, >>>> boolean) copies types from bean attributes. It prevents any further >> changes >>>> and https://gist.github.com/antoinesd/3097661ca99fa61900fb becomes not >>>> portable (if set is filled before we copy it it works otherwise it is >>>> broken) >>>> >>>> do you think we should just take the input value without any >> modification? >>>> >>>> Romain Manni-Bucau >>>> @rmannibucau <https://twitter.com/rmannibucau> | Blog >>>> <http://rmannibucau.wordpress.com> | Github >>>> <https://github.com/rmannibucau> | LinkedIn >>>> <https://www.linkedin.com/in/rmannibucau> | Tomitriber >>>> <http://www.tomitribe.com> >>>> >> >>
