| php-bugs at t43 dot mine dot nu 2006-03-30, 6:58 pm |
| ID: 36877
User updated by: php-bugs at t43 dot mine dot nu
Reported By: php-bugs at t43 dot mine dot nu
Status: Open
Bug Type: Documentation problem
Operating System: na
PHP Version: Irrelevant
New Comment:
IMHO
(1) should only be documented what is intended. ZEND should say about
comma in expr2 before it makes it into the manual. Such deviation from
C should be intentional if at all.
(2) in C++, by 5.2 of its definition, only assigments can be
comma-sequenced. Simple boolean expressions are not yet assignments.
The value of a sequenced expression is the value of the last assigment,
not an OR of them.
(3) it would be more logical to give comma in expr-2 the meaning of
AND: continue while (data available AND buffer available). But better
nothing.
Previous Comments:
------------------------------------------------------------------------
[2006-03-30 16:30:09] colder@php.net
Thanks for the note arpad,
here is the updated patch:
http://patches.colder.ch/control-structures-for-2.patch
------------------------------------------------------------------------
[2006-03-30 11:33:58] arpad@php.net
You can use a comma in expr2 too, it's equivalent to || but much lower
in precedence.
------------------------------------------------------------------------
[2006-03-30 09:29:48] php-bugs at t43 dot mine dot nu
I thank you sincerely for the answer.
------------------------------------------------------------------------
[2006-03-30 04:10:15] simp@php.net
This bug has been fixed in the documentation's XML sources. Since the
online and downloadable versions of the documentation need some time
to get updated, we would like to ask you to be a bit patient.
Thank you for the report, and for helping us make our documentation
better.
thanks for the patch
------------------------------------------------------------------------
[2006-03-28 17:46:01] colder@php.net
Note that the use of multiple expressions in expr3 was already
mentionned in example4.
Anyway, here is a suggestion of a possible improvement:
http://patches.colder.ch/control-structures-for.patch
Thanks in advance.
------------------------------------------------------------------------
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
http://bugs.php.net/36877
--
Edit this bug report at http://bugs.php.net/?id=36877&edit=1
|