Merge branch 'jk/avoid-redef-system-functions-2.30'

Redefining system functions for a few functions did not follow our
usual "implement git_foo() and #define foo(args) git_foo(args)"
pattern, which has broken build for some folks.

* jk/avoid-redef-system-functions-2.30:
  git-compat-util: undefine system names before redeclaring them
  git-compat-util: avoid redefining system function names

diff --cc git-compat-util.h
index a76d0526f79266e1dc512072a4236876df8f8726,09a5f338594a85b98d833d19e06da826245da736..76e4b111315339449cd48d17f735f727935c08d8
--- 1/git-compat-util.h
--- 2/git-compat-util.h
+++ b/git-compat-util.h
@@@ -341,11 -274,13 +341,13 @@@ struct itimerval 
  #endif
  
  #ifdef NO_SETITIMER
- static inline int setitimer(int which UNUSED,
-                           const struct itimerval *value UNUSED,
-                           struct itimerval *newvalue UNUSED) {
 -static inline int git_setitimer(int which,
 -                              const struct itimerval *value,
 -                              struct itimerval *newvalue) {
++static inline int git_setitimer(int which UNUSED,
++                              const struct itimerval *value UNUSED,
++                              struct itimerval *newvalue UNUSED) {
        return 0; /* pretend success */
  }
+ #undef setitimer
+ #define setitimer(which,value,ovalue) git_setitimer(which,value,ovalue)
  #endif
  
  #ifndef NO_LIBGEN_H
@@@ -1471,11 -1316,11 +1473,11 @@@ int open_nofollow(const char *path, in
  #endif
  
  #ifndef _POSIX_THREAD_SAFE_FUNCTIONS
- static inline void flockfile(FILE *fh UNUSED)
 -static inline void git_flockfile(FILE *fh)
++static inline void git_flockfile(FILE *fh UNUSED)
  {
        ; /* nothing */
  }
- static inline void funlockfile(FILE *fh UNUSED)
 -static inline void git_funlockfile(FILE *fh)
++static inline void git_funlockfile(FILE *fh UNUSED)
  {
        ; /* nothing */
  }