Re: [hackers] [sent] [PATCH 2/2] replace farbfeld with libnetpbm

From: Grant Mathews <grant.m.mathews_AT_gmail.com>
Date: Thu, 10 Dec 2015 21:55:08 -0800

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
Received on Fri Dec 11 2015 - 06:55:08 CET