All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pavel Roskin <proski@gnu.org>
To: The development of GRUB 2 <grub-devel@gnu.org>
Subject: Re: [PATCH] support of hfsx ( case comparaison )
Date: Sun, 07 Jun 2009 13:40:44 -0400	[thread overview]
Message-ID: <1244396444.27559.17.camel@mj> (raw)
In-Reply-To: <d7ead6de0906070210t1ecfc9f1o64b201df258c0940@mail.gmail.com>

On Sun, 2009-06-07 at 11:10 +0200, Vladimir 'phcoder' Serbinenko wrote:
> On Sun, Jun 7, 2009 at 4:18 AM, Pavel Roskin<proski@gnu.org> wrote:
> >>  Improving
> >> strcasecmp is possible and may even be compact. Even if unicode counts
> >> a lot of alphabets only few are bicameral. AFAIK main ones are Latin,
> >> Greek, Cyrillic and Armenian. I hope that in most cases the lowercase
> >> conversion can be done with some simple arithmetic operations
> >
> > We are using grub_strcasecmp() on data in other encodings in some cases.
> > I think short names on the FAT filesystem are never in UTF-8.
> Normally not but fat code passes short filenames to grub without any
> encoding conversion. In other words fat code assumes short filenames
> are in utf8. A problem in both hfs and fat is that it uses local
> encoding and at least in case of fat it's impossible to know which one
> from filesystem alone. We would need something like mount option to
> get this right. It could be sth like
> hd0_0_codepage=437

Fortunately, it would only matter if the user tries to access a file
with non-ASCII characters using a capitalization different from the one
used by the filesystem.

> > Also,
> > case comparison depends on the locale.  That's too much complexity for
> > the code GRUB.  I would prefer the case comparison for hfsplus to be in
> > the hfsplus module if we decide to implement it correctly.
> I agree with you. I see no reason for users to have unicode characters
> in kernel names. And so getting it right is too much work for an
> almost zero benefit

I case of hfsplus, there is a chance to miss a file in the binary tree
if there are non-ASCII files in the same directory.  But I agree, it's
not worth the trouble at this point.

-- 
Regards,
Pavel Roskin



  reply	other threads:[~2009-06-07 17:40 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-12 21:52 [PATCH] support of hfsx ( case comparaison ) Michael Scherer
2009-02-07 21:02 ` Robert Millan
2009-05-03 17:19   ` Michael Scherer
2009-06-01 10:11     ` Vladimir 'phcoder' Serbinenko
2009-06-03  7:56       ` Michael Scherer
2009-06-03  9:26         ` Vladimir 'phcoder' Serbinenko
2009-06-03 10:54           ` Michael Scherer
2009-06-03 21:23             ` Pavel Roskin
2009-06-05 19:23               ` Michael Scherer
2009-06-05 19:46                 ` Vladimir 'phcoder' Serbinenko
2009-06-07  2:18                   ` Pavel Roskin
2009-06-07  9:10                     ` Vladimir 'phcoder' Serbinenko
2009-06-07 17:40                       ` Pavel Roskin [this message]
2009-06-05 21:08                 ` Pavel Roskin

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=1244396444.27559.17.camel@mj \
    --to=proski@gnu.org \
    --cc=grub-devel@gnu.org \
    /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.