Re: [cdi-dev] TCK and spec question
On 12/22/2014 09:33 AM, Mark Struberg wrote:
No you can NOT I fear.
How do you make sure that the AnnotatedType someone added doesn't get changed later?
We have no whatever control over it. If some programmer like to make it mutable, well
then it is that way and we have no chance to detect that :(
Imo the only thing we can do is to point programmers to the fact that this is stupid
-> non-portable behaviour.
Correct. This is stupid. But allowing
setAnnotatedType() to be called
anytime does not help this at all.
The fact that we throw the exception on setAnnotatedType() prevents a
class of bad things from happening. At the same time it does not prevent
the other one that you mentioned. The fact that we cannot prevent all of
them does not mean that we should not try to prevent those we can.
LieGrue,
strub
> On Monday, 22 December 2014, 9:29, Jozef Hartinger <jharting(a)redhat.com>
wrote:
> On 12/21/2014 09:47 PM, Mark Struberg wrote:
>> Hi!
>>
>> I just came across this little sentence in the spec
>>
>>
>>
>> 11.5.6 "If any ProcessAnnotatedType method is called outside of the
> observer method invocation, an IllegalStateException is thrown."
>>
>> I don't believe such a limitation helps much. What about extensions who
> do a setAnnotatedType and change this instance in a later phase?
> Such as?
>> We have no whatever chance to prevent this anyway.
> We can prevent that by throwing the exception, rather than silently
> ignoring that the extension is doing something wrong.
>
>> So why not just say that if a CDI System Event gets modified outside of the
> method it gets injected into then non portable behaviour results.
>> LieGrue,
>> strub
>> _______________________________________________
>> cdi-dev mailing list
>> cdi-dev(a)lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/cdi-dev
>>
>> Note that for all code provided on this list, the provider licenses the
> code under the Apache License, Version 2
> (http://www.apache.org/licenses/LICENSE-2.0.html). For all other ideas provided
> on this list, the provider waives all patent and other intellectual property
> rights inherent in such information.
>