[w3m-dev-en 00248] Re: fm.h patch

From: Alex Lyons (alex.lyons@aeat.co.uk)
Date: Tue Oct 17 2000 - 03:41:20 CDT

  • Next message: Ambrose Li: "[w3m-dev-en 00249] Re: fm.h patch"

    Michael T. Babcock wrote:

    > I had to make a quick modification to fm.h for w3c 0.1.10 to compile
    > on my Linux machine:

    (snip)

    > I can provide my config.h if necessary. Point being, there are
    > parts of the code that require PC_KANJIx to be #define'd even if
    > not running in Japanese.

    I only found one instance of PC_KANJI2 being used outside an
    ifdef JP_CHARSET, so found it easier to fix that:

    *** display.c.orig Tue Jun 6 06:39:13 2000
    --- display.c Thu Jun 22 10:08:44 2000
    ***************
    *** 717,724 ****
    --- 717,726 ----
          }
          else {
              buf->pos = l->len -1;
    + #ifdef JP_CHARSET
              if (CharType(p[buf->pos]) == PC_KANJI2)
                  buf->pos--;
    + #endif /* JP_CHARSET */
          }
          cpos = COLPOS(l, buf->pos);
          buf->visualpos = cpos - buf->currentColumn;

    Mind you, I'm on Solaris: there might be others on Linux.

    I note that display.c is fixed in w3m-0.1.11-pre, although there
    was a different leakage of JP_CHARSET code in this newer version,
    this time in file.c:

    *** file.c.orig Wed Jun 14 11:35:40 2000
    --- file.c Thu Jul 6 09:24:17 2000
    ***************
    *** 5169,5174 ****
    --- 5169,5176 ----
          URLFile uf;
      #ifdef JP_CHARSET
          char code = buf->document_code, ic;
    + #else
    + char code = '\0';
      #endif
          int squeeze_flag = 0;
          
    This may be fixed as well by now, I haven't checked to see whether
    the prerelease source has changed since June (does w3m-dev-en get
    notified when a new release or prerelease becomes available for
    download?).

    Alex.

    ***********************************************************************
    This transmission contains information which may be confidential and
    which may also be privileged. It is intended for the named addressee
    only. Unless you are the named addressee, or authorised to receive it
    on behalf of the addressee you may not copy or use it, or disclose it
    to anyone else. If you have received this transmission in error please
    contact the sender. Thank you for your cooperation.
    ***********************************************************************

    For more information about AEA Technology please visit our website at http://www.aeat.co.uk

    AEA Technology plc registered office 329 Harwell, Didcot, Oxfordshire OX11 0QJ.
    Registered in England and Wales, number 3095862.



    This archive was generated by hypermail 2b29 : Tue Oct 17 2000 - 04:00:37 CDT