makes sense. I don't think I ever actually tested Matteo's patch. I still support this feature and behavior.
- Rick On Thu, Feb 25, 2010 at 7:55 AM, binary.koala <[email protected]>wrote: > 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. > > -- > Allow binary dumping to stdout > https://bugs.launchpad.net/bugs/192174 > You received this bug notification because you are a direct subscriber > of the bug. > -- Allow binary dumping to stdout https://bugs.launchpad.net/bugs/192174 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
