Re: cvs: php4 /ext/zlib zlib_fopen_wrapper.c /main streams.c

On Sat, 6 Apr 2002, Wez Furlong wrote: > wez Sat Apr 6 12:29:39 2002 EDT > > Modified files: > /php4/main streams.c > /php4/ext/zlib zlib_fopen_wrapper.c > Log: > Added BC support for old style zlib: wrapper. > Added notice when a requested wrapper is not found. > # This BC thing was giving me nightmares. > # It took me 10 minutes to realize I hadn't compiled in zlib support also. > > > Index: php4/main/streams.c > diff -u php4/main/streams.c:1.45 php4/main/streams.c:1.46 > --- php4/main/streams.c:1.45 Tue Apr 2 11:46:33 2002 > +++ php4/main/streams.c Sat Apr 6 12:29:38 2002 > @@ -1081,10 +1081,24 @@ > > if ((*p == ':') && (n > 1) && !strncmp("://", p, 3)) { > protocol = path; > + } else if (strncasecmp(path, "zlib:", 5) == 0) { > + /* BC with older php scripts and zlib wrapper */ > + protocol = path; > + n = 4; > + zend_error(E_NOTICE, "Use of \"zlib:\" wrapper is deprecated; please use > \"zlib://\" instead."); > } I'd say to make this an E_WARNING instead. regards, Derick ----------------------------------------------------------------------- Did I help you? Consider a gift: http://www.amazon.co.uk/exec/obidos/registry/SLCB276UZU8B ----------------------------------------------------------------------- PHP: Scripting the Web - derick@php.net All your branches are belong to me! SRM: Script Running Machine - www.vl-srm.net -----------------------------------------------------------------------