running ident on modular xorg tree

[Jim Gettys]

Jon,

On Wed, 2004-09-08 at 18:26, Jon Smirl wrote:
> Pick whatever rules you want. Now is just a good time to run indent so
> that we don't mess up the CVS change logs. The refactoring has already
> toasted the change histories so nothing is lost. This opportunity
> doesn't happen every day.

The refactoring will not toast the change histories.  We are
certainly preserving the CVS histories on files.

And we will *not* be doing this blindly over the source pool;
only on egregious offenders.

First thing to determine is what "X Window System" indenting
style is, and to get it documented on the web site.
As Alan says, there is a predominant style, not
that I can currently describe it off the top of my head.

And, by the way, if subsystems are already consistent in some
other style, that doesn't mean we should re-indent them.

I believe the only things that should be cleaned up are places
where the style is all screwed up inside a single subsystem/file
so that the code is actually hard to maintain.

I don't want to see any of these changes checked in until there
is pretty good agreement on what is going to get done.

			- Jim


> 
> On Wed, 8 Sep 2004 13:59:59 -0700 (PDT), Jeremy C. Reed
> <reed at reedmedia.net> wrote:
> > On Tue, 7 Sep 2004, Jon Smirl wrote:
> > 
> > > Would it make sense to run indent over the new xorg modular source
> > > tree before a lot of people start using it? Then make it a policy that
> > > all check-ins are run through indent too? Given that the tree has been
> > > so heavily edited there's no real log trail that will be lost. Would
> > > it be good to use the same indent rules that the kernel uses? I'm not
> > 
> > What kernel?  :)
> > 
> > Like
> > http://cvsweb.netbsd.org/bsdweb.cgi/src/share/misc/indent.pro
> > http://cvsweb.netbsd.org/bsdweb.cgi/src/share/misc/style
> > http://www.openbsd.org/cgi-bin/man.cgi?query=style&apropos=0&sektion=0&manpath=OpenBSD%20Current&arch=i386&format=html
> > http://www.FreeBSD.org/cgi/man.cgi?query=style&sektion=9&apropos=0&manpath=FreeBSD+5.2.1-RELEASE
> > 
> > > working in the xorg tree currently, but we're talking about doing this
> > > to the drm tree.
> > 
> > Since the code is getting farther from other implementations, it does make
> > sense and seems like it would be a good idea to have a consistent style.
> > 
> >  Jeremy C. Reed
> > 
> >                          open source, Unix, *BSD, Linux training
> >                          http://www.pugetsoundtechnology.com/
> > 
> > 
> 
>