Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Feb 26, 2024

We previously [1] changed wording in the ambiguity resolution specification from

Return an Iterator, that iterates over the set of contextual references
for the resulting beans and required type, as defined in <<contextual_reference>>.

to

Return an Iterator, that iterates over the set of contextual references
for the resulting beans, as defined in <<contextual_reference>>.

This was an attempt to simplify the text, but is in fact incorrect, because a contextual reference is always determined by the bean and the bean type.

This commit reverts to the previous wording.

[1] 74827d2

We previously [1] changed wording in the ambiguity resolution specification from

> Return an `Iterator`, that iterates over the set of contextual references
> for the resulting beans and required type, as defined in <<contextual_reference>>.

to

> Return an `Iterator`, that iterates over the set of contextual references
> for the resulting beans, as defined in <<contextual_reference>>.

This was an attempt to simplify the text, but is in fact incorrect, because
a contextual reference is always determined by the bean and the bean type.

This commit reverts to the previous wording.

[1] jakartaee@74827d2
@Ladicek Ladicek added this to the CDI 4.1 milestone Feb 26, 2024
@Ladicek Ladicek requested a review from manovotn February 26, 2024 13:02
Copy link
Contributor

@manovotn manovotn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, I do recall we had that discussion about contextual references but I didn't recall we did this change earlier.

@manovotn manovotn merged commit d9f95b4 into jakartaee:main Feb 26, 2024
@Ladicek Ladicek deleted the fix-ambiguity-resolution branch February 27, 2024 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants