Re: [PATCH] Fix warning in fs/xfs.c

[Pavel Roskin <proski@gnu.org>]

On Sun, 2008-07-20 at 20:50 +0200, Marco Gerards wrote:
> Pavel Roskin <proski@gnu.org> writes:
> > i4 and i8 are generally used by Linux XFS code to describe 32-bit and
> > 64-bit values if either can be used.  The "smallino" field was highly
> > misleading because it had to be negated.  It's the number of "big" (i8
> > or 64-bit) entries.  If it's 0, then the entries are "small".
> >
> > So it was natural to call it "i8count".  And once it was "i8count", it
> > was natural to call the first value "count".
> >
> > If you prefer another naming convention, let's rename the entries
> > according to it.  I was thinking having 2 32-bit integers "parent_hi"
> > and "parent_lo" or something like that.  Anyway, let's not use
> > "smallino" - "bigentries" would be better.
> 
> What I suggest is that you pick the names yourself or from a standard,
> instead of from Linux code.

OK, I'll try to be more creative.

-- 
Regards,
Pavel Roskin