ID: 43908
User updated by: Arne dot Heizmann at csr dot com
Reported By: Arne dot Heizmann at csr dot com
Status: Open
Bug Type: Program Execution
Operating System: Windows 2000
PHP Version: 5.2.5
New Comment:
Could we have this fix in the next version please? Just remove all the
code that removes the whitespace?
Previous Comments:
------------------------------------------------------------------------
[2008-01-25 18:43:54] Arne dot Heizmann at csr dot com
Twice already I've asked "WHY" and you still haven't answered that
question. Why is the whitespace being removed at all?
------------------------------------------------------------------------
[2008-01-25 18:24:43] [EMAIL PROTECTED]
Because that is intentional, however, it affect results as your case,
when the string is consisting entirely of whitespace.
Well, anyway, maybe this isn't a good idea, it was only my idea.
------------------------------------------------------------------------
[2008-01-25 17:59:46] Arne dot Heizmann at csr dot com
> The actual implementation strip trailing whitespaces, my patch also
do
> it, but when the string contains only whitespaces, it remove just \r
> and \n.
So why are you explicitly fixing only half of the bug and leaving the
other half in?
------------------------------------------------------------------------
[2008-01-24 20:37:30] [EMAIL PROTECTED]
The actual implementation strip trailing whitespaces, my patch also do
it, but when the string contains only whitespaces, it remove just \r and
\n.
------------------------------------------------------------------------
[2008-01-24 15:46:45] Arne dot Heizmann at csr dot com
In your patch it appears that you are still deliberately removing
whitespace. Why?
------------------------------------------------------------------------
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/43908
--
Edit this bug report at http://bugs.php.net/?id=43908&edit=1
