[wp-trac] [WordPress Trac] #28409: switch statements should be formatted consistently

WordPress Trac noreply at wordpress.org
Fri May 30 17:42:12 UTC 2014


#28409: switch statements should be formatted consistently
----------------------------+-------------------
 Reporter:  wonderboymusic  |      Owner:
     Type:  enhancement     |     Status:  new
 Priority:  normal          |  Milestone:  4.0
Component:  General         |    Version:  trunk
 Severity:  normal          |   Keywords:
  Focuses:                  |
----------------------------+-------------------
 I have been looking at a number of issues over at #27882.

 There are a variety of places in core that format `switch` statements
 differently. IMO, `default` should always be at the bottom. It's more
 consistent, and it doesn't confuse static analysis tools (we don't have to
 do everything that is suggested by automated tools).

 {{{
 switch ( $var ) {
     default:
         break;
     case 1:
         $return = 'match';
         break;
 }
 }}}

 is better than:

 {{{
 switch ( $var ) {
     case 1:
         $return = 'match';
         break;
     default:
         break;
 }
 }}}

--
Ticket URL: <https://core.trac.wordpress.org/ticket/28409>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list