Closed Bug 791643 Opened 12 years ago Closed 12 years ago

Clean up PHP trailing whitespace and closing tags

Categories

(Tree Management Graveyard :: TBPL, defect)

Product:
Tree Management Graveyard
Component:
TBPL
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file)

Patch v1
7.95 KB, patch
Swatinem
: review+
Details | Diff | Splinter Review
Attached patch Patch v1 — — Splinter 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: 12 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.

Attachment

General

Creator:
Created:
Updated:
Size: