On Fri, Dec 11, 2015 at 01:30:52AM +0100, Markus Teich wrote: > sent with farbfeld uses a separate process instead of linking to a > library and is therefore more UNIXy and sucks less.
This patch still does all the file conversion in a separate process, and sent still uses an external library regardless of whether this patch is merged. I'm a big fan of process-level code reuse, but no tool is right for every job, and sometimes a library is the correct way to go. Aside from those points: would rewriting the patch to not use libnetpbm, but to use the Netpbm helpers instead, be acceptable? - Grant