Date: Mon, 7 Aug 2006 11:51:59 -0500
From: Dean Roehrich <dean.roehrich@sun.com>
To: Chris Wedgwood <cw@f00f.org>
Cc: Russell Cattelan <cattelan@thebarn.com>, Bill Kendall <wkendall@sgi.com>,
        Vlad Apostolov <vapo@sgi.com>, xfs@oss.sgi.com
Subject: Re: review: Simple patch to remove the dmapi support from xfsdump
Message-ID: <20060807165159.GA10521@kickball-mn.Central.Sun.COM>
References: <44D2CA85.3040208@sgi.com> <20060804141012.GA26@kickball-mn.Central.Sun.COM> <44D36985.1090006@thebarn.com> <20060804155850.GA3338@kickball-mn.Central.Sun.COM> <44D379A6.9040200@sgi.com> <44D38D34.1010503@thebarn.com> <44D3C351.7060109@sgi.com> <20060807150324.GA8421@kickball-mn.Central.Sun.COM> <44D75C87.8050402@thebarn.com> <20060807155248.GA22411@tuatara.stupidest.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20060807155248.GA22411@tuatara.stupidest.org>
User-Agent: Mutt/1.5.9i
Sender: xfs-bounce@oss.sgi.com
Errors-to: xfs-bounce@oss.sgi.com
Precedence: bulk
Content-Length: 760
Lines: 18

On Mon, Aug 07, 2006 at 08:52:48AM -0700, Chris Wedgwood wrote:
> On Mon, Aug 07, 2006 at 10:30:15AM -0500, Russell Cattelan wrote:
> 
> > It seems rather dangerous to have a #define floating around that
> > could potentially get out of sync with the original, especially if
> > you transport the number and not the enum table. (It make it really
> > hard for cscope to find :-)
> 
> If it saves a header I don't see any harm in the original approach
> with a comment preceding it explain what the bits are and why not to
> screw them up.

I agree with Russell that you want cscope to tell you about every place those
bits are being used, so you want those defines or at least those symbols to
show up here.  Having them in a comment won't help cscope.

Dean