send.c:_pipecmd(): fix [81473f8]: again ensure *result is set..
commitdee1fed19eea6e2765c8e24c577b624eb3bc6e54
authorSteffen (Daode) Nurpmeso <sdaoden@users.sf.net>
Mon, 11 Aug 2014 19:38:29 +0000 (11 21:38 +0200)
committerSteffen (Daode) Nurpmeso <sdaoden@users.sf.net>
Mon, 11 Aug 2014 19:39:22 +0000 (11 21:39 +0200)
tree110c9340d7d3ed6a02865d9b406f839600ae36c2
parent1112375c2b4e2cd8e7d5af46e6ce158c4f6c9230
send.c:_pipecmd(): fix [81473f8]: again ensure *result is set..

In [81473f8] (Add *pipe-EXTENSION*, ext. *mime-counter-evidence*..,
2014-07-11) i unfortunately broke the special `@' text handlers by
no longer setting *result to NULL at function entry.

The real cause of this bug however is the temporary colourization
hack (i don't want to miss it again), since when i implemented the
extended *mime-counter-evidence* stuff i reduced _pipecmd() to
it's essence, the problem being that the local variable
send.c:sendpart():pipecomm gets misused as temporary storage for
the colourization -- due to this it is *not* NULL if _pipecmd()
returns PIPE_TEXT to indicate forced treatment as text.

I haven't yet seen this rather ultimate MCA because i do ignore
all header fields (which come into play for MIME parts) for `p',
and only display them for `P'.
v14.8 will come with a test series.  (sigh.)
send.c