Closed Bug 791643 Opened 8 years ago Closed 8 years ago

Clean up PHP trailing whitespace and closing tags

Categories

(Tree Management Graveyard :: TBPL, defect, minor)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

Attached patch Patch v1Splinter Review
At some point I want to go through TBPL's php with a style checker, but for now here are a few trailing whitespace & closing php tag tweaks (to match the rest of the php files and for http://framework.zend.com/manual/1.12/en/coding-standard.php-file-formatting.html#coding-standard.php-file-formatting.general).
Attachment #661739 - Flags: review?(mstange)
Meant to add: The review is mainly for the closing tag removal, since whilst the majority of the files do already omit it, there is a mixture of opinions on matter for the various google searches I did.
Assignee: nobody → bmo
Status: NEW → ASSIGNED
Attachment #661739 - Flags: review?(mstange) → review?(arpad.borsos)
Attachment #661739 - Flags: review?(arpad.borsos) → review+
Thank you :-)

http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/9c7f496beb97
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Depends on: 799720
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.