static-check hits

Hi Taylor,

Just a quick note about new hits from my 'static-check.pl' script caused by the 'tb/cruft-packs' branch. This script notes any symbols that are not referenced outside the defining compilation unit. (So they could be declared static in that compilation unit). Comparing the current 'next' and 'seen' branches:

  $ diff nsc ssc
  ...
  62a63,64
  > pack-mtimes.o	- pack_has_mtimes
  > packfile.o	- close_pack_mtimes
  ...
  $ 

This is not necessarily a problem, of course, if you have patches/plans to add callers in the future (or that they simply 'round out' an API). I haven't looked (so can't comment), beyond:

  $ git grep -n pack_has_mtimes
  pack-mtimes.c:14:int pack_has_mtimes(struct packed_git *p)
  pack-mtimes.h:11:int pack_has_mtimes(struct packed_git *p);
  $ git grep -n close_pack_mtimes
  packfile.c:336:void close_pack_mtimes(struct packed_git *p) {
  packfile.c:350: close_pack_mtimes(p);
  packfile.h:94:void close_pack_mtimes(struct packed_git *p);
  $ 

Note that 'pack_has_mtimes()' has no callers at all. Also, the function definition of 'close_pack_mtimes()' has the opening { of the body on the function header line, rather than by itself on the following line.

Just an FYI.

ATB, Ramsay Jones