This is what was breaking my debug_backtrace. Please commit immediately. This is a much-needed fix!
| Comment | File | Size | Author |
|---|---|---|---|
| no_end_php_tag.patch | 2.1 KB | dmitrig01 |
This is what was breaking my debug_backtrace. Please commit immediately. This is a much-needed fix!
| Comment | File | Size | Author |
|---|---|---|---|
| no_end_php_tag.patch | 2.1 KB | dmitrig01 |
Comments
Comment #1
jp.stacey commentedHi,
this is my first patch test, so please let me know if this isn't right.
I tested this patch on Drupal HEAD (http://drupal.org/node/28245). Before and after the patch I swept the codebase with:
grep "?>" * -rI | grep -v .tpl.php | grep -v "<?xml"Prior to the patch there was indeed a set of files with "?>" at the end of them:
After the patch these were removed from the output of the
grep.The rest of the results returned from the grep were either regular expressions, comments or similar.
The patch installed fine with no errors or line-number warnings.
Comment #2
gábor hojtsyThanks, committed!
Comment #3
dmitrig01 commentedjp.stacey - Excellent review!
Comment #4
(not verified) commented