Trivial manual fixes.

For literal strings, we need to use \- instead of just - as some troff
processors turn plain "-" into "proper" unicode hyphens (as opposed to
hyphen-minus), which breaks copy+paste and searching.

Ensure that .Os follows .Dt in the mdoc prologue as apparently not doing
this results in broken manpage headings with groff 1.23.

Replace gnulib patch with new common helper macro.

Using the new DX_PATCH_GNULIB macro allows m4 to implement the
conditional dependency patches that were previously being done
in bootstrap.

Stop using gnulib's flexmember module.

The only thing we're actually using from this module is provided
directly by Autoconf, via AC_C_FLEXIBLE_ARRAY_MEMBER, so we can
just use that macro instead.

Avoid Gnulib std-gnu11 module.

It has come to my attention that this module rewrites AC_PROG_CC in
a way that actually breaks Automake's AM_PROG_CC_C_O functionality.
This results in the "compile" script not being included or used in
packages bootstrapped with Autoconf 2.69.

With later versions of Autoconf things work because this module
doesn't touch things and thus disables itself.

I don't care about building with C11 one way or the other.  Let's
just skip the module.

Update gitmodule URLs.

Use gnulib flexmember module.

For increased portability to older C compilers.

Work around shell redirection bug on HP-UX /bin/sh.

On HP-UX /bin/sh, when "read" is used in a subshell with redirections,
it seems that some of the input gets dropped on the floor.  For example:

  % cat >test.txt <<'EOF'
  hello
  world
EOF
  % ( exec 3<test.txt; read a <&3; read b <&3; echo $a $b )
  hello

In TEST_CHECK_PAM_MD5, we hit this bug because AT_CHECK runs its
commands in a subshell, and as a result some of the image data is
lost after reading the header and verification fails.

Work around the issue by performing only a single AT_CHECK with a
single redirection.

Use AC_CONFIG_HEADERS rather than AC_CONFIG_HEADER.

The former name has been supported since approximately forever ago and
the latter form is now formally deprecated (with a warning) in recent
versions of Autoconf.

Improve lbx_colour structure layout.

There is no point in using bit-fields for the components of this
structure.  The structure will never be smaller than 4 bytes so
there is no space savings compared to just using four unsigned
char members.  This avoids the compiler needing to do any kind
of bit shuffling.

In the typical case where such a structure has no padding, this
means the layout of these members will now exactly match the PAM
format, enabling a modest speedup.

lbximg: Improve performance of PAM output.

The PAM output currently calls fwrite for every pixel, which amounts to
about half a million fwrite calls to output a full screen image.  For a
fullscreen animation like logo.lbx.001, the majority of the execution
time is spent in fwrite.

Let's reduce this a lot by formatting and outputting an entire row at
a time, which almost completely eliminates fwrite from the profile.

lbximg: Avoid %zu printf format.

Some older library implementations don't support %zu conversions.
Since the use in lbximg is entirely pointless, this is just
gratuitious incompatibility.

Back out incomplete i18n support in the library.

We are a long way from actually having working i18n, so all this
is doing right now is adding unneeeded build dependencies.

Improve portability of testcase MD5 generation.

GNU md5sum is not always available, but many systems have some form of
MD5 utility available in the base install.  Use the new DX_PROG_MD5
macro to locate such a utility and adjust the test suite to use it.

This will enable the original-data testcases to run on more systems.

lbximg: Simplify pnm text output.

The ASCII output routine here is pointlessly complex, we are only
printing very short lines and a fixed length buffer will do just
fine.

Moreover, let's just optimize it out completely in the case where
the C character set is an ASCII superset and assertions are disabled.

Use (size_t)-1 instead of SIZE_MAX.

We do not require the special characteristics of SIZE_MAX (that it be
suitable for use in #if directives), and older library implementations
do not define it.  Outside of the preprocessor, (size_t)-1 is exactly
equivalent so let's just use that instead.

Add missing <config.h> include to fops.c.

Don't use [static n] in function prototypes.

This modern C feature is mostly useless and introduces compatibility
problems for no good reason.

Port all test cases to Autotest.

I can't say I'm a huge fan of the Automake test harness.  All my other
projects use Autotest, let's convert this one to use it too.

Convert tools to use option generator script from dxcommon.

This generator makes it simple to maintain long and short option lists,
as well as provide an actually useful --help output.

Use shared code from dxcommon.

The needed library tests and integer packing code have been part of
dxcommon since forever ago, probably adapted originally from this very
package.  No point carrying around redundant and outdated copies.

Update gnulib/dxcommon to lastest bits.

liblbx: Correct EOF handling in lbx_file_read.

We should never see an EOF from the underlying I/O when reading archive
members, because we setup the read request to only return bytes that are
supposed to be there.

So if EOF happens, flag this as an error and do *not* set the eof flag
in the state structure.  Also fixup lbxtool to properly handle this.

tests: Add test for truncated archives.

If a real EOF is encountered while reading an archive member this
will not be properly reported as a read error.  So we'll never notice
if extracted files are missing a chunk because the archive is truncated.

Add a test which exposes this behaviour.

liblbx: Remove use of SEEK_CUR in the image reader.

Other than this one seek, all other seeks used by the library are of the
SEEK_SET type.  Only requiring SEEK_SET will simplify implementations of
lbx_file_ops quite a lot, so let's get rid of this one.

The whole point of this seek is to skip just one byte — a normal read
works just as well.

liblbx: Remove tell function from lbx_file_ops.

There is no longer any user of this function, so we don't really need
it in the lbx_file_ops structure.

liblbx: Remove declaration of lbx_mopen.

This function was removed a long time ago, but its declaration was
never deleted from the header file.  Better late than never.

liblbx: Add a hard check for image leadin versus frame count.

Our "official" image spec says that the leadin is a frame number, so
it does not make sense for the leadin to be greater than or equal to
the frame count.

Convert the existing "soft" check into a hard failure.

liblbx: Kill lbx_img_getinfo

There is no longer any reason to use this function.  Its removal has
been a long time coming; it is time for it to go.

liblbx: Clean up embedded palette handling

We don't need to read the embedded palette information with the rest of
the header, we can defer that to much later when the palette is actually
needed.  This makes more sense conceptually, simplifies the code a bit,
and saves some bytes in the image state structure.

Since we're basically rewriting lbx_img_getpalette anyway, take this
opportunity to give it a better interface.  It now returns the number
of embedded palette entries, allowing us to eliminate the palette size
feature in lbx_img_getinfo.

liblbx: Kill the old LBX image API.

Now that both users have been ported we can get rid of this cruft.

lbxgui: Render LBX directly to Cairo surfaces.

This replaces the GDK pixbuf with a Cairo image surface, and updates
the render path to the new liblbx API.

lbxgui: Use Cairo for drawing.

The GDK stuff is deprecated so let's convert all the GDK drawing bits to
Cairo.  Unlike GDK, Cairo does not include an XBM loader so we have to
implement that from scratch.

Keep the GDK pixbuf bits around for now.

liblbx: Fix uninitialized value warning in pipe_seek.

If whence is invalid then we will proceed with an uninitialized value.
However, unlike the similar problem in lbx_file_seek, pipe_seek is an
internal function so this scenario can Never Happen™.

Add an assertion to silence the warning.

liblbx: Always return a value in file_seek.

This function did not return any value on success so it was only by
chance that it seemed to work.

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.