You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@etch.apache.org by scott comer <we...@mac.com> on 2009/02/04 20:27:04 UTC
code cleanup...
while trying to change the headers in the java code to be apache
compliant i found
a code clean up tool in eclipse (like organize imports but more). it
removes superfluous
this.blah and unused private members and parens and casts and a few
other things.
sadly it doesn't change out file headers.
nonetheless, i ran it and tested the result and it is good and it found
good stuff to fix.
(i turned off code formatting changes.). since we've already muddied the
waters by
moving all these files around, this might a good time to pile on and
commit the result
of the code cleanup tool, too.
what say you good people?
scott out
RE: code cleanup...
Posted by "Rene Barraza (rebarraz)" <re...@cisco.com>.
+1, er... I mean. Yes. Please. :)
-René
-----Original Message-----
From: scott comer [mailto:wert1y@mac.com]
Sent: Wednesday, February 04, 2009 1:27 PM
To: etch-dev@incubator.apache.org
Subject: code cleanup...
while trying to change the headers in the java code to be apache
compliant i found
a code clean up tool in eclipse (like organize imports but more). it
removes superfluous
this.blah and unused private members and parens and casts and a few
other things.
sadly it doesn't change out file headers.
nonetheless, i ran it and tested the result and it is good and it found
good stuff to fix.
(i turned off code formatting changes.). since we've already muddied the
waters by
moving all these files around, this might a good time to pile on and
commit the result
of the code cleanup tool, too.
what say you good people?
scott out
No virus found in this incoming message.
Checked by AVG - www.avg.com
Version: 8.0.233 / Virus Database: 270.10.18/1935 - Release Date: 02/04/09 16:35:00
Re: code cleanup...
Posted by James Dixson <ja...@cisco.com>.
Here, here!
On 2/4/09 1:27 PM, "scott comer" <we...@mac.com> wrote:
> while trying to change the headers in the java code to be apache
> compliant i found
> a code clean up tool in eclipse (like organize imports but more). it
> removes superfluous
> this.blah and unused private members and parens and casts and a few
> other things.
> sadly it doesn't change out file headers.
>
> nonetheless, i ran it and tested the result and it is good and it found
> good stuff to fix.
> (i turned off code formatting changes.). since we've already muddied the
> waters by
> moving all these files around, this might a good time to pile on and
> commit the result
> of the code cleanup tool, too.
>
> what say you good people?
>
> scott out
>
--
James Dixson
Manager, Software Development
CUAE Engineering, Cisco Systems
(e) jadixson@cisco.com
(p) 512-336-3305
(m) 512-968-2116