theme_enable() and theme_disable() have a return; at the end of each function for no apparent reason.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

boombatower’s picture

Assigned: Unassigned » boombatower
Status: Active » Needs review
FileSize
611 bytes
David_Rothstein’s picture

Title: Remove unnecessary return's » Remove unnecessary return statements
Version: 8.x-dev » 7.x-dev
Status: Needs review » Reviewed & tested by the community

They do indeed! And no reason to wait until D8 to kill them, either :)

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 1049144-remove-return.patch, failed testing.

Dave Reid’s picture

Needs to be rolled from the root directory...

boombatower’s picture

Status: Needs work » Reviewed & tested by the community
FileSize
638 bytes

Woops :)

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.