#39260 [NEW]: A misleading bit in the Zend API

From: Date: Thu, 26 Oct 2006 01:57:41 +0000
Subject: #39260 [NEW]: A misleading bit in the Zend API
Groups: php.bugs 
Request: Send a blank email to php-bugs+get-103992@lists.php.net to get a copy of this message
From: sfox@php.net Operating system: irrelevant PHP version: 5.2.0RC5 PHP Bug Type: Scripting Engine problem Bug description: A misleading bit in the Zend API Description: ------------ Not the biggest bug in the world, more a mosquito... In zend_compile.h: ZEND_API int zend_unmangle_property_name(char *mangled_property, int mangled_property_len, char **prop_name, char **class_name); In zend_compile.c: ZEND_API int zend_unmangle_property_name(char *mangled_property, int len, char **class_name, char **prop_name) Someone's obviously noticed this a long time ago - on SUCCESS we have: *class_name = mangled_property+1; *prop_name = (*class_name)+class_name_len; and nobody writing for PECL seems to have made the obvious mistake (yet). But somebody, somewhere, will. The declaration in the header file obviously needs changing to match the source file, and as for the implementation - well, it might just be nice to make it read more sanely :) If I trusted myself to do that without somehow messing it up along the way and breaking practically every extension out there, there'd be a patch with this report. I'm afraid I don't, sorry. -- Edit bug report at http://bugs.php.net/?id=39260&edit=1 -- Try a CVS snapshot (PHP 4.4): http://bugs.php.net/fix.php?id=39260&r=trysnapshot44 Try a CVS snapshot (PHP 5.2): http://bugs.php.net/fix.php?id=39260&r=trysnapshot52 Try a CVS snapshot (PHP 6.0): http://bugs.php.net/fix.php?id=39260&r=trysnapshot60 Fixed in CVS: http://bugs.php.net/fix.php?id=39260&r=fixedcvs Fixed in release: http://bugs.php.net/fix.php?id=39260&r=alreadyfixed Need backtrace: http://bugs.php.net/fix.php?id=39260&r=needtrace Need Reproduce Script: http://bugs.php.net/fix.php?id=39260&r=needscript Try newer version: http://bugs.php.net/fix.php?id=39260&r=oldversion Not developer issue: http://bugs.php.net/fix.php?id=39260&r=support Expected behavior: http://bugs.php.net/fix.php?id=39260&r=notwrong Not enough info: http://bugs.php.net/fix.php?id=39260&r=notenoughinfo Submitted twice: http://bugs.php.net/fix.php?id=39260&r=submittedtwice register_globals: http://bugs.php.net/fix.php?id=39260&r=globals PHP 3 support discontinued: http://bugs.php.net/fix.php?id=39260&r=php3 Daylight Savings: http://bugs.php.net/fix.php?id=39260&r=dst IIS Stability: http://bugs.php.net/fix.php?id=39260&r=isapi Install GNU Sed: http://bugs.php.net/fix.php?id=39260&r=gnused Floating point limitations: http://bugs.php.net/fix.php?id=39260&r=float No Zend Extensions: http://bugs.php.net/fix.php?id=39260&r=nozend MySQL Configuration Error: http://bugs.php.net/fix.php?id=39260&r=mysqlcfg

« previous php.bugs (#103992) next »