[PATCH] fixes #26194 View as Markdown

Take a look at iconv.h line 80

#define iconv libiconv

This line is the problem I'll change the php_iconv.h file to include.

You must be right. Seems I overlooked that stuff totally though I
recognised the evil #define is there. I'll commit a patch for this
shortly if you don't mind. Thanks.

Moriyoshi

erm, I did ... but that line wasn't the thing that broke this.

(ZEND_FENTRY was.)

-----Original Message-----
From: Frank M. Kromann [mailto:frank@kromann.info]
Sent: 22 November 2003 05:09
To: Moriyoshi Koizumi
Cc: Steph; internals; Derick Rethans
Subject: Re: [PHP-DEV] [PATCH] fixes #26194

Take a look at iconv.h line 80

#define iconv libiconv

This line is the problem I'll change the php_iconv.h file to include.

• Frank

That will fix it

Works on my box (no idea how sane this is, or what it may break on
other
systems, please handle like barbed wire).

I think I know why this might happen... perhaps "iconv" is defined as
macro on windhoos. Can you please see if get_defined_functions() does
something weird? (Tip: only load the iconv ext other wise there are
too
many funcs in the output).

I don't think this is Windows specific issue. From my experience, this
might be caused by random heap corruption. (most likely spotted in
another extension) And there's never been a macro like "iconv" or
whatever
in the standard MS C library.

Moriyoshi