grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Split big and little endian BeFS and AtheFS


From: Pavel Roskin
Subject: Re: [PATCH] Split big and little endian BeFS and AtheFS
Date: Thu, 20 Aug 2009 18:16:24 -0400

On Wed, 2009-08-19 at 12:05 +0200, Vladimir 'phcoder' Serbinenko wrote:

> > I'm fine with the split, but please use something more descriptive than
> > U16, U32 and U64.  Maybe fs_to_cpu16() etc.
> >
> U* was here before me but here is a patch w/o them

Sorry, I didn't realize that.  Then maybe it would be better to apply
that part separately after the split.

Please define macros with arguments.  It makes the code more readable.

Please don't introduce excessively long lines in *.rmk files.  Newlines
are cheap :-)

I don't think we should rename "byte_order" to "unused".  Just because
we doesn't use it now to determine the endianess, it doesn't mean that
the field becomes meaningless.  We may want to check it in the future.

There are other fields in the superblock that we don't use, such as
"flags", yet we don't rename them to "unusedN".

Apart from that, I'm fine with the patch.

-- 
Regards,
Pavel Roskin




reply via email to

[Prev in Thread] Current Thread [Next in Thread]