The patch looks fine to me. Personally, I prefer a consistent style to have the "Override" tag in a full line, for example:
+ @Override public String run() { Please let me know if you want a sponsor to push the patch. Thanks for the contribution. Xuelei On 11/16/2012 10:32 PM, Florian Weimer wrote: > The attached patch fixes a few warnings in javax.net.ssl.*. All the > overrides appear to be intentional. >