Re: What about a indent target?

"Jack O'Quin" <[email protected]>
Newsgroups gmane.comp.audio.jamin.devel
Message-ID <[email protected]>
On 5/14/07, Emmanuel Saracco <[email protected]> wrote:
> Hi,
>
> I saw that "indent" was used after the dynamic creation of "state.h".
>
> What about a "indent" target for all the source code, automatically used
> on "dist-all" target call before application packaging?
>
> If you are ok, just apply the following tiny patch (change the
> ".indent.pro" content to fit your coding rules):
>
> * Added "indent" target in "Makefile.am"
>
> * Added ".indent.pro" for "indent" options
>
> Yes, "esthetic" only, again... :-)

Not just aesthetic: seems like it could interact badly with
CVS, making source files appear modified when they are
not.

If the indentation of our code is bad enough to need this,
why not just fix it in the CVS tree?
-- 
 joq

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.