@@@ BROKEN wip

@@@ more mess

@@@ so much mess

@@@ adjust bench timings

@@@ sink bench

@@@ tvec error return

@@@ wip type definitions in manpage synopses

struct/hash.3: Zap spurious space before pointer argument name.

sys/mdup.3: Add missing `.nf'...`.fi' around the synopsis.

codec/base64.3: Supply missing Oxford comma.

*/*.3: Align arguments properly to the right of the opening `('.

This involves a little stunt roff, but we'll manage somehow.

test/testrig.3: Correct return type of `test_do' in the synopsis.

The later text explains what the function returns, so it clearly is
meant to have a meaningful return value, so the synopsis was just always
wrong.

codec/url.3: Add missing documentation for the `URLF_SEMI' flag.

codec/url.h: Delete stray blank line.

@@@ all the mess ever

utils/control.h: Allow statement block without terminating semicolon.

@@@ major overhaul, new primitives

@@@ better tests

@@@ wip@g

@@@ control wip@cb

codec/tests.at: Delete stray space before tab.

utils/macros.h: Add `MUST_CHECK', so return codes aren't ignored.

utils/macros.3: Create manpage for the `CLANG_WARNING' macro.

.gitignore: Absorb `t/.gitignore'.

The ignored files are mostly global machinery which happens to be tucked
away in `t/'.

utils/macros.h (STATIC_ASSERT): Boolean-canonify the expression.

6.7.10p3 insists that the expression be an integer constant expression,
but this isn't in a constraint, so the behaviour is undefined if it's
not an integer constant expression.  Apply `!!' to force pointerish
expressions to integer type.

*headdesk*

configure.ac: Provide a non-empty default manpage extension.

This has been a bug forever, but there's been a corresponding bug
in the `mdw_MANEXT' macro forever which ignores the supplied default
and uses `mLib' instead, so nobody noticed.  Until I fixed the
other bug and my build broke.

vars.am: Quote `$(manext)', or things break if it's empty.

It's not usually empty because it's defaulted in `configure.ac', but
it can be overridden to be empty.  Also, there might be bugs.

.gitignore: Organize and clean up.

Divide into sections and add headings; add leading `/' to patterns which
are only expected to match in the same directory as the `.gitignore'
file; and add trailing `/' to patterns which should match directories.

utils/compiler.h: Prefer `ifdef' rather than `if defined'.

struct/buf.h: Muffle annoying new GCC warning.

This one seems like more trouble than it's worth.  Just squash it in
GCC versions recent enough to know about the warning.

Release 2.5.0.

utils/versioncmp.h: Add a wrapper macro for `versioncmp'.

Similar to the macros introduced for `strcmp' end friends in
3618811496a6d131fd4bffa19e262c521d39e819.

struct/assoc.[ch]: Fix some commentary typos.

*/t/*.py: Fix the various testing scripts for Python 3 compatibility.

  * Put parentheses around `print' arguments.  (This is most of the
    churn.)

  * Cope with `xrange' being renamed to `range' in Python 3.

  * Cope with `MAP.keys()' not returning a list any more in Python 3.

  * Cope with integer division not returning an integer any more in
    Python 3.  (Use a shift, because that's not changed.)

utils/macros.h: Add <ctype.h> and `foocmp' helper macros.

Both of these have been a nuisance for years.  Do something about it.

The command

git grep -En '\<(is|to)[a-z]+\(|\<(str|mem)[a-z]*cmp\(' -- '*.[ch]'

finds code which it might be good to change.

The hairy Emacs command

(query-replace-regexp
 "\\<\\(\\(str\\|mem\\)\\sw*cmp\\)(\\([^,]*\\), \\([^)]*\\)) \\(==\\|!=\\|<\\|<=\\|>\\|>=\\) 0"
 '(replace-eval-replacement
   . (concat (replace-quote (upcase (match-string 1)))
     "(\\3, \\5, \\4)"))
 nil
 (if (and transient-mark-mode mark-active) (region-beginning))
 (if (and transient-mark-mode mark-active) (region-end))
 nil)

will convert easy cases of `foocmp' calls, but hard ones have to be done
by hand.

Merge branch '2.4.x'

* 2.4.x:
  Release 2.4.2.
  sel/bres-adns.c: Fix incorrect array bound.

Release 2.4.2.

sel/bres-adns.c: Fix incorrect array bound.

GCC helpfully points out that `N(av)' does nothing very useful.  In
fact, I expect that it's almost always zero, which means that we don't
end up assembling addresses correctly.  And we never have done.

How embarrassing.

(cherry picked from commit 5086ffd7745337f5bee3c5c771cfcf1da71b03c3)

sel/bres-adns.c: Fix incorrect array bound.

GCC helpfully points out that `N(av)' does nothing very useful.  In
fact, I expect that it's almost always zero, which means that we don't
end up assembling addresses correctly.  And we never have done.

How embarrassing.

Merge branch '2.4.x'

* 2.4.x:
  struct/buf.c (buf_getmem*): Add an explicit `BENSURE'.

struct/buf.c (buf_getmem*): Add an explicit `BENSURE'.

Otherwise we fail when processing a 64-bit length on a 32-bit platform:
an overlong 64-bit length gets truncated to a possibly-acceptable 32-bit
length in the implicit conversion to `size_t' in the call to `buf_get'.
Using `BENSURE' directly ensures that the comparison gets done with the
correct integer type.

Merge branch '2.4.x'

* 2.4.x:
  utils/str.3: Fix typeface for mathematics.
  sys/tests.at: Add full stops after header comments.
  codec/codec.3: Fix spacing after ellipsis.
  utils/t/bits-testgen.py: Set the `SEED' from the command-line correctly.
  utils/t/bits-testgen.py: Make `arg' return arguments in the right order.

debian/changelog: Prepare for the next version.

utils/str.3: Fix typeface for mathematics.

sys/tests.at: Add full stops after header comments.

codec/codec.3: Fix spacing after ellipsis.

utils/t/bits-testgen.py: Set the `SEED' from the command-line correctly.

This never stood a chance before, because `arg' always returned the
script path.  Now that's fixed, instead `int' complains that the hex
number it's being given isn't acceptable.  I can't fix this by
explicitly passing a radix of 0 because that doesn't allow an existing
integer object.  So we have this circumlocution.

utils/t/bits-testgen.py: Make `arg' return arguments in the right order.

Obviously I never watched this run before.

debian/: Bump to Debhelper 10.

Release 2.4.1.

debian/: Ship correct symbols files.

debian/.gitignore: Ignore generated `lib2-adns.install' file.

mem/track.[ch]: Delete ancient debris.

Apparently these got forgotten in the source-tree reorganization back in
2.2.0.  They've been languishing in the version-control system, but
haven't been built or distributed since then -- and nothing has
noticed.  Also, they've never been documented, and don't do anything
very useful anyway.

So just delete them and let history forget they ever existed.

hash/unihash.c: Replace a dynamic assertion with a static one.

utils/macros.h: Introduce a `STATIC_ASSERT' macro.

Release 2.4.0.

struct/buf.c: Muffle an annoying Clang warning about a pointless assertion.

Of course, since this code is macro-generated, it's not at all
pointless.

utils/macros.h: Sprinkle `__...__' over GCC (and Clang) attribute names.

I suspect that things like `noreturn' might already be macros.  Though,
honestly, this header file is not a shining example of namespace hygiene
in countless other ways.

hash/Makefile.am, hash/crc32.h: Make the CRC table be `const'.

Yay.  That's an extra kilobyte of sharable data.

hash/crc-mktab.c, hash/unihash-mkstatic.c: Add `const' option.

Add `-C' option to produce `const' tables.

I'd probably like this to be the default, at least for `crc-mktab', but
currently it isn't for (a) compatibility and (b) consistency between the
two.

Release 2.3.3.1.

utils/macros.h: Define missing fallback for `NORETURN'.

Release 2.3.3.

sys/fdpass.c: Allocate extra cmsg space to hack around a Qemu bug.

*shakes head sadly*

sys/t/fdpass-test.c: Add a simple test for file-descriptor passing.

sys/fdpass.c: Fix expected size of descriptor message.

This typo breaks 64-bit targets, and I'd never noticed because of
inadequate testing.

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.