Re: cvs: php4 /ext/pcre php_pcre.h /main SAPI.c

At 09:53 AM 10/21/01 -0700, Rasmus Lerdorf wrote:

That's a good point. What was the point of of that switch anyway? It's not like the pcre library is all that large.

-rw-r--r--    1 root     root        41452 Oct 21 09:48 libpcre.a

Was there ever some functional reason for leaving it out? Does it not compile everywhere? I don't see any recently reported bugs related to that, so I think it should be ok to remove. Although, I could make the code check for the presence and if it isn't there just do a stupid replacement of the realm instead of trying to be smart about it. Perhaps that is a better resolution.

There is a reason for it, but it's buried deep in the archives. Although I think that we can remove that switch now that PCRE has proven to be pretty stable and widely used. -Andrei