Move base::va_copy from a function to a macro.

base/port.h

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef BASE_PORT_H_
 #define BASE_PORT_H_
 
 #include <stdarg.h>
 #include "build/build_config.h"
 
@@ skipping 14 matching lines @@
 #define GG_INT8_C(x)    (x)
 #define GG_INT16_C(x)   (x)
 #define GG_INT32_C(x)   (x)
 #define GG_INT64_C(x)   GG_LONGLONG(x)
 
 #define GG_UINT8_C(x)   (x ## U)
 #define GG_UINT16_C(x)  (x ## U)
 #define GG_UINT32_C(x)  (x ## U)
 #define GG_UINT64_C(x)  GG_ULONGLONG(x)
 
-namespace base {
-
 // It's possible for functions that use a va_list, such as StringPrintf, to
 // invalidate the data in it upon use.  The fix is to make a copy of the
 // structure before using it and use that copy instead.  va_copy is provided
 // for this purpose.  MSVC does not provide va_copy, so define an
 // implementation here.  It is not guaranteed that assignment is a copy, so the
 // StringUtil.VariableArgsFunc unit test tests this capability.
-inline void va_copy(va_list& a, va_list& b) {
 #if defined(COMPILER_GCC)
-  ::va_copy(a, b);
+#define GG_VA_COPY(a, b) (::va_copy(a, b))

[Mark Mentovai, 2009/07/27 21:07:30]

You don't need the :: any longer.

 #elif defined(COMPILER_MSVC)
-  a = b;
+#define GG_VA_COPY(a, b) (a = b)
 #endif
-}
-
-}  // namespace base
 
 // Define an OS-neutral wrapper for shared library entry points
 #if defined(OS_WIN)
 #define API_CALL __stdcall
 #elif defined(OS_LINUX) || defined(OS_MACOSX)
 #define API_CALL
 #endif
 
 #endif  // BASE_PORT_H_