On 2/24/18 1:33 AM, Herbert Xu wrote:
On Wed, Feb 21, 2018 at 11:47:58PM +0100, Harald van Dijk wrote:
On 2/21/18 2:50 PM, Denys Vlasenko wrote:
I propose replacing this:
Agreed, that looks better. Thanks!
Good work guys. However, could you check to see if this patch
works too? It passes all my tests so far.
It seems like the new control character doesn't get escaped in one place
where it should be, and gets lost because of that:
Unpatched:
$ dash -c 'x=`printf \\\211X`; echo $x | cat -v'
M-^IX
Patched:
$ src/dash -c 'x=`printf \\\211X`; echo $x | cat -v'
X
Cheers,
Harald van Dijk
--
To unsubscribe from this list: send the line "unsubscribe dash" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
