Re: cvs: php4 /ext/mbstring mbstring.c

From: Date: Tue, 23 Apr 2002 09:49:31 +0000
Subject: Re: cvs: php4 /ext/mbstring mbstring.c
References: 1  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-11340@lists.php.net to get a copy of this message
On Tue, 23 Apr 2002, Yasuo Ohgaki wrote: > yohgaki Tue Apr 23 04:07:20 2002 EDT > > Modified files: > /php4/ext/mbstring mbstring.c > Log: > Fix possible wide char prolem. > Fix empty output when no conversion is performed. > # Derick, this needs to be merged. Go ahead... the branch is open again :) Derick > > > Index: php4/ext/mbstring/mbstring.c > diff -u php4/ext/mbstring/mbstring.c:1.63 php4/ext/mbstring/mbstring.c:1.64 > --- php4/ext/mbstring/mbstring.c:1.63 Sun Apr 21 18:06:55 2002 > +++ php4/ext/mbstring/mbstring.c Tue Apr 23 04:07:19 2002 > @@ -17,7 +17,7 @@ > +----------------------------------------------------------------------+ > */ > > -/* $Id: mbstring.c,v 1.63 2002/04/21 22:06:55 hirokawa Exp $ */ > +/* $Id: mbstring.c,v 1.64 2002/04/23 08:07:19 yohgaki Exp $ */ > > /* > * PHP4 Multibyte String module "mbstring" (currently only for Japanese) > @@ -1539,8 +1539,7 @@ > MBSTRG(outconv) = NULL; > } > if (encoding == mbfl_no_encoding_pass) { > - RETVAL_STRING(arg_string, 1); > - return; > + RETURN_STRINGL(arg_string, arg_string_len, 1); > } > /* if content-type is not yet set, set it and activate the converter */ > if (SG(sapi_headers).send_default_content_type ) { > @@ -1563,9 +1562,7 @@ > > /* just return if the converter is not activated. */ > if (MBSTRG(outconv) == NULL) { > - zval_dtor(return_value); > - Z_STRVAL_P(return_value) = arg_string; > - zval_copy_ctor(return_value); > + RETURN_STRINGL(arg_string, arg_string_len, 1); > return; > } > > > > > -- > PHP CVS Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php > ----------------------------------------------------------------------- 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 -----------------------------------------------------------------------

« previous php.cvs (#11340) next »