[
https://issues.jboss.org/browse/CDI-514?page=com.atlassian.jira.plugin.sy...
]
Jozef Hartinger commented on CDI-514:
-------------------------------------
Right, the spec is very clear about this and the TCK follows that.
As for the spec being "broken" it seems that the spec just enforces that the SPI
usage is consistent with what the language allows regarding annotations (at most one
annotation of a type on a given location).
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)