From 4e7e54c4c522ab2f6a7abb92cefc4f707e9568fb Mon Sep 17 00:00:00 2001 From: Lasse Collin Date: Sun, 6 Jan 2008 16:27:41 +0200 Subject: Introduced compatibility with systems that have pre-C99 or no inttypes.h. This is useful when the compiler has good enough support for C99, but libc headers don't. Changed liblzma API so that sys/types.h and inttypes.h have to be #included before #including lzma.h. On systems that don't have C99 inttypes.h, it's the problem of the applications to provide the required types and macros before #including lzma.h. If lzma.h defined the missing types and macros, it could conflict with third-party applications whose configure has detected that the types are missing and defined them in config.h already. An alternative would have been introducing lzma_uint32 and similar types, but that would just be an extra pain on modern systems. --- src/liblzma/api/lzma.h | 40 +++++++++++++++++++++++++------------- src/liblzma/check/crc32_table.c | 4 +--- src/liblzma/check/crc32_table_be.h | 2 -- src/liblzma/check/crc32_table_le.h | 2 -- src/liblzma/check/crc32_tablegen.c | 1 - src/liblzma/check/crc64_table.c | 4 +--- src/liblzma/check/crc64_table_be.h | 2 -- src/liblzma/check/crc64_table_le.h | 2 -- src/liblzma/check/crc64_tablegen.c | 1 - 9 files changed, 28 insertions(+), 30 deletions(-) (limited to 'src/liblzma') diff --git a/src/liblzma/api/lzma.h b/src/liblzma/api/lzma.h index 186ae12c..ad39d349 100644 --- a/src/liblzma/api/lzma.h +++ b/src/liblzma/api/lzma.h @@ -17,25 +17,37 @@ * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU * Lesser General Public License for more details. + * + * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * + * + * Before #including this file, you must make the following types available: + * - size_t + * - uint8_t + * - int32_t + * - uint32_t + * - int64_t + * - uint64_t + * + * Before #including this file, you must make the following macros available: + * - UINT32_C(n) + * - UINT64_C(n) + * - UINT32_MAX + * - UINT64_MAX + * + * Easiest way to achieve the above is to #include sys/types.h and inttypes.h + * before #including lzma.h. However, some pre-C99 libc headers don't provide + * all the required types in inttypes.h (that file may even be missing). + * Portable applications need to provide these types themselves. This way + * liblzma API can use the standard types instead of defining its own + * (e.g. lzma_uint32). + * + * Note that the API still has lzma_bool, because using stdbool.h would + * break C89 and C++ programs on many systems. */ #ifndef LZMA_H #define LZMA_H -/******************** - * External headers * - ********************/ - -/* size_t */ -#include - -/* NULL */ -#include - -/* uint8_t, uint32_t, uint64_t, UINT32_C, UINT64_C, UINT64_MAX. */ -#include - - /****************** * GCC extensions * ******************/ diff --git a/src/liblzma/check/crc32_table.c b/src/liblzma/check/crc32_table.c index b59642d4..f1cc0daf 100644 --- a/src/liblzma/check/crc32_table.c +++ b/src/liblzma/check/crc32_table.c @@ -11,9 +11,7 @@ // /////////////////////////////////////////////////////////////////////////////// -#ifdef HAVE_CONFIG_H -# include -#endif +#include "sysdefs.h" #ifdef WORDS_BIGENDIAN # include "crc32_table_be.h" diff --git a/src/liblzma/check/crc32_table_be.h b/src/liblzma/check/crc32_table_be.h index bc5a5fbd..f4f4e365 100644 --- a/src/liblzma/check/crc32_table_be.h +++ b/src/liblzma/check/crc32_table_be.h @@ -1,7 +1,5 @@ /* This file has been automatically generated by crc32_tablegen.c. */ -#include - const uint32_t lzma_crc32_table[8][256] = { { 0x00000000, 0x96300777, 0x2C610EEE, 0xBA510999, diff --git a/src/liblzma/check/crc32_table_le.h b/src/liblzma/check/crc32_table_le.h index a96d772a..7b816665 100644 --- a/src/liblzma/check/crc32_table_le.h +++ b/src/liblzma/check/crc32_table_le.h @@ -1,7 +1,5 @@ /* This file has been automatically generated by crc32_tablegen.c. */ -#include - const uint32_t lzma_crc32_table[8][256] = { { 0x00000000, 0x77073096, 0xEE0E612C, 0x990951BA, diff --git a/src/liblzma/check/crc32_tablegen.c b/src/liblzma/check/crc32_tablegen.c index 280d3b12..f05d7536 100644 --- a/src/liblzma/check/crc32_tablegen.c +++ b/src/liblzma/check/crc32_tablegen.c @@ -31,7 +31,6 @@ main() printf("/* This file has been automatically generated by " "crc32_tablegen.c. */\n\n" - "#include \n\n" "const uint32_t lzma_crc32_table[8][256] = {\n\t{"); for (size_t s = 0; s < 8; ++s) { diff --git a/src/liblzma/check/crc64_table.c b/src/liblzma/check/crc64_table.c index 0f2d1fb1..2f227319 100644 --- a/src/liblzma/check/crc64_table.c +++ b/src/liblzma/check/crc64_table.c @@ -11,9 +11,7 @@ // /////////////////////////////////////////////////////////////////////////////// -#ifdef HAVE_CONFIG_H -# include -#endif +#include "sysdefs.h" #ifdef WORDS_BIGENDIAN # include "crc64_table_be.h" diff --git a/src/liblzma/check/crc64_table_be.h b/src/liblzma/check/crc64_table_be.h index 99518400..2d2794fe 100644 --- a/src/liblzma/check/crc64_table_be.h +++ b/src/liblzma/check/crc64_table_be.h @@ -1,7 +1,5 @@ /* This file has been automatically generated by crc64_tablegen.c. */ -#include - const uint64_t lzma_crc64_table[4][256] = { { UINT64_C(0x0000000000000000), UINT64_C(0x6F5FA703BE4C2EB3), diff --git a/src/liblzma/check/crc64_table_le.h b/src/liblzma/check/crc64_table_le.h index 3047ea16..6f82cad1 100644 --- a/src/liblzma/check/crc64_table_le.h +++ b/src/liblzma/check/crc64_table_le.h @@ -1,7 +1,5 @@ /* This file has been automatically generated by crc64_tablegen.c. */ -#include - const uint64_t lzma_crc64_table[4][256] = { { UINT64_C(0x0000000000000000), UINT64_C(0xB32E4CBE03A75F6F), diff --git a/src/liblzma/check/crc64_tablegen.c b/src/liblzma/check/crc64_tablegen.c index 0f1005f4..4b35ac1f 100644 --- a/src/liblzma/check/crc64_tablegen.c +++ b/src/liblzma/check/crc64_tablegen.c @@ -31,7 +31,6 @@ main() printf("/* This file has been automatically generated by " "crc64_tablegen.c. */\n\n" - "#include \n\n" "const uint64_t lzma_crc64_table[4][256] = {\n\t{"); for (size_t s = 0; s < 4; ++s) { -- cgit v1.2.3