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

From: FRIGN <dev_AT_frign.de>
Date: Thu, 10 Dec 2015 16:39:49 +0100

On Wed, 9 Dec 2015 20:56:38 -0800
Grant Mathews <grant.m.mathews_AT_gmail.com> wrote:

Hey Grant,

> Since the PNM/PAM format already exist as a minimal intermediate
> representation with a rich set of commandline tools to manipulate them,
> use Netpbm to handle images.

back to your desk, Grant, you've missed the purpose of farbfeld.

No, to be fair, we discussed the PAM-format at slcon2 in more than one
coffee break.
Netpbm is arguably almost more complex than BMP and not easy to handle.
We could discuss it if it was widely used, but the main point is:
__You need a library to handle it__ and it's not that much of a popular
format to justify installing a library for it. I don't know about you
guys, but I don't have libpbm installed on my computer and even though
ffmpeg for instance offers support, I might be having a hard time
finding a format ffmpeg _doesn't_ support.

You've replaced the entire farbfeld parsing code in sent with the
boilerplate "offered" by netpbm. Good job!

A question for the diligent reader: Can you read in a netpbm file
without first looking into the docs?

Cheers

FRIGN

-- 
FRIGN <dev_AT_frign.de>
Received on Thu Dec 10 2015 - 16:39:49 CET

This archive was generated by hypermail 2.3.0 : Thu Dec 10 2015 - 16:48:12 CET