Makefile.am: Link the main library with $(MLIB_LIBS).

Otherwise we miss out `-ladns' and there is sadness.  I have no idea why
this has only become a problem just now.

sys/t/mdup-test.c: Cope with Cygwin setting top inode-number bits.

Maybe it's because the files have been deleted, but in any event it's
rather annoying: Cygwin seems to set high-numbered bits in the result
inode numbers, which causes spurious test failures.  Compensate by
masking the numbers.

sys/t/mdup-test.c: Introduce a `-v' option to help debug failures.

In particular, just reporting `ino FD[I] wrong' isn't very helpful.  But
adding inode numbers won't help unless we've already printed the initial
numbers, and that would add unnecessary verbosity in the usual case.

sys/t/mdup-test.c: Cope with the idea of arguments which aren't fd pairs.

The code used to assume a fixed relationship between command-line
argument indices and indices in the `mdup_fd' table.  Fix this by
introducing separate index variables in the argument-parsing loop, and
set the table length correctly afterwards.

sys/t/mdup-test.c: Improve the layout.

utils/macros.h (N): Properly parenthesize the argument.

WTF?  How did I write this wrong?

Fortunately, I don't think it actually matters.  The argument must have
array type.  Postfix operators are higher precedence, so that's not a
problem; prefix operators are in the right place already; and binary
operators don't yield array types.

build: Configure `pkg-config' correctly for static linking.

Release 2.3.2.

codec/url.c (encode): Fix bungled cast in previous refactoring.

Release 2.3.1.

Merge branch 'master' of git.distorted.org.uk:~mdw/publish/public-git/mLib

* 'master' of git.distorted.org.uk:~mdw/publish/public-git/mLib:
  Release 2.2.5.
  sel/ident.c: Don't close the descriptor twice on immediate connect failure.

Release 2.3.0.

codec/url.c (encode): Rewrite more compactly and perspicuously.

The condition in the `default' case was a bit gnarly: rewrite it as a
ladder of if/else.  Take the opportunity to compress the code, because
the individual case labels really don't deserve all of that vertical space.

Release 2.2.5.

sel/ident.c: Don't close the descriptor twice on immediate connect failure.

Confusingly, `conn_init' has already done that.

codec/url.c: Always encode whitespace characters.

Vertical whitespace is obviously bad, so this is a longstanding bug; but
all whitespace should really be escaped.

utils/bits.h: Support compiler magic for unaligned loads and stores.

Introduce `RAWw' macros to access the raw memory, and implement `LOADwe'
and `STOREwe' in terms of these and `eTOHw'/`HTOew'.

These are remarkably tricky because GCC (I think mistakenly) thinks that
type-based aliasing is applicable when in fact the other type in
question is `unsigned char', which is known to be able to alias
anything.  Hit things with the `may_alias' hammer and hope they quieten
down.

utils/bits.h: Add macros for swapping endianness in place.

Many processors and compilers provide handy instructions for doing this,
and it's a handy primitive for loading and storing.

The HTOLw etc. macros turn out to be remarkably slippery to define.
It's unfortunately nonsensical to speak about an integer value having an
endianness.

utils/bits.h: Reformat `LOAD64_[BL]_' for consistency.

No functional change, just whitespace.

utils/bits.h: Refactor the load/store macros.

  * Each macro is now only defined conditionally.  This allows earlier
    parts of the header file to define a compiler-specific
    implementation, with the generic definitions filling in the blanks.

  * Instead of writing the byte manipulations out longhand, build each
    one out of smaller pieces.  This has two benefits: firstly, the new
    definitions are smaller and easier to understand; and, secondly, if
    there's a compiler-specific trick for, say 16-bit loads, then 32-bit
    loads can win some of the benefit.

utils/bits.3: Basically rewrite it, filling in the missing things.

Oh, dear.  This manpage has been neglected for years.

.skelrc: Fix for Skel's GPL rubric reorganization.

utils/: Split compiler-version macros into a separate header.

This way we can use them in other headers without the namespace mess of
`macros.h'.

Also, actually document `CLANG_VERSION_P'.

utils/macros.3: Document `CLANG_WARNING'.

Added back in ff3d3f01025b881c7cadf093b0f65309c35f2c53.

.mailmap: Add file to fix CVS-ish names in history.

Release 2.2.4.

Revert "debian: Update for Debhelper 10."

I think I might still have to build for wheezy.
This reverts commit 6d061820334c97ed4f3266d7130e089de6b21b55.

struct/buf.3: Fix synopsis for `BSTEP'.

debian/copyright: Fix copyright dates.

debian: Update for Debhelper 10.

debian: Multi-Arch support.

debian/copyright: Convert to machine-readable format.

debian/control: Use ${binary:Version} instead of ${Source-Version}.

vars.am: Tweak `silent-rules' machinery.

Since Automake 1.11, the advice for setting up custom silent-rules
recipes has changed, so use the new machinery.

Also, I'm no longer mainly working on wheezy, and Automake has made the
operation field two spaces wider while I wasn't looking, so make the
output line up properly.

This means that mLib now requires Automake 1.11.2 or later to build from
the Git tree.

utils/macros.h: Add support for Clang-specific hacks.

These are mostly the same as GCC, but (fortunately!) Clang's diagnostics
handling is less broken than GCC's, so we can do the same job much more
simply.

utils/macros.h: Refactor the GCC-ish compiler support.

Fully define `GCC_VERSION_P' early, and then use it without any other
guard to define the GCC-specific hacks.  This way, it's easy to support
other compilers which behave similarly, by defining an appropriate
`FOO_VERSION_P' macro, and then adding appropriate extra alternatives to
the individual guards.

No functional changes.

sel/sig.c: Use `DISCARD' to ignore errors from the signal-pipe `write'.

Annoyingly, Clang complains about the empty consequent being on one line
otherwise.  Still, this is what `DISCARD' was made for.

trace/trace.3: Fix misformatting.

vars.am: Some reformatting.

sys/tests.at: Actually make sure the `mdup' tests work.

Oops.  It's not actually been testing anything.  You can tell because it
tried to run the wrong program.

Fortunately, once this is fixed, the test does seem to run OK.

sys/tests.at: Use the correct name for the `mdup' test program.

configure.ac: Look in the right place for `socklen_t'.

The old-fashioned version of `AC_CHECK_TYPE' doesn't look in
<sys/socket.h>, which is the proper place.  Android doesn't appear to
define the type in the usual places where Autoconf looks.

So use the modern version of the check and look in <sys/socket.h>
explicitly.  This seems to fix the build on Android.

Release 2.2.3.

build: Abolish the `$e', `$o' and `$t' variables.

I don't think anything ever used `$o'.  This project didn't use `$t'.
And use of `$e' gets in the way of Automake's magic handling of
executable suffixes under Cygwin.

Makefile.am: Say `-no-undefined' to libtool for the sake of Cygwin.

struct/dstr-putf.c: Remove apparently redundant inclusion of <math.h>.

Just to make it clear that we don't depend on the floating-point library
in any important way.

utils/t/exc-test.c: Print null pointers in a consistent way.

Cygwin prints `0x0' instead of `(nil)'.  Fortunately, this is the only
pointer value we can reliably check for, so handle this ourselves.

sel/bres.c: Don't capture `h->h_name' in non-`const' pointer.

It turns out to be `const' under Cygwin.  In fact, we don't need to
capture it at all, so restructure the logic so we don't bother trying.

build: Cope with the `subdir-objects' world Automake wants us to live in.

Essentially, the Automake developers want to put the objects for
`PATH/TO/FILE.c' in `PATH/TO/FILE.o'.  This is wrongheaded, but we don't
seem to get much choice.  Unfortunately, it's also buggered.

This causes trouble for our precomputed source files.  The obvious
trouble happens if the source file we reference is explicitly in the
source tree, so we'll need to refer to the files differently in
`mumble_SOURCES' lines and the machinery which makes the generates the
files.  The obvious answer would be to introduce two variables for
referring to the precomptations tree.  This is where Automake's bugs
start to really bite.

The main problem is with Automake's automatic dependency-tracking
machinery.  For each object `FILE.o' which is going to be built, it
wants to make a `.deps/FILE.Po' file to track the detected dependencies.
Furthermore, the generated makefiles get unhappy if these files don't
already exist, so there's magic hung off the side of `config.status' to
make them.

This would be great, but the Automake machinery doesn't actually work
properly.  If you refer to a source file via a variable reference,
something like `$(things)/file.c', then Automake's `config.status' magic
creates a dependency-tracking file which is literally named
`.deps/$(things)/file.Po', and then the makefile gets upset when it
tries to include `$(things)/.deps/file.Po'.

So we have to write explicit relative paths to precomputed source file
names in `nodist_mumble_SOURCES' lists (because we make our own
arrangements for distributing them).

Merge branch 'master' of git.distorted.org.uk:~mdw/publish/public-git/mLib

* 'master' of git.distorted.org.uk:~mdw/publish/public-git/mLib:
  Release 2.2.2.2.
  debian/source/format: Apparently these are now mandatory.
  debian/control: Add Build-Depends on python for improved testing.

Release 2.2.2.2.

debian/source/format: Apparently these are now mandatory.

debian/control: Add Build-Depends on python for improved testing.

struct/dstr-putf.c: Fix typo in commentary.

codec/codec.3: Minor formatting fix.

Release 2.2.2.1.

struct/dstr-putf.c: Don't leak the argument and spec vectors.

Release 2.2.2.

struct/dstr-putf.c: Different way to represent type lengths; new types.

Support the full menagerie of C99 and POSIX types.

struct/dstr-putf.c (dstr_vputf): Rewrite to support `%n$...' specs.

The internals are generally just better now, and it should be easier to
support other interesting things.

sel/bres.c (zap): Don't scramble the freelist when a query is aborted.

If a query is aborted after it's been committed to a server process,
then we kill the server and return its control block to the freelist.
Unfortunately, the function which does this, `zap', unconditionally
tries to unlink the control block from its current position in the
freelist; but it wasn't actually there before.  The result is that
another server control block might be linked back into the freelist.
Attaching a second client to it while it's already in use fails when
`attach' tries to remove the server's idle timer, which isn't active: at
this point, we get a segfault.

struct/dstr-putf.c: Don't  segfault on `*' width/precision indicators.

Must have always been broken.  Strange: I thought I'd tested that.

codec: New flag to ignore linear whitespace.

Why was this missed the first time around?  Set this by default in the
`bincode' program.

struct/tests.at: Skip tests if we don't have a working Python.

Release 2.2.1.

sys/mdup-test.sh: Remove obsolete test script.

utils/macros.h: Support for compiler-specific annotations.

Add some macros for attaching attributes to functions and so on, for
controlling warnings, and for marking variables as being unused or
ignorable.  Also use them for marking exported functions.

debian: Drop CDBS in favour of using Debhelper directly.

General build system spring-cleaning.

Most of this comes from the shiny new build system for Catacomb.

configure.ac: Some random formatting tweaks.

struct/dstr.[ch3] (dstr_putc): Accept an `int' argument.

This is a rather annoying problem.  It's useful to store binary data in
`dstr' objects, and, indeed, characters encoded (as <stdio.h> does) as
`unsigned char' values in an `int' object; but the `char' type of the
argument requires an implicit conversion, which may cause undefined
behaviour, and raises warnings if called with an out-of-range constant
value.

sel/bres.h: Fix include guard macro name.

sel/bres.c: Rewrite list hacking to avoid strict-aliasing badness.

The circular list stuff was quite pretty but involved some really
unpleasant casting which modern GCC (quite properly) complains about
vociferously.

Replace it with more traditional doubly-linked-list hacking with
null-pointer sentinels, with the slightly nasty pointer swizzling tucked
away in useful macros.  Some of the uses of these macros (e.g.,
unlinking the first or last item in a list) could be made more efficient
by using special-case versions, but it doesn't seem worthwhile.

utils/t/exc-test.c: Fix `%p' format/argument type mismatch.

struct/t/: Use better format for printing `size_t values.

configure.ac, sel/: Check for and use `socklen_t'.

Rather than using `size_t' and hoping for the best.

hash/crc-mktab.c: Fix severe breakage on 64-bit systems.

The output table was completely screwed.

sys/mdup.[ch]: Fix licensing notice to refer to LGPL2.

This is debris from when the original version of mdup was written
out-of-tree, and had the default GPL slapped on it.

For clarity: the `mdup' code in mLib has in fact always been released
under the terms of the GNU Library General Public License, version 2 or
(at your option) any later version.  It's just that the notice was
wrong.

build: Don't make libraries for precomputation utilities on demand.

If `make' is in parallel mode, it will fire off multiple simultaneous
builds of the same library, and they interfere with each other.
Instead, just arrange that the libraries are made when we try to
distribute things.

vars.am: Use `AM_CPPFLAGS' for the includes list.

This frees up plain `CPPFLAGS' for users.

codec/{base32,hex}.h: Include `codec.h'.

Brown paper-bag.  Release 2.2.0.1.

Release 2.2.0.  Yay.

sel/sig.c: Discard return value without provoking other warnings.

sig.c: Ignore return code from write(2) to pipe.

We get warnings nowadays about this.  These are annoying.  The only
thing that can go wrong is that the pipe is already full, so the write
will block.  But if that happens, our job is already done: the select(2)
loop will certainly wake up soon because the pipe is full of stuff to be
read.  So ignoring the error here is harmless.

Insert a stupid bodge to shut the compiler up.

Build system: Use Automake 1.11 `silent-rules'.

Infrastructure: Switch testing over to Autotest.

Currently mLib testing is a hopeless mess.  Maybe this will help sort it
out.

Some minor fixing was applied to the data-structure tests:

  * assoc and sym needed fixing to avoid corrupting their internal
    table-size count.  There are now additional assertions that this
    internal count matches the hash table's own count.

  * darray needed fixing to report empty-array on `first' and `last'
    requests.

codec, baseconv: Cleanup of the various binary encoding functions.

The new codec interface provides an object-based interface to binary
encodings.

The baseconv code is a new unified implementation of Base64, Base32 and
hex encoding and decoding, built from a digit-at-a-time base conversion
core.  It implements a number of variations on the basic encodings,
including the URL-and-filename-safe variant.

I've adapted the current tests to use the new test program, but that's
not long for this world.

precomp: New directory for precomputed files.

The `precomp' directory in the distribution tarball contains pre-built
tables, for CRC and unihash computations.  Distributing the tables means
that the library can be built by cross-compilers.  Indeed, we don't even
try to generate the tables if cross compilation is detected.

The precomp directory shouldn't exist in revision-control, but it's
populated during the build process if necessary, and it's distributed in
release tarballs.

Infrastructure: Split the files into subdirectories.

This makes the file tree rather easier to work with.

Headers: Guard inclusion of mLib headers.

A number of headers include other mLib headers without #ifndef guards.
This will slow down compilation slightly on some (less sensible)
compilers.

Infrastructure: Strip away crufty CVS $Id$ tags.

debian: Prepare stuff for new version.

bres: Use mdup to duplicate file descriptors for the child.

mdup.h: Remove spurious duplicate summary line from comment.

Release version 2.1.1.

buf.c: Step over terminating null byte.

buf.h: Spurious `\' prevents declaration of `buf_putstr*'.

Makefile.am: Use $(mkdir_p) instead of $(mkinstalldirs).

It works, for one thing.  For some reason, mkinstalldirs is set to
$(SHELL) $(install_sh), and install_sh is set to $(SHELL) .../install_sh.
Can you see the problem here?

Commit as 2.1.0.

Merge branch 'work'

* work:
  mdup: New unit for juggling file descriptors.
  align: Add trivial manpage.
  Manpages: Move manpages (back?) into the top-level directory.