[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

FW: [frogs] indentation

From: Carl D. Sorensen
Subject: FW: [frogs] indentation
Date: Mon, 12 Jan 2009 17:35:16 -0700


Can you add a feature request for identifying automatic style programs that
will work for each of our source files, defining the necessary syntax files,
and figuring out how to automatically modify the source files?



------ Forwarded Message
From: Graham Percival <address@hidden>
Reply-To: <address@hidden>
Date: Fri, 9 Jan 2009 08:10:13 -0700
To: <address@hidden>
Conversation: [frogs] indentation
Subject: Re: [frogs] indentation

On Thu, Jan 08, 2009 at 06:38:53PM -0700, Carl D. Sorensen wrote:
> On 1/8/09 2:18 PM, "Valentin Villenave" <address@hidden> 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.)

- Graham


Join the Frogs!

------ End of Forwarded Message

reply via email to

[Prev in Thread] Current Thread [Next in Thread]