Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 11 Jun 2004 16:07:40 +0200
From:      Jens Schweikhardt <schweikh@schweikhardt.net>
To:        Chip Norkus <wd@teleri.net>
Cc:        hackers@freebsd.org
Subject:   Re: indent bugfix / added features
Message-ID:  <20040611140740.GA3355@schweikhardt.net>
In-Reply-To: <20040611025307.GG17367@teleri.net>
References:  <20040611025307.GG17367@teleri.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Jun 10, 2004 at 09:53:07PM -0500, Chip Norkus wrote:
# 
# Hi all,
# 
#   I've been working on a major project, and decided to use indent to
# normalize the code a bit.  In doing so I discovered a few deficencies in
# the stock FreeBSD (5.2-CURRENT) indent and decided to fix them, I
# thought these might be fairly common wishes (and one of them is a
# bugfix) and have attached a patch which does the following:
# * Adds an option to control tab use in output code (-ut and -nut)
#   (This is in the GNU indent)
# * Adds an option to control brace placement after function declaration
#   (-fbs and -nfbs).  What this does, specifically, is allow you to have
#   indent produce 'func(args..) {' instead of 'func(args..)\n{'.  The old
#   behavior is the default.
# * Fixes the indent bug when aligning variable names with an indentation
#   level that is not a multiple of 8.  This works for both the 'tab use'
#   and 'no tab use' varieties of code output.
# 
#   I don't know if anyone would be interested in committing the bugfix (I
# believe it is correct) or added features, but I hope someone else finds
# this useful.
# 
# -chip

I'm willing to commit this if you could demonstrate that it will not
produce different output in the default case than it does now. Say,
run the old an new versions against the FreeBSD src tree and make a diff
which should vanish apart from the bug-fixing effects. If you want to
make yourself known as a quality software engineer, do the same for
various sets of indent options :-)

Regards,

	Jens
-- 
Jens Schweikhardt http://www.schweikhardt.net/
SIGSIG -- signature too long (core dumped)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040611140740.GA3355>