[PATCH 02/12] MINGW: compat/bswap.h: include stdint.h

2014-04-28 Thread Marat Radchenko
bswap.h uses uint32_t type which might not be defined.
This patch adds direct include so bswap.h can be safely included.

Signed-off-by: Marat Radchenko ma...@slonopotamus.org
---
 compat/bswap.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/compat/bswap.h b/compat/bswap.h
index 120c6c1..d170447 100644
--- a/compat/bswap.h
+++ b/compat/bswap.h
@@ -5,6 +5,8 @@
  * operation.
  */
 
+#include stdint.h
+
 /*
  * Default version that the compiler ought to optimize properly with
  * constant values.
-- 
1.9.1

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 02/12] MINGW: compat/bswap.h: include stdint.h

2014-04-28 Thread Erik Faye-Lund
On Mon, Apr 28, 2014 at 3:51 PM, Marat Radchenko ma...@slonopotamus.org wrote:
 bswap.h uses uint32_t type which might not be defined.
 This patch adds direct include so bswap.h can be safely included.

 Signed-off-by: Marat Radchenko ma...@slonopotamus.org
 ---
  compat/bswap.h | 2 ++
  1 file changed, 2 insertions(+)

 diff --git a/compat/bswap.h b/compat/bswap.h
 index 120c6c1..d170447 100644
 --- a/compat/bswap.h
 +++ b/compat/bswap.h
 @@ -5,6 +5,8 @@
   * operation.
   */

 +#include stdint.h
 +
  /*
   * Default version that the compiler ought to optimize properly with
   * constant values.

Hmm, what's the symptom this fixes? From what I can tell, bswap.h is
included after stdint.h from git-compat-util.h anyway...
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 02/12] MINGW: compat/bswap.h: include stdint.h

2014-04-28 Thread Marat Radchenko
On Mon, Apr 28, 2014 at 04:45:43PM +0200, Erik Faye-Lund wrote:
 bswap.h is included after stdint.h from git-compat-util.h anyway...

That only becomes true after PATCH 05 when talking about MinGW.

Will drop this one.
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 02/12] MINGW: compat/bswap.h: include stdint.h

2014-04-28 Thread Erik Faye-Lund
On Mon, Apr 28, 2014 at 4:52 PM, Marat Radchenko ma...@slonopotamus.org wrote:
 On Mon, Apr 28, 2014 at 04:45:43PM +0200, Erik Faye-Lund wrote:
 bswap.h is included after stdint.h from git-compat-util.h anyway...

 That only becomes true after PATCH 05 when talking about MinGW.

 Will drop this one.

Right, thanks.
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html