Re: 3.0.5 beta

[email protected] (Zeev Suraski) Sun, 04 Oct 1998 18:20:13
Newsgroups php.beta
Message-ID <[email protected]>
At 18:09 04/10/98 +0200, Sascha Schumann wrote:
>Thanks for enlighting me. I would have looked more carefully, if the
>server would have responded with connection refused ... packet dropping
>firewall?

Yep...

>3.0.5 compiled cleanly in the following environment:
>
>- FreeBSD 3.0 (-current) with gcc 2.7.2.1 (that's the one they are
>shipping)
>- Apache 1.3.2
>- Postgres 6.3.2
>- imap-4.4.BETA
>- gd 1.3
>
>There is a name clash between Apache and Postgres' Libpq (C frontend)
>which results in a warning. palloc is defined in libpq-fe.h and
>ap_compat.h (this isn't a result of a failure in PHP, of course).

It should be ok.

>If compiled with -Wall, some more warnings occur:
>
>functions/imap.c uses fs_get() which wasn't declared before. I couldn't
>find the proper header file, but it's a defined symbol in c-client.a
>(probably due to the beta status of my imap version)

That's not that serious.

>functions/string.c tries to calculate source_end out of a undefined value.
>Maybe fix:
>
>--- string.c.orig	Sun Oct  4 18:00:50 1998
>+++ string.c	Sun Oct  4 18:01:00 1998
>@@ -986,11 +986,11 @@
> 
> static void _php3_char_to_str(char *str,uint len,char from,char *to,int
to_len,pval *result)
> {
> 	int char_count=0;
>-	char *source,*target,*tmp,*source_end=source+len;
>+	char *source = str,*target,*tmp,*source_end=source+len;
> 	
>-	for (source=str; *source; source++) {
>+	for (; *source; source++) {
> 		if (*source==from) {
> 			char_count++;
> 		}
> 	}

Already fixed.

Thanks for everything,


Zeev
--
Zeev Suraski   <[email protected]>
For a PGP public key, finger [email protected]