Qute: fix regression in perf optimization of ReflectionValueResolver

[mkouba]

• fixes Quarkus Qute broken in 3.16 #44797

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 06f4006.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

[geoand]

@mkouba I think the title of this is incorrect, no?

[mkouba]

@mkouba I think the title of this is incorrect, no?

Well, it's a fix of a regression caused by a performance optimization of ReflectionValueResolver 🤷.

[geoand]

Yeah I know, but someone at looking at the PR (through the release notes for example) would be misled into believing this is a performance fix, when it's in fact a bug fix

…

On Fri, Nov 29, 2024, 10:51 Martin Kouba ***@***.***> wrote: @mkouba <https://github.com/mkouba> I think the title of this is incorrect, no? Well, it's a fix of a regression caused by a performance optimization of ReflectionValueResolver 🤷. — Reply to this email directly, view it on GitHub <#44817 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABBMDP3KMD6OHF4XQB4UUQL2DATKDAVCNFSM6AAAAABSVPNXN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBXGM2DMOJYGE> . You are receiving this because you commented.Message ID: ***@***.***>

[mkouba]

Yeah I know, but someone at looking at the PR (through the release notes for example) would be misled into believing this is a performance fix, when it's in fact a bug fix

Ok, is it better now? ;-)

[geoand]

Thanks!