RE: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend.c
| From: | Dmitry Stogov | Date: | Tue, 16 Aug 2005 06:41:56 +0000 |
| Subject: | RE: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend.c | ||
| References: | 1 | Groups: | php.zend-engine.cvs |
| Request: | Send a blank email to php-zend-engine-cvs+get-4017@lists.php.net to get a copy of this message | ||
You aren't right.
It is posiible to have unicode property in non-unicode mode (class name is
string and property name is unicode).
<?php
$obj = new stdClass();
$obj->{(unicode)"asd"} = 1;
var_dump($obj);
?>
Dmitry.
> -----Original Message-----
> From: Marcus Boerger [mailto:helly@php.net]
> Sent: Monday, August 15, 2005 11:36 PM
> To: Dmitry Stogov
> Cc: zend-engine-cvs@lists.php.net
> Subject: Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend.c
>
>
> Hello Dmitry,
>
> ah that's the point i didn't get. Anyway still prop_name
> and class_name are the same string type so still it should be
> zend_printf("%R:%R:private", ztype, prop_name, ztype,
> class_name); instead of zend_printf("%R:%v:private", ztype,
> prop_name, class_name);
>
> can you please check?
>
> regards
> marcus
>
>
> Monday, August 15, 2005, 4:43:49 PM, you wrote:
>
> > In unicode mode class_names are always unicode.
>
> > The key_type is not IS_UNICODE but ZEND_HASH_KEY_UNICODE in
> this code.
>
> > Dmitry.
>
> >> -----Original Message-----
> >> From: Marcus Boerger [mailto:helly@php.net]
> >> Sent: Monday, August 15, 2005 12:34 PM
> >> To: Dmitry Stogov
> >> Cc: zend-engine-cvs@lists.php.net
> >> Subject: Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend.c
> >>
> >>
> >> Hello Dmitry,
> >>
> >> what does this fix? The names are stored in the same value
> >> and thus should either be both unicode or native strings. Did
> >> you find something else?
> >>
> >> marcus
> >>
> >> Monday, August 15, 2005, 9:12:55 AM, you wrote:
> >>
> >> > dmitry Mon Aug 15 03:12:55 2005 EDT
> >>
> >> > Modified files:
> >> > /ZendEngine2 zend.c
> >> > Log:
> >> > Fixed another print_r() bug
> >>
> >>
> >> >
> >>
>
> =_$
3http://cvs.php.net/diff.php/ZendEngine2/zend.c?r1=1.314&r2=1.315&ty=u
> >> > Index: ZendEngine2/zend.c
> >> > diff -u ZendEngine2/zend.c:1.314 ZendEngine2/zend.c:1.315
> >> > --- ZendEngine2/zend.c:1.314 Mon Aug 15 02:34:20 2005
> >> > +++ ZendEngine2/zend.c Mon Aug 15 03:12:52 2005
> >> > @@ -17,7 +17,7 @@
> >> >
> >> >
> >> +-------------------------------------------------------------
> >> ---------+
> >> > */
> >>
> >> > -/* $Id: zend.c,v 1.314 2005/08/15 06:34:20 dmitry Exp $ */
> >> > +/* $Id: zend.c,v 1.315 2005/08/15 07:12:52 dmitry Exp $ */
> >>
> >> > #include "zend.h"
> >> > #include "zend_extensions.h"
> >> > @@ -276,7 +276,7 @@
> >> > if
> >> > (class_name[0]=='*') {
> >> >
> >> > zend_printf("%R:protected", ztype, prop_name);
> >> > } else {
> >> > -
> >> > zend_printf("%R:%R:private", ztype, prop_name, key_type,
> >> class_name);
> >> > +
> >> > zend_printf("%R:%v:private", ztype, prop_name, class_name);
> >> > }
> >> > } else {
> >> > zend_printf("%R",
> >> > ztype, prop_name);
>
> --
> Zend Engine CVS Mailing List (http://cvs.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>