[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20210128070541.ynzsgpniyo2xe23k@kari-VirtualBox>
Date: Thu, 28 Jan 2021 09:05:41 +0200
From: Kari Argillander <kari.argillander@...il.com>
To: Mark Harmstone <mark@...mstone.com>
Cc: Konstantin Komarov <almaz.alexandrovich@...agon-software.com>,
linux-fsdevel@...r.kernel.org, viro@...iv.linux.org.uk,
linux-kernel@...r.kernel.org, pali@...nel.org, dsterba@...e.cz,
aaptel@...e.com, willy@...radead.org, rdunlap@...radead.org,
joe@...ches.com, nborisov@...e.com,
linux-ntfs-dev@...ts.sourceforge.net, anton@...era.com,
dan.carpenter@...cle.com, hch@....de, ebiggers@...nel.org,
andy.lavr@...il.com
Subject: Re: [PATCH v18 01/10] fs/ntfs3: Add headers and misc files
On Fri, Jan 22, 2021 at 02:55:30PM +0000, Mark Harmstone wrote:
> On 22/1/21 2:01 pm, Konstantin Komarov wrote:
> > diff --git a/fs/ntfs3/upcase.c b/fs/ntfs3/upcase.c
> > +static inline u16 upcase_unicode_char(const u16 *upcase, u16 chr)
> > +{
> > + if (chr < 'a')
> > + return chr;
> > +
> > + if (chr <= 'z')
> > + return chr - ('a' - 'A');
> > +
> > + return upcase[chr];
> > +}
>
> Shouldn't upcase_unicode_char be using the NTFS pseudo-file $UpCase?
> That way you should also be covered for other bicameral alphabets.
return upcase[chr] is just for that? Upcase table from $UpCase is constucted
in super.c and this will get it in and use it.
Powered by blists - more mailing lists