[cdi-dev] Ordering of AfterTypeDiscovery#getAlternatives() et al

[Jozef Hartinger]

Ordering in the list defines the ordering of the invocation. The 
ordering of invocation for interceptors and decorators is explicitly 
defined as ascending. That implies that the ordering of 
interceptors/decorator list should be ascending as well.

Analogously, the list of alternatives is expected to be ascending.

On 12/22/2014 01:36 PM, Mark Struberg wrote:
>> Why did you change that for interceptors and decorators?
> Oh that's totally fine. Remember that the ordering in the list defines the ordering of the invocation. An if someone would re-sort that list then he must know if it is ascending or descending.
>
> LieGrue,
> strub
>
>
>
>
>
>> On Monday, 22 December 2014, 10:42, Jozef Hartinger <jharting at redhat.com> wrote:
>> On 12/22/2014 09:24 AM, Romain Manni-Bucau wrote:
>>>   hi
>>>
>>>   2014-12-22 9:23 GMT+01:00 Jozef Hartinger <jharting at redhat.com>:
>>>>   On 12/20/2014 11:26 AM, Mark Struberg wrote:
>>>>>   Hi!
>>>>>
>>>>>   If I have a
>>>>>
>>>>>
>>>>>   @Priority(100)
>>>>>   public class MyAlternative implements Foo ..
>>>>>
>>>>>   and
>>>>>
>>>>>
>>>>>   @Priority(101)
>>>>>   public class ABetterAlternative implements Foo ..
>>>>>
>>>>>
>>>>>   Then ABetterAlternative will finally be chosen.
>>>>>
>>>>>   This ordering can be changed via
>> AfterTypeDiscovery#getAlternatives().
>>>>>   But the spec only says "returns the ordered list of enabled
>> alternatives for the application.."
>>>>>   But it does NOT define in which sorting this list is ordered ;)
>>>>>
>>>>>   In OWB we hat the 'most important' alternative come first.
>> It seems in Weld it is the other way around.
>>>>   What sorting do you use for interceptors and decorators?
>>>   until yesterday the same as for runtime
>> Why did you change that for interceptors and decorators?
>>
>>>>>   I have no problem with changing this in OWB, but I would like to
>> get this clarified in our JavaDocs and spec.
>>>>   Agreed. Go ahead and file an issue.
>>>>>   LieGrue,
>>>>>   strub
>>>>>   _______________________________________________
>>>>>   cdi-dev mailing list
>>>>>   cdi-dev at 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.
>>>>   _______________________________________________
>>>>   cdi-dev mailing list
>>>>   cdi-dev at 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.
>>