On Fri, Jan 18, 2008 at 01:07:56AM +0000, sebb wrote: > 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? >
I have a (bad) habit of always qualifying instance variables with "this.xxx". It helps me distinguish them from parameters, but I understand not everyone may like such notation. Oleg > > 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] > > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
