Posted by Dave Reid on September 15, 2008 at 5:42am
| Project: | Drupal core |
| Version: | 7.x-dev |
| Component: | base system |
| Category: | bug report |
| Priority: | minor |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Inspired by #308521: DBTNG is full of stray whitespace, did a search for [ \t]+$ in core and found a lot of them.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| spaces.patch | 73.26 KB | Ignored: Check issue status. | None | None |
Comments
#1
Re-rolled. Anyone able to help review? It's easy...
#2
no functional changes in the patch. it removes the trailing spaces, nothing more..
notes:
make smaller patches (by module)
next issue: remove tabs
#3
I'll be sure to do that for future reference. Thanks Pasqualle!
#4
Actually, for this type of thing that's extremely minor, one patch that does the exact same thing across all of core is a lot easier to review and commit.
Just running the test suite to make sure nothing breaks.
#5
Ok, looks good.
Committed. Thanks!
temp
Automatically closed -- issue fixed for two weeks with no activity.
#6
Automatically closed -- issue fixed for two weeks with no activity.