<div dir="ltr">It looks like the checkstyle cleanup in r4481 caused some method parameter names to change in MemcachedStorageService. I see that I violated the "don't name parameters same as arguments" rule. In any case this breaks Spring wiring that uses parameter name references in constructor arguments:<div><br></div><div><div>    <bean id="shibboleth.MemcachedStorageService"</div><div>          class="org.opensaml.storage.impl.memcached.MemcachedStorageService"</div><div>          c:timeout="5"></div><div>        <constructor-arg name="client"></div><div>            ...</div><div>        </constructor-arg></div><div>    </bean></div></div><div><br></div><div>It's easy to fix but unfortunately that's the style that I used in the documentation because I find it more readable. (The cost of the additional readability is painfully clear.) That means that this change will probably break any deployers using memcached on the 3.3.0 upgrade. Probably the safest thing to do is change the private field names and revert the parameter names. Thoughts?</div><div><br></div><div>Bigger picture we may want to take a stand on using this style of Spring wiring.</div><div><br></div><div>M</div><div><br></div></div>