From: Robert Millan <rmh@aybabtu.com>
To: "Michel Dänzer" <michel@daenzer.net>
Cc: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: hfs patch (Re: State of GRUB on PowerPC)
Date: Tue, 10 Feb 2009 15:47:33 +0100 [thread overview]
Message-ID: <20090210144733.GC25305@thorin> (raw)
In-Reply-To: <1234259695.5081.680.camel@thor.local>
On Tue, Feb 10, 2009 at 10:54:55AM +0100, Michel Dänzer wrote:
> > >> +static unsigned char caseorder[256] = {
> > >> + 0x00,0x01,0x02,0x03,0x04,0x05,0x06,0x07,0x08,0x09,0x0A,0x0B,0x0C,0x0D,0x0E,0x0F,
> > >
> > > Could you be more specific about what the table contents mean?
>
> BTW, let me point out again that this table including comment is a
> verbatim copy from the Linux kernel fs/hfs/string.c . Sorry if I didn't
> make this clear enough in my original post.
Since it's probably not copyright-significant, it may not be a problem to
copy it from Linux sources. However, it's also possible for a list of
numbers to be copyright significant, depending on their meaning (which
is linked to my other question below).
> > Michel may know better, but I think it's the order of characters.
> > Those with the lower order go first in the sorted binary tree. Those
> > with the same order are equivalent on the filesystem level. That is,
> > "foo" can only be between "bar" and "quux" in the node tree. "foo"
> > and "Foo" are the same tree node and thus the same file.
>
> I think that's a nice summary, thanks.
Ok. There's a pair of things that need to be cleaned up though. If I
understand correctly, the definition of caseorder[i] is such that given too
distinct values of i, it can be used to sort them (if this is so, I think
it should be explicitly mentioned in the comment).
So if the table is basicaly storing values that enumerate something, why are
we using hex to represent them? Hex gives the impression they're an opaque
sort of thing, like code, bitmasks or magic numbers.
The reference to "the Macintosh" is a bit confusing, it usually means a
computer, or an OS. I assume it refers to HFS?
We'd also need to know what are these "'casefold' and 'order' tables from
ARDI's code", and what exactly means this is a "composition" of them.
--
Robert Millan
The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
how) you may access your data; but nobody's threatening your freedom: we
still allow you to remove your data and not access it at all."
next prev parent reply other threads:[~2009-02-10 14:48 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-27 7:19 State of GRUB on PowerPC Michel Dänzer
2009-01-27 16:54 ` Vesa Jääskeläinen
2009-01-27 17:15 ` Michel Dänzer
2009-01-29 2:57 ` Pavel Roskin
2009-01-29 16:18 ` Michel Dänzer
2009-02-07 19:48 ` hfs patch (Re: State of GRUB on PowerPC) Robert Millan
2009-02-08 4:38 ` Pavel Roskin
2009-02-09 14:19 ` Robert Millan
2009-02-10 9:55 ` Michel Dänzer
2009-02-10 9:54 ` Michel Dänzer
2009-02-10 10:50 ` Jordi Mallach
2009-02-10 12:06 ` Jordi Mallach
2009-02-10 12:12 ` Michel Dänzer
2009-02-21 12:46 ` Robert Millan
2009-02-10 14:47 ` Robert Millan [this message]
2009-02-11 9:24 ` Michel Dänzer
2009-02-21 13:01 ` Robert Millan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090210144733.GC25305@thorin \
--to=rmh@aybabtu.com \
--cc=grub-devel@gnu.org \
--cc=michel@daenzer.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.