Support for generic aspects about aspect-injector

[timeshift92]

hi .net 8 added support for generics in annotations, is this a possible solution?

[pamidur]

hi!
It actually is. although idk how important this feature will be?
Do you have an use case where generic aspects are needed ?

[timeshift92]

at the moment I'm looking for an opportunity to remove boxing and unboxing, so I thought maybe just the generics in the annotations will help with this, but I'm weak, so I can only guess

[pamidur]

you may be onto something, could you please me an example where boxing happens and how generics help?

[timeshift92]

 [Mixin(typeof(INotifyPropertyChanged))]
    [Aspect(Scope.Global)]
    public class NotifyAspect : INotifyPropertyChanged
    {
        public event PropertyChangedEventHandler PropertyChanged = (s, e) => { };

        [Advice(Kind.After, Targets = Target.Public | Target.Setter)]
        public void AfterSetter(
            [Argument(Source.Instance)] object source,
            [Argument(Source.Name)] string propName,
            [Argument(Source.Triggers)] Attribute[] triggers
            )
        {
            if (triggers.OfType<NotifyAttribute>().Any())
                PropertyChanged(source, new PropertyChangedEventArgs(propName));

            foreach (var attr in triggers.OfType<NotifyAlsoAttribute>())
                foreach (var additional in attr.NotifyAlso ?? new string[] { })
                    PropertyChanged(source, new PropertyChangedEventArgs(additional));
        }
    }

let's take your code, object is something universal, but in fact we can’t do anything with this object by type, we don’t bring it to what we need, both in notify and in interceptors, I think it would be convenient to do something like

  object[] Arguments to ArgumentList Arguments

for example, this object is to be replaced by a certain list of typed arguments, we would not spend on unboxin