Re: [PATCH] Removed OLD_ICONV in favor of checking _LIBICONV_VERSION directly

From: Junio C Hamano <gitster@pobox.com>
Date: Nov 3, 2007, 00:07 UTC
Message-ID: <7vbqacyyx1.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <1194048010-65955-1-git-send-email-blaker@gmail.com>

Blake Ramsdell <blaker@gmail.com> writes:

Show 13 quoted lines

> diff --git a/utf8.c b/utf8.c
> index 4efef6f..a7feb4f 100644
> --- a/utf8.c
> +++ b/utf8.c
> @@ -300,7 +300,7 @@ int is_encoding_utf8(const char *name)
>   * with iconv.  If the conversion fails, returns NULL.
>   */
>  #ifndef NO_ICONV
> -#ifdef OLD_ICONV
> +#if _LIBICONV_VERSION <= 0x0109
>  	typedef const char * iconv_ibp;
>  #else
>  	typedef char * iconv_ibp;

Does everybody's iconv use the same _LIBICONV_VERSION scheme?

Compiling this:

        #include <iconv.h>
        int i = _LIBICONV_VERSION;

on a Linux box with GNU C library gives:

	i.c:3: error: '_LIBICONV_VERSION' undeclared here (not in a function)

here.