lkml.org 
[lkml]   [2007]   [Apr]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] console UTF-8 fixes
    On Wed, Apr 11, 2007 at 11:36:40AM -0700, H. Peter Anvin wrote:
    > Egmont Koblinger wrote:
    > >+static int is_zero_width(long ucs)
    > >+{
    > >+ static const struct interval zero_width[] = {
    > /* lots */
    > >+ };
    >
    > I'm still unhappy about these large search tables in the kernel, not
    > because they take a huge amount of space (it's not that much), but
    > because they're invariably going to be stale, as they're Unicode-version
    > dependent.

    I don't think width information for characters in BMP is going to change
    that often.

    By the way, a note about the size: the larger one of the two tables is
    unused and hence optimised away by the compiler. I just left in the source
    so that it only takes a minor modification for people go get a different
    sane behavior (ie. ignore combining chars). So only the small table, with 11
    pairs of longs (88 bytes) are compiled to the kernel.

    > At least please put them in a separate .c file and include a script to
    > generate them clean from UnicodeData.txt.

    I'll look at it, but I didn't want to alter the building procedure, modify
    Makefiles... Or do you mean I should only ship the generated .c file plus
    the script, instead of the (1MB) UnicodeData.txt and generating it compile
    time? Sounds reasonable...

    > Besides, would it not make more sense to have a single table with the
    > width information, if you insist on having one, instead of multiple ones?

    I've been thinking on it and I'm not sure which one the right way is. The
    reason for choosing this was probably that this way information that is not
    used by the code can be omitted by the compiler.



    --
    Egmont
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2007-04-12 11:13    [W:0.027 / U:0.084 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site