Menu

#50 patch for w3m m17n

open
nobody
None
5
2013-11-25
2003-12-03
No

I created this patch for w3m-m17n 0.4.1. It
automagically sets the display charset to the current
locale's charset. Please apply it. The m17n author did
not reply when I sent him the patch. This patch is
already in the SuSE 9.0 w3m-m17n.
Thanks
Bjoern Jacke

Discussion

  • Bjoern Jacke

    Bjoern Jacke - 2003-12-03

    patch to set display charset from locale

     
  • Andreas Metzler

    Andreas Metzler - 2004-01-30

    Logged In: YES
    user_id=696153

    Afaict this patch is broken:
    | DisplayCharset = wc_guess_locale_charset(Locale,
    DisplayCharset)

    "Locale" holds the name of the locale (i.e. en_US) instead
    of a charset. You want to use nl_langinfo(CODESET).

    BTW it seems to have been applied in CVS.

     
  • Andreas Metzler

    Andreas Metzler - 2004-01-30

    Logged In: YES
    user_id=696153

    After testing I can confirm that current CVS is broken. - It
    completely ignores the locale setting, defaulting to UTF-8.

     
  • Nobody/Anonymous

    Logged In: NO

    no it's not borken, the charset (what
    nl_langinfo(CODESET) returns) does belong to the locale.
    W3m does not use the POSIX way (nl_langinfo()) to find
    out about the locale settings but uses its own lib for
    that. This patch also uses the same mechanism,
    w3m-m17n uses and conforms to that.

     
  • Nobody/Anonymous

    Logged In: NO

    I meant the patch is not broken. It should be applied to
    CVS.

     
  • Tatsuya Kinoshita

    Seems fixed in w3m 0.5.1.

     

Log in to post a comment.