[
https://issues.jboss.org/browse/CDI-514?page=com.atlassian.jira.plugin.sy...
]
Jozef Hartinger commented on CDI-514:
-------------------------------------
If an invariant of a method is well-defined then the method should validate it.
As for why it is defined this way then I think it enforces consistency between what you
can achieve using annotation on classes and what you can achieve programatically.
Let's revisit this constraint for CDI 2.0
FireEventTest#testDuplicateBindingsToFireEventFails() is wrong
--------------------------------------------------------------
Key: CDI-514
URL:
https://issues.jboss.org/browse/CDI-514
Project: CDI Specification Issues
Issue Type: Clarification
Affects Versions: 1.2.Final
Reporter: Mark Struberg
testDuplicateBindingsToFireEventFails() tests for 2 Lifted literals with different
values. But this is perfectly fine as value is NOT annotated as @Nonbinding. Thus the 2
literals are NOT equals according to CDI rules. They are essentially 2 different
annotations...
Plz remove this test. It also makes no sense to add all those very performance costly
tests at runtime. The worst case which can happen is that the 2 annotations make no sense.
But they don't break anything. Wheras checking all the nasty conditions each and every
time is really mad from a performance aspect.
--
This message was sent by Atlassian JIRA
(v6.3.11#6341)