Re: cvs: php4 /ext/mbstring mbstring.c
| From: | derick@php.net | 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
-----------------------------------------------------------------------