>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
>close.

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

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.

--Ken

_______________________________________________
Nmh-workers mailing list
[email protected]
https://lists.nongnu.org/mailman/listinfo/nmh-workers

Reply via email to