[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Nmh-workers] mhshow display bug
From: |
Paul Fox |
Subject: |
Re: [Nmh-workers] mhshow display bug |
Date: |
Sat, 12 Apr 2014 08:24:37 -0400 |
david wrote:
> part text/plain 1270
> > >right. i've now verified that InitText() is never being called, which
> > >would have initialized ct->c_termproc. so that means an initialization
> > >of ct->c_ctinitfnx is being missed. and now i really _am_ lost. :-)
> >
> > How did you verify that InitText() was never called? I set a breakpoint
> > for it and it is definitely being called for text parts, at least on the
> > messages I tested.
>
> Same for me. Your profile entry, Paul, looks like it works
> properly for me. I even inserted "tr '[a-z]' '[A-Z]'" in the
> pipeline and it upper-cased the content.
i've done some more work, with gdb this time. InitText is called
just fine -- sorry for that noise.
do you have -concat set? the breakage is happening for me in
show_text, where buffer is set to either a display program ("%lless
%F") or not ("%l") based on concatsw. i'm in the latter case, so
when i later get into parse_display_string, the "%l" is consumed, and
then there's nothing left of buffer to plug into ct->c_termproc.
having now given this all of 5 minutes of thought, i'd say that
the conditional in show_text should go away. whether concatsw
is set or not, it should always be okay to run the user's charset
converter.
paul
>
> Is your locale something like en_US.iso88591? If so, the
> check_charset() at the end of InitText() would pass, so
> ct->c_termproc wouldn't be loaded. If InitText() is in fact
> called, that could explain what you're seeing. But that
> part of InitText() hasn't changed since 2008, and appears to
> work as documented in the mhshow(1) man page.
>
> parse_display_string() is new, but it mainly factors out two
> bunches of nearly identical code. It also has fixes to the
> quoting of escape sequences. It's handling of
> ct->c_termproc should be identical to 1.5.
>
> David
>
> _______________________________________________
> Nmh-workers mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/nmh-workers
----------------------
paul fox, address@hidden (arlington, ma, where it's 47.5 degrees)
Re: [Nmh-workers] mhshow display bug, David Levine, 2014/04/13
Re: [Nmh-workers] mhshow display bug, David Levine, 2014/04/13