[Bro-Dev] #754: Complete implementation of switch statement

Bro Tracker bro at tracker.bro-ids.org
Mon Jan 14 15:46:15 PST 2013


#754: Complete implementation of switch statement
----------------------------+------------------------
  Reporter:  seth           |      Owner:  robin
      Type:  Merge Request  |     Status:  reopened
  Priority:  Normal         |  Milestone:  Bro2.2
 Component:  Bro            |    Version:  git/master
Resolution:                 |   Keywords:  language
----------------------------+------------------------

Comment (by robin):

 Sorry to keep going here but now I have a problem with this: our syntax is
 pretty much exactly like C, and C does fall-through. Having the same
 syntax but no fall-through seems wrong to me, and actually I find *that*
 to be unsafe in the sense that when I read code like the following, I
 *expect* it to fall-through and hence will misunderstand what's happening:

 {{{
 switch ( v ) {
 case A:
     x();
 case B:
     y();

 }}}

 I think we should do either fall-through or use a different syntax for the
 switch-statement.

-- 
Ticket URL: <http://tracker.bro-ids.org/bro/ticket/754#comment:26>
Bro Tracker <http://tracker.bro-ids.org/bro>
Bro Issue Tracker



More information about the bro-dev mailing list