there seems to be a slight logical mistake in Matteo's patch. Unless it is a 'feature' to have '\n' after each packet the very last lines of the patch should read:
<code> - putchar('\n'); + if(strip_nonprint) + putchar('\n'); fflush(stdout); </code>
i.e. if(strip_nonprint) statement needs to be inverted.
there seems to be a slight logical mistake in Matteo's patch.
Unless it is a 'feature' to have '\n' after each packet the very last lines of the patch should read:
<code>
- putchar('\n');
+ if(strip_nonprint)
+ putchar('\n');
fflush(stdout);
</code>
i.e. if(strip_nonprint) statement needs to be inverted.