message: [Trivial] Move class below the global function declaration
diff: @@ -47,6 +47,20 @@ LEAVE_CRITICAL_SECTION(mutex); // no RAII
  // //
  ///////////////////////////////
  +#ifdef DEBUG_LOCKORDER
  +void EnterCritical(const char* pszName, const char* pszFile, int nLine, void* cs, bool fTry = false);
  +void LeaveCritical();
  +std::string LocksHeld();
  +void AssertLockHeldInternal(const char* pszName, const char* pszFile, int nLine, void* cs);
  +void DeleteLock(void* cs);
  +#else
  +void static inline EnterCritical(const char* pszName, const char* pszFile, int nLine, void* cs, bool fTry = false) {}
  +void static inline LeaveCritical() {}
  +void static inline AssertLockHeldInternal(const char* pszName, const char* pszFile, int nLine, void* cs) {}
  +void static inline DeleteLock(void* cs) {}
  +#endif
  +#define AssertLockHeld(cs) AssertLockHeldInternal(#cs, __FILE__, __LINE__, &cs)
  +
  /**
  * Template mixin that adds -Wthread-safety locking
  * annotations to a subset of the mutex API.
  @@ -71,20 +85,6 @@ public:
  }
  };
  -#ifdef DEBUG_LOCKORDER
  -void EnterCritical(const char* pszName, const char* pszFile, int nLine, void* cs, bool fTry = false);
  -void LeaveCritical();
  -std::string LocksHeld();
  -void AssertLockHeldInternal(const char* pszName, const char* pszFile, int nLine, void* cs);
  -void DeleteLock(void* cs);
  -#else
  -void static inline EnterCritical(const char* pszName, const char* pszFile, int nLine, void* cs, bool fTry = false) {}
  -void static inline LeaveCritical() {}
  -void static inline AssertLockHeldInternal(const char* pszName, const char* pszFile, int nLine, void* cs) {}
  -void static inline DeleteLock(void* cs) {}
  -#endif
  -#define AssertLockHeld(cs) AssertLockHeldInternal(#cs, __FILE__, __LINE__, &cs)
  -
  /**
  * Wrapped mutex: supports recursive locking, but no waiting
  * TODO: We should move away from using the recursive lock by default.
  