In particular, gcc warns about cast from int to pointer.

Instead, directly control write().

%.* => unsigned, not size_t.

%zu for size_t printf.

Make sure we use 1UL to bitshift, not 1.

printf's %.* means an unsigned arg, not a size_t.

check_free_list returns a length, not bool.

Particularly useful for reproduction, like tdbtorture.

      
If a Swedish-speaking Finn writes code in English, why should I complain about
a few weird spellings?

(Was succeeding in finding "Free bucket 8" string, because of "Free bucket 80:".

Now we get long options!

The former encapsulates a common "just exit on error" case, the latter
avoids having to repeat the extra usage string.

(We actually would have crashed before if someone passed NULL there, even
 though the documentation said you could).

A little less typing for users.

compiler, list, noerr, sparse_bsearch, str, str_talloc, stringmap, talloc_link, tdb, tdb2, typesafe_cb: fix examples

Phew, now they call compile!

Now (with fixes) every ccanlint example compiles!

      
      /**
       * struct foo - some description
      
So, <identifier> here can have a space in it.

      
This hit my doc extraction tool, so fix it!

This is everything up to the list module... now it's time for sleep.

Including intuiting ccan depends based on headers...

In particular, we can assume "somefunc()" exists in our examples.

Try to glue consecutive examples together if all else fails (works well for
ccan/alloc/alloc.h).  This means we should put examples into list in order.

  
These are from hash: the entire example is indented, there is no main(), and
there are unused static functions.

Some of the compulsory tests don't have a total_score; treat that as "1".
And don't print out a score if the total is <= 1.