R4481 Checkstyle Parameter Rename Issue

[Cantor, Scott]

On 9/14/16, 1:11 PM, "dev on behalf of Brent Putman" <dev-bounces at shibboleth.net on behalf of putmanb at georgetown.edu> wrote:
    
> Aside from that: Looks like debug is also turned on in the v2 parent. So I guess we've
> been doing this as SOP for a long time.  I wasn't aware of this.  Does anyone remember
> why we did that?  I just wonder if we are incurring any bytecode size or performance
> penalties, etc for building with debug=true.

Figured you might know, but I certainly wouldn't.

>    Just turning it off, though, might be risky, obviously. So not saying we should anytime
> soon.  Just wondering about the reasons, and tradeoffs.
    
Is there an annotation we can use without Java 8 to get the names available to Spring wiring without debug? I guess that's the essential question. I'm hearing maybe the answer is no.

I would be in favor of "opt-in" for c:name working, meaning that we get to a place where we can default to it not working, and having to explicitly annotate the constructor to make it work so it becomes a conscious choice to do so.

-- Scott