[weld-dev] Question about @Decorates

[Pete Muir]

On 24 Oct 2009, at 08:39, David Allen wrote:

> On Fri, 2009-10-23 at 16:57 -0400, Gavin King wrote:
>> Does Weld currently require the use of @Inject on delegate  
>> injection points?
>
> No, it does not.
>
>>
>> I've just noticed that with the change to the 330 annotations, it
>> perhaps makes more sense to change the name of this annotation and
>> require that @Inject be used, like this:
>>
>>
>>   @Decorator
>>   class TimestampLogger implements Logger {
>>      private Logger logger;
>>
>>      @Inject
>>      public TimestampLogger(@Delegate @Debug Logger logger) {
>>         this.logger=logger;
>>       }
>>       ...
>>   }
>>
>>   @Decorator
>>   class TimestampLogger implements Logger {
>>      @Inject @Delegate @Any Logger logger;
>>      ...
>>   }
>>
>> We could perhaps even say that @Delegate is a qualifier.
>
> This appears to be more consistent and thus easier to understand IMO.

Yup, I agree with this too.

I'll file a JIRA issue.