Fix order of (trim) and (decode) in repl*comps
authormarkus schnalke <meillo@marmaro.de>
Wed, 20 Apr 2016 07:02:06 +0000 (09:02 +0200)
committermarkus schnalke <meillo@marmaro.de>
Thu, 21 Apr 2016 08:07:59 +0000 (10:07 +0200)
commitf3575c51862ea3ef58aa9acae72505054d510143
treec0ad79a223c82804991809bc4ae5c22af7f11d66
parent72d6c7764974246384bb5daa7cc8b4f0a516621b
Fix order of (trim) and (decode) in repl*comps

Before MH had (decode), the format escapes used to be:
%(void{comp})%(trim)%(putstr)
i.e. set the internal str register to the component text, trim
trailing whitespace from it, then print the value. Later (putstr)
was simply replaced by (decode), which first RFC-2047 decodes the
string and then prints it (if it is the outermost function). That
almost always produced the correct result. But this way the
whitespace trimming is done at the wrong point. Correct is to first
decode the string and then trim it. tests/scan/test-mh-format
checks for that.
etc/replcomps
etc/replgroupcomps
man/mh-format.man5