Re: [frogs] indentation

[ Thread Index | Date Index | More lilynet.net/frogs Archives ]


On Thu, Jan 08, 2009 at 06:38:53PM -0700, Carl D. Sorensen wrote:
> 
> On 1/8/09 2:18 PM, "Valentin Villenave" <v.villenave@xxxxxxxxx> wrote:
> 
> > Perhaps "we" (meaning "you guys") could start listing such
> > inconsistencies you may find through the source code... This will make
> > code cleaning easier when the time comes (since I believe code
> > cleaning is also one of the long-term forg's tasks).
> 
> I agree.  I think we'll have frogs do some "code-janitor" work as described
> in the Request for Help page.  But that will be later.
> 
> If we're going to do that, I think we should just do it by file.  No need to
> track individual occurrences.

For things like tabs vs. spaces, I suggest looking into something
like astyle.  First, it's automagic, and second, it's a lot nicer
to see "whitespace changes" in the changelog -- when you're
looking for a bug, you knwo that you can skip over that stuff.

If you think you have nice style options, be it for astyle,
indent, or any other program, check on -devel for both comments on
the automatic style, and for overall permission to modify the
files automatically.

(I can't imagine Han-Wen refusing such permission, but the owner
of my second-biggest open-source project refused it for over a
year.  In the end I just did it and dared him to object.  It
helped that he was the supervisor for my Master's degree.)

Cheers,
- Graham


---

----
Join the Frogs!


Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/