On 17/01/2008, Oleg Kalnichevski <[EMAIL PROTECTED]> wrote: > > On Thu, 2008-01-17 at 21:22 +0000, sebb wrote: > > There are quite a few missing @Override markers in the nio source > > files, e.g. for toString(), clone(), etc. > > > > Would it be OK if I fix those that are clearly intended to be overrides? > > > > Likewise, now that generics are being used, there are some casts that > > can be removed. Shall I do those as well? > > > > S/// > > > > Sure. Go for it. >
OK, done. There are also quite a few warnings about variable hiding, e.g. where a parameter has the same name as a field in the class or super-class. This can lead to subtle bugs. Is there a policy on this? Or maybe a naming convention for parameters and local variables? > Oleg > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
