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:         *General Issues
 Operating System: Windows 2000
 PHP Version:      5.2.5
 New Comment:

Twice already I've asked "WHY" and you still haven't answered that
question. Why is the whitespace being removed at all?


Previous Comments:
------------------------------------------------------------------------

[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?

------------------------------------------------------------------------

[2008-01-24 10:35:40] [EMAIL PROTECTED]

My suggestion:

http://ecl.zoone.com.br/etc/patches/bug43908.patch

------------------------------------------------------------------------

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

Reply via email to