#43908 [Opn]: exec() swallows whitespace in output

From: Date: Mon, 02 Jun 2008 13:26:52 +0000
Subject: #43908 [Opn]: exec() swallows whitespace in output
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-125428@lists.php.net to get a copy of this message
 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] felipe@php.net

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] felipe@php.net

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



Thread (17 messages)

« previous php.bugs (#125428) next »