R4481 Checkstyle Parameter Rename Issue

[Marvin Addison]

On Tue, Sep 13, 2016 at 9:19 AM Rod Widdowson <rdw at steadingsoftware.com>
wrote:

> At the risk of further opening a rathole which I _really_ don't want to go
> into this feels like a case when an annotation _might_ be useful; we can
> annotate what is being exposed _now_ quite easily.  I guess it wouldn't be
> too hard to go back over the release since 3.0.0 if we felt it was worth it.
>

That's preference on the basis that constructor arguments are useful in
many cases and the name-based syntax is helpful for documentation and
support. It's more work, of course. We would need a custom annotation
applied to constructors and an implementation
of org.springframework.core.ParameterNameDiscoverer that reads it. The
tooling to apply the annotation based on existing method signatures ought
to be straightforward. So while it's more work, it doesn't seem
unreasonable.

M
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://shibboleth.net/pipermail/dev/attachments/20160913/0676ff6f/attachment.html>