running ident on modular xorg tree
Jim.Gettys at hp.com
Thu Sep 9 07:17:38 PDT 2004
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.
> 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/
More information about the xorg