[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Nmh-workers] configurable part separator lines

From: Ken Hornstein
Subject: Re: [Nmh-workers] configurable part separator lines
Date: Wed, 23 Apr 2014 22:44:08 -0400

>this issue was rather a big itch for me, so i had to scratch it.  i
>agree about "not 1.6", but i think i might have it, or something

I'm mostly okay with this, but there's one thing I'd like to see

I'm not fond of the idea of the new pseudo-component you created called
"hidden".  I know you can use it as a boolean, but defining it to a fixed
string seems wrong to me.

There is an existing function called %(unseen); I think it would be perfect
for this.  The way you pass in values to unseen is via the "dat" array
(the 5th argument to fmt_scan).  Index value 4 will be the value of
%(unseen); see the comments in h/fmt_scan.h for more information.  That
just feels closer to how the format engine should work.  There is precedent
to overloading those dat[] entries to be used for other things, so this
isn't new ground here.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]