Edit report at https://bugs.php.net/bug.php?id=60809&edit=1
ID: 60809
Updated by: larue...@php.net
Reported by: micronix at gmx dot net
Summary: TRAITS - PHPDoc Comment Style Bug
Status: Critical
Type: Bug
Package: *General Issues
Operating System: Windows
PHP Version: 5.4.0RC5
Assigned To: pierrick
Block user comment: N
Private report: N
New Comment:
ah, you are right, ingore my noise :)
Previous Comments:
------------------------------------------------------------------------
[2012-01-20 02:28:53] pierr...@php.net
I can do it like that yes. I'll just have to declare the variable inside the
loop
to make sure the doc_comment is reinitialized and NULL if no doc_comment is
there.
------------------------------------------------------------------------
[2012-01-20 02:26:32] larue...@php.net
pierrick, I suggest doing like:
char *doccomment = NULL;
if () {
doccomment = estrndup();
}
thanks :)
------------------------------------------------------------------------
[2012-01-20 02:26:31] larue...@php.net
pierrick, I suggest doing like:
char *doccomment = NULL;
if () {
doccomment = estrndup();
}
thanks :)
------------------------------------------------------------------------
[2012-01-20 02:21:52] pierr...@php.net
Problem is because zend_do_traits_property_binding use zend_declare_property_ex
without doing a estrndup of the doc_comment. I attached a patch please Stas or
dsp confirm that I can commit it :)
------------------------------------------------------------------------
[2012-01-20 02:19:54] pierr...@php.net
The following patch has been added/updated:
Patch Name: 60809.diff
Revision: 1327025994
URL:
https://bugs.php.net/patch-display.php?bug=60809&patch=60809.diff&revision=1327025994
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=60809
--
Edit this bug report at https://bugs.php.net/bug.php?id=60809&edit=1
