liblbx: Improve error handling in lbx_file_seek.

  - Return an error code for an invaled whence value instead of
    proceeding with uninitialized values.

  - Set an error code on one of the error paths that did not set one.

tests: Enable shell traces early in all scripts.

The result_ function we snarfed from Automake turns on set -x in our
test suite, but since we do not enable it from the beginning the test
logs have traces starting from the first test result and not before.

The result is very strange; enable traces early so we see everything.

lbximg: Fix management of external palette file handles.

We leak file handles for the external and override palettes because the
file is opened every time the --palette or --override options are set.
Fix that up, and also close the handles when finished.

tests: Allow custom test command wrappers

Add a new environment variable, TESTWRAPPER, which allows the tests
to be run easily in tools like valgrind or qemu.

docs: Add palette information for the tanm_xxx animations.

I thought these all had embedded palettes but at least tanm_001
has some unspecified palette entries.  The only palette for this
image that does not look horrible is fonts.lbx.002, so let's go
with that one.

build: Remove call to AC_CONFIG_MACRO_DIR.

This macro never did anything useful and should not be used.  The new-
fangled way is AC_CONFIG_MACRO_DIRS which might even work, but stick
with the tried-and-true method for now.

lbximg: Convert to new decoding API.

Convert lbximg to the new API.  The implementation looks a lot
like getframe, but we can make the memory usage much more compact.
Not really a coincidence because getframe was originally designed
to support this application.

liblbx: Implement new image reading API.

Add a new API which allows the application to essentially loop over
the drawing commands in an LBX frame.  This requires no runtime memory
allocations in the library, pushing all that out to the caller.

Conceptually this simplifies many applications because they can use
this to handle image data row-at-a-time.

For now, keep the old getframe function but change it to use the new
API under the hood.

liblbx: Rename LBX_ENOENT to LBX_EINVAL.

General invalid argument makes more sense to have.

tests: Source image header in a subshell.

Use a subshell when sourcing the image header in pam-formats so that
the shell variables do not persist past the end of the test.  While
unlikely, a value left over from an earlier test might result in
false positives in later tests.

tests: Add info on what each MOO2 image is actually testing.

Each MOO2 test image was selected to test a particular feature of the
image decoder.  That information was in the old regress.zsh script but
for some reason got dropped in the conversion.  Add that information
back as a TAP diagnostic line for each image.

tests: Zero out transparent pixels before calculating digest.

The colour value of transparent pixels is irrelevant from a correctness
perspective, but we are nevertheless including that information in the
tests.  While we could zero out the values in the PAM output code it's
better to make the tests more robust.

This adds an external dependency on netpbm's pamarith and pamchannel
tools for the test suite.  Restructure the tests so that these tools
are optional.  If the tools are not available (or not working) then we
must accept the fact that images with an alpha channel may have false
negatives (but not false positives) -- record a SKIP result for an MD5
mismatch in that case.

tests: Remove toplevel set -e usage.

Using set -e at the top level affects the entire script and makes it
difficult to work with commands that may legitimately return an error.
Regardless, set -e is still not so great for error checks because it
has some incredibly non-obvious semantics.

Better to just have correct error checks everywhere.  Fortunately, this
is the case for almost all the existing test code -- only the testdir
creation needs fixing.

liblbx: Expose image lead-in in public struct.

Since we've finally defined the meaning of this value it makes sense to
expose it to the user.  This eliminates another need for lbx_img_getinfo,
but maintain the old functionality there for now.

lbximg: Fix wart in frame scanline header format

The format of frame scanlines has been a little weird since day 1: We
had 2 "types" of scanline headers disambiguated by some non-trivial
logic, including whether or not it was the first decoded scanline in
the frame.

Here's a new, simpler approach, which has only one type of scanline
header, consistent meanings to the header fields, and by interpreting
the first 4 bytes as a "frame header" we get rid of the special case
for the first scanline, too.

Add a document in English which describes the LBX image file format, to
be used as the baseline format specification going forward.

Finally, update the contrived test cases for the new format.  All the
test cases using actual MOO2 data still pass.

lbximg: Return failure if there was a decoding error.

If any frame was not decoded because liblbx returned an error, we should
return a non-zero exit status from the tool.  Previously this would only
happen if *all* frames failed.

tests: Migrate remaining regress.zsh tests to TAP.

tests: Migrate the mainmenu.lbx tests to the TAP system.

This moves the first set of regress.zsh tests to the new framework.

build: Bump dxcommon for relevant fix-gnulib updates.

liblbx: Fix framebuffer allocation for 0x0 images.

The allocation strategy for the framebuffer is the "array2" method from
c-faq: an array of row pointers into a single large array containing all
rows.  This was not being freed correctly in the case with 0 rows, since
there would be no pointer to the row data at all in order to free it.

Fix that up by simply allocating a 1x1 framebuffer when it would
otherwise be empty.  While we're at it, also add an overflow check
on the multiplication.

tests: Add a simple test to validate PAM output formats.

This tests that the PAM output produces the correct image type for
all four of the lbximg modes.

lbximg: Add an option to control the output filenames.

This will be needed for postmortem analysis of test cases that extract
multiple images, since otherwise there is no easy way to prevent the
tool from overwriting the earlier outputs.

This was clearly intended to be a user setting from the start but for
some reason it was never wired up to a command-line option.

lbximg: Allow --identify as a long option.

There is no reason to not have a full word here.  The getopt_long
function allows abbreviations anyway, so --ident remains perfectly
fine as an option.

lbximg: Fix exit status in identify mode.

The tool was always returning an exit status of failure, because in
identify mode there was no case to set the exit status to success.

lbximg: Fix --override long option.

The long option for this feature was clearly never tested, as only the
short option spec was actually correct.

tests: First stab at a proper test suite.

Not much of a suite at the moment since it consists of a single test,
which exposes a corner-case in the memory allocation spotted by clang.

Eventually we will move the regress.zsh tests into this framework.

lbximg: Make libpng a soft dependency.

Now that we have Netpbm output formats which do not depend on
any external libraries, we can make it possible to build lbximg
without libpng.  Adjust the build system so that lbximg is built
unconditionally and PNG output can be selectively disabled.

lbximg: Add support for Netpbm output.

These netpbm formats are extremely simple image formats which can
be written with a very small amount of code.  Implement the ASCII
PBM and PPM formats for eventual use in test scripts, and PAM as a
fully-featured format so that we can make libpng optional.

liblbx: Don't scale palette values internally.

Output the 18-bit palette values unadulterated, leaving it up to the
users to do conversion if necessary.  Push the conversion down into the
consumers (currently PNG output and the GTK GUI) where a 24-bit palette
is required.

While we're at it, adjust the conversion to scale to the full output
range of [0, 255].  This is a slight change to the output, so all the
image hashes need to be adjusted in the regression test.

lbximg: Fix printf format string in identify mode.

One of the flag strings would never be printed because there was
simply not enough conversion specifiers in this format string.  Fix
that up, and also adjust the length modifiers to match the actual
passed integer types.

lbximg: Push no-palette mode down into the PNG writer.

This allows the writer to select the most appropriate output format to
support this mode.  Switch the PNG writer to emit true grayscale images
in this case.

liblbx: Properly report errors in lbx_(img_)fopen.

The lbx_fopen and lbx_img_fopen functions were not properly pushing an
error code when they failed, leading to extremely helpful messages of
the form:

  lbximg: failed to open image: Success.

It further turns out that pushing negative errors (i.e., -errno) was
documented but never actually tested or used, so fix that up, too.
It is likely that many other sites are lacking appropriate calls to
lbx_error_raise, which can be added later.

lbximg: Add a new PNG writing routine.

This splits out the PNG output from the rest of the lbximg processing,
so that it will be easier to add additional output formats (in par-
ticular, it would be nice to have some sort of text-based format to
facilitate automated testing).

Take this opportunity to vastly improve the error handling.

tools: Add some smarter error printing routines.

These ones can fill in strerror(errno) automatically.

tools: Move progname handling into a common source file.

Less code duplication is nice.

build: Remove use of per-target CFLAGS.

These are not necessary, and result in ugly object file names for no
reason.

build: clean up XML -> C conversion.

Use a suffix rule so the rule avoids hardcoding names and other details.

build: Flatten the build system.

Move everything into a single Makefile.am so that it's all in one place.

docs: Add a more realistic statement about development pace.

I don't think "under heavy development" even begins to resemble reality.

build: Integrate Gnulib non-recursively using fix-gnulib.

gnulib: Update to latest version.

build: Fixes for newer tools.

Some trivial updates to fix the build with latest Automake.  Drop
support for Automake versions < 1.11.

doc: Update moo2-data.

liblbx: Remove now-redundant fields from lbx_imginfo.

Ideally I'd like to kill this structure completely, but there's still
some non-trivial logic in it.  So leave those fields for now.

liblbx: Kill the LBXimg typedef.

liblbx: Start conversion of LBXimg to public/private structs.

liblbx: Kill the LBX typedef.

More stuff rendered redundant.

liblbx: Kill lbx_numfiles.

This function is now redundant.

liblbx: Start conversion to public/private structure bits.

This sort of interface is really a lot nicer, and avoids stupid
accessor functions.

build: Reset pkg_failed before checking pkg-config.

If this is not done, the first pkg-config check to fail will cause all
subsequent pkg-config checks to "fail".

liblbx: Namespace cleanup.

lbxgui: Fix an error handling regression.

If the framebuffer allocation fails, it still gets passed to
render_to_pixbuf.  This causes an assertion failure.

lbxgui: Implement frame dropping for animation.

Currently, if it takes longer than 1/15th of a second to draw a frame,
the next frame will be queued before it has a chance to finish, which
then takes longer than 1/15th of a second to draw, so the next one gets
queued, ad infinitum.

We do two things to solve this problem.  First, the timer runs at a
low priority so that a redraw doesn't get queued while a redraw is in
progress.  Second, the timer skips queueing any frames that have missed
their time.

liblbx: Use unconditional includes of <config.h>.

liblbx: Implement improved error reporting mechanism.

We now use a FIFO-of-errors approach (similar to libltdl) as opposed to
an errno-esque approach.

liblbx: Replace feof call with lbx_file_ops.

This feof call was missed when converting this function.

liblbx: Reset EOF flag after a successful seek on a pipe.

This will probably never happen in practice.

liblbx: Rename lbx_stat to lbx_file_stat.

liblbx: Rename LBX_IMG to LBXimg.

This makes it consistent with LBXfile.

build: Implement a better GTK+ detection macro.

This macro uses the same ideas behind the libpng detection, does not
suffer from the POSIXLY_CORRECT nonsense, doesn't duplicate checks
for pkg-config, supports caching, and properly fails when it should.

build: Split out core logic from libpng detection.

This allows it to be used for other libraries.  This new method also
better handles the various mechanisms for setting cflags/libs.

lbxgui: Fix pointer type error.

The XBM format is an array of unsigned char, yet for some reason
unknown to me, the gdk_bitmap_create_from_data function wants a
char pointer.  Add the appropriate cast.

lbxgui: Compile in the glade UI description.

I don't think it makes any sense to try and jump through hoops to find
this data at runtime, given that the data and the code are so intimately
connected.

license: Fix copyright years in source files.

lbxgui: Add copyright notices.

doc: Update man pages to reflect reality.

lbximg: Clean up option list.

Despite it being very useful, --nopalette did not have a single
character option.  While we're at it, use --no-palette for consistency
with GNU tools.

lbximg: Use the new lbximg_fopen interface.

liblbx: Update lbximg_fopen to work like lbx_fopen.

lbximg: Fix improper free.

We were freeing the row buffer twice on the error path, and not at all
on the success path.

lbximg: Zero frame bitmap before use.

lbxgui: Add a toggle between patterned and solid background.

The patterned background makes it hard to see images that consist mostly
of transparent pixels.

lbxgui: Add a scrolled window around the image viewport.

This solves a weird problem with the canvas resizing when one clears the
image selection.

lbxgui: Simplify image loading.

This also fixes an oversight whereby the framebuffer was re-allocated
every time the image is re-rendered.

lbxgui: Split frame rendering into a separate source file.

lbxgui: Clean up function names and linkage.

lbxgui: Add interface to clear a combo box.

doc: Update moo2-data.

liblbx: Add support for "chunked" images.

It appears that the frame count fields aren't actually two 16-bit
fields, but rather four 8-bit fields.  The images in monster.lbx
appear to have "chunks" of animation and what was previously thought to
be the most significant byte of the lead-in now seems to be the length
of a "chunk".

These images are much like any other, except that the first frame of
every chunk is drawn on a clean slate.  Examples are found in
monster.lbx.

liblbx: Add support for "raw" LBX images.

There is a new image format in town.  If a particular flag bit is set,
then there are no row headers and data for every pixel of a frame is
simply stored in row-major order.  An example of such an image is
starbg.lbx.009, as well as several others in the same archive.

Don't you just *love* the designers of this format?

lbxgui: Make an effort to not leak resources.

lbxgui: Make transparent pixels more obvious.

lbxgui: Don't use as much horizontal space in GUI.

build: Update project dependencies.

doc: Update moo2-data.

liblbx: Clear the image mask when backtracking frames.

Without this, undrawn pixels in the first frame will have the pixel
values from the previously drawn frame.

doc: Update moo2-data.

liblbx: Add new flag.

Building images have a previously-unknown flag, 0x0800.  The images have
a part that looks like a shadow, but it does not have any palette entry.
I suspect that this flag is related to that shadow.

lbxgui: Add support for override palettes.

lbxgui: Add support for external palettes.

liblbx: Use lbx_file_ops for lbximg_loadpalette.

lbxgui: Recognise *.LBX as an LBX archive.

The files on the Moo2 CD show up in ALL CAPITALS, so it's useful to
match them with the file filter.

lbxgui: Add support for animating images.

lbxgui: Allow frame to be specified.

liblbx: Set the file offset in lbx_file_seek.

This function previously would do the underlying seek correctly, but not
actually record what it did.  This breaks lots of things, fix it.

lbxgui: Add initial support for decoding and displaying images.

liblbx: Add a field to lbx_colour indicating which entries are used.

An LBX palette might not contain values for all 256 entries.  Currently,
there is no way to tell which entries were assigned when the palette is
retrieved from an image.  This patch adds an extra marker which can be
used to skip over unused palette entries.

lbximg: Properly convert lbx_colour to png_color.

Now lbximg does not depend on these two libraries using the same
underlying palette representation.

liblbx: Add file operations for handling LBX archive members.

This allows, for example, the image handling code to read directly from
the archive.

lbxgui: New tool.