Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#10736 closed defect (bug) (fixed)

Unneeded initialization of $output

Reported by: nbachiyski Owned by:
Milestone: 2.9 Priority: lowest
Severity: trivial Version: 2.9
Component: General Keywords: has-patch tested
Focuses: Cc:


The variable is initialized and on the next line it is set to another value. In this case there is no need of the initialization.

Attachments (1)

unneeded-initialization.diff (456 bytes) - added by nbachiyski 11 years ago.

Download all attachments as: .zip

Change History (5)

#1 @nbachiyski
11 years ago

  • Keywords has-patch added

#2 @scribu
11 years ago

  • Milestone changed from Unassigned to 2.9

#3 @miqrogroove
11 years ago

  • Cc miqrogroove@… added
  • Keywords tested added

I'll second that. $output === NULL if the assignment is ever invalid, so the extra string assignment was meaningless in this scope.

Line numbers are slightly different in Revision 12149.

#4 @ryan
11 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [12194]) Remove unnecessary variable initialization. Props nbachiyski. fixes #10736

Note: See TracTickets for help on using tickets.