Description: ------------ Accoding to Text/Diff/Engine/xdiff.php line 45 states that it is looking for an array item of 0 $line[0]. Unfortunetly the content of line is actually a string at this point. The change outlined below explains what I needed to do in order to get the switch working as expected. $ diff xdiff.php /usr/share/php/Text/Diff/Engine/xdiff.php 45c45 < switch ($line[0])) { --- > switch (substr($line, 0, 1)) { Please excuse my lack of bug reporting skills. Test script: --------------- <?php $diff = array('foo', 'bar', 'baz'); foreach ($diff as $line) { switch ($line[0]) { case ' foo': break; } } ?> Actual result: -------------- /usr/share/php/Text/Diff/Engine/xdiff.php [47]: Uninitialized string offset: 0

The following patch has been added/updated: Patch Name: bugfix-16240.patch Revision: 1242747673 URL: http://pear.php.net/bugs/patch-display.php?bug=16240&patch=bugfix-16240.patch&revision=1242747673&display=1

Corrected summary.

Thanks for the patch Scott!

Thank you for taking the time to write to us, but this is not a bug. There is nothing wrong with that.

Ah, I didn't see the full report. The warning shouldn't happen of course.

This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.