Use of <cinttypes> in memcache.h is C++11 specific

Bug #1402026 reported by Scott Hunt on 2014-12-12
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libmemcached
Undecided
Unassigned

Bug Description

I have it on good authority (including gcc-4.4.6 on CentOS-6.2) that <cinttypes> wasn't added until C++11. Seems like using <inttypes.h> instead would address that simply. (Unless the expectation is that C++11 is required.)

diff --git a/libmemcached-1.0/memcached.h b/libmemcached-1.0/memcached.h
index bc16e73..b258fbe 100644
--- a/libmemcached-1.0/memcached.h
+++ b/libmemcached-1.0/memcached.h
@@ -42,12 +42,12 @@
 # define __STDC_FORMAT_MACROS
 #endif

+#include <inttypes.h>
+
 #ifdef __cplusplus
-# include <cinttypes>
 # include <cstddef>
 # include <cstdlib>
 #else
-# include <inttypes.h>
 # include <stddef.h>
 # include <stdlib.h>
 # include <stdbool.h>

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers