During the verification of bug #26600, I just noticed fgetcsv() now behaves differently than the previous release 4.3.4.
After a quick examination, I found the change you made on r-1.279.2.41 is related to this issue.
http://cvs.php.net/diff.php/php-src/ext/standard/file.c? r1=1.279.2.41&r2=1.279.2.42&ty=h
Before the patch, the test case attached below had given the following result:
array(3) { [0]=> string(1) "a" [1]=> string(1) "b" [2]=> string(1) "c" }
Since it was patched, fgetcsv() returns the following:
array(3) { [0]=> string(1) "a" [1]=> string(2) " b" [2]=> string(2) " c" }
While I think the new behaviour is consistent with the CSV format used by Microsoft Excel, it'd be a BC problem also.
Test case: <?php $file = '/tmp/test.csv'; $fp = fopen($file, 'w'); fwrite($fp, "a, b, c\n"); fclose($fp); $fp = fopen($file, 'r'); var_dump(fgetcsv($fp, filesize($file))); fclose($fp); ?>
What do you think of this?
Moriyoshi
-- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php