I've been tidying up a bit tonight getting GLib ready for -Wmissing-prototypes.  It's just about ready to turn on by default.

The last remaining issue is that G_DISABLE_DEPRECATED interacts pretty badly with it.  When we define that, we skip the declarations in our headers, which of course triggers the warning.  imho, this is a pretty insane thing to be doing anyway, so we should stop it.

There are two ways forward that I can see:

  1) now that we do attribute-based deprecations, stop it with the wrapping
     of functions (at least) in G_DISABLE_DEPRECATED.

     We could use G_DISABLE_DEPRECATED only for types, or ditch it entirely.


  2) Stop using -DG_DISABLE_DEPRECATED in the CFLAGS used to build GLib.



I think I like option #1.  It's always seemed weird to me that we deprecate merely by hiding declarations.  It also means that you don't see the new attribute-based warnings if you -DG_DISABLE_DEPRECATED which is a bit lame...