Some of the older DB unit tests do not use the right whitespace format. This patch adds the necessary carriage returns.
| Comment | File | Size | Author |
|---|---|---|---|
| test_formatting.patch | 4.48 KB | Crell |
Some of the older DB unit tests do not use the right whitespace format. This patch adds the necessary carriage returns.
| Comment | File | Size | Author |
|---|---|---|---|
| test_formatting.patch | 4.48 KB | Crell |
Comments
Comment #1
c960657 commentedI'm not sure what exactly the right format is (it's not mentioned in the coding standards), but it looks like some more lines could use some wrapping.
The list was generated using this command:
egrep -n "\)->(fields|condition|execute|where)" modules/simpletest/tests/database_test.test | egrep ".{81}"Comment #2
cburschkaThe syntax clean-up in the patch looks good, but since c960657 implies there is more to be done, the patch should probably be expanded.
Comment #3
davyvdb commentedLooks as if this got already in somehow.