On 07/13/2012 04:05 PM, Pádraig Brady wrote: > On 07/13/2012 04:03 PM, Jim Meyering wrote: >> Pádraig Brady wrote: >>> On 07/12/2012 05:11 PM, Pádraig Brady wrote: > >>> * src/df.c (hide_problematic_chars): Replace '\n' with '\?'. >> >> Thanks. That looks fine, but shouldn't the '\?' above be just '?' > > oops thanks. I adjusted a bit and now replace all control chars (< 0x20) with '?'. They're easy to handle in all encodings and can only be problematic. I added a test too. I've pushed the attached, so I'm marking this done. cheers, Pádraig.