On Mon, 15 Nov 2004, Dominique Devienne <[EMAIL PROTECTED]> wrote:

> Stefan, could you explain the main categories of warning your
> changes are fixing?

Sure:

<http://brutus.apache.org/gump/kaffe/ant/bootstrap-ant/gump_work/buildscript_ant_bootstrap-ant.html>
but be fast ;-)

> I see a lot of local and member variables name changes, and I
> suspect some of them relate to name hiding,

almost all of them.  The one in extension/Specification even pointed
to a real bug.

> while some others kind of puzzle me. Why does Jikes care between
> newValue and newV?

,----
| Local "newValue" shadows a field of the same name in
| "org.apache.tools.ant.taskdefs.optional.PropertyFile$Entry
`----

> I guess I also picked up using static methods directly instead of
> thru an instance ref, but what else is there?

Yes.

All of the fixes I committed have either been for shadowed defintions
or statics used as non-statics.  There also is a warning about switch
fall-throughs in JavaEnvUtils but they are wanted.

Stefan

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to