Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unnecessary warning #21901

Closed
wants to merge 1 commit into from

Conversation

stuartwdouglas
Copy link
Member

This warning can be triggered by legitimate code, but won't identify
real problems as a private type would be in the same index as the
current class.

This warning can be triggered by legitimate code, but won't identify
real problems as a private type would be in the same index as the
current class.
@mkouba
Copy link
Contributor

mkouba commented Dec 3, 2021

This warning can be triggered by legitimate code,

What legitimate code do you have in mind?

@stuartwdouglas
Copy link
Member Author

Won't it get triggered if you have a class as a parameter that is outside an application archive?

@mkouba
Copy link
Contributor

mkouba commented Dec 3, 2021

Won't it get triggered if you have a class as a parameter that is outside an application archive?

Only if it's not possible to index the class using the Thread.currentThread().getContextClassLoader(); see https://github.com/quarkusio/quarkus/blob/main/independent-projects/arc/processor/src/main/java/io/quarkus/arc/processor/BeanArchives.java#L102-L109.

@gsmet gsmet changed the title Remove unessesary warning Remove unnecessary warning Dec 3, 2021
@quarkus-bot quarkus-bot bot added the triage/invalid This doesn't seem right label Dec 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/arc Issue related to ARC (dependency injection) triage/invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants