Hi Philipp,

A few notes:

1. You can do PR on github if it helps
2. I'd cache the resolution of supports (if true) to reuse it to get the
instance, speeds up things in a suite
3. We need a @CdiInjection marker (whatever name) otherwise we can break
injections for things which are beans but also injected with another
extension quite easily (mode=all makes it common). If no marker we can use
your impl but the marker should enable to skip it so either something
like @Cdiinjection(true|false) or @SkipCdi maybe?

Hope it makes sense.

Romain Manni-Bucau
@rmannibucau <https://twitter.com/rmannibucau> |  Blog
<https://rmannibucau.metawerx.net/> | Old Blog
<http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> |
LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book
<https://www.packtpub.com/application-development/java-ee-8-high-performance>


Le sam. 13 juin 2020 à 15:34, Philipp Kunz <philipp.k...@paratix.ch> a
écrit :

> Hi,
>
> Wouldn't it be nice, if CDI would also resolve method parameters?
> See patch.
>
> Regards,
> Philipp
>

Reply via email to