diff options
author | Paul Phillips <paulp@improving.org> | 2011-12-07 10:13:43 -0800 |
---|---|---|
committer | Paul Phillips <paulp@improving.org> | 2011-12-07 10:56:07 -0800 |
commit | 33f3c60ce1931b450053ba4635f7227727aed668 (patch) | |
tree | 401dbf5b7163d70ebc6c39d4c1f22d751851ca56 /lib | |
parent | 332fec96e31840878bed41dd7b5314b97d8da7c2 (diff) | |
download | scala-33f3c60ce1931b450053ba4635f7227727aed668.tar.gz scala-33f3c60ce1931b450053ba4635f7227727aed668.tar.bz2 scala-33f3c60ce1931b450053ba4635f7227727aed668.zip |
Changed shadowed constructor parameter names.
One of my more recent warnings tells us this a bunch of places
if we compile with -Xlint, one example:
scala/actors/ReplyReactorTask.scala:26: warning:
private[this] value reactor in class ReplyReactorTask shadows
mutable reactor inherited from class ReactorTask. Changes to reactor
will not be visible within class ReplyReactorTask - you may want to
give them distinct names.
In some cases (like that one) I was not confident whether the shadowing
was intentional, in which case I left the semantics but changed the name
anyway because it will be eternally confusing otherwise.
Review by @phaller.
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions