ISDN:divert: beautify code: useless 'break', 'return (0)', additional comments.
authorChen Gang <gang.chen@asianux.com>
Tue, 2 Apr 2013 01:37:56 +0000 (09:37 +0800)
committerDavid S. Miller <davem@davemloft.net>
Tue, 2 Apr 2013 18:31:24 +0000 (14:31 -0400)
  delete useless 'break' after 'return'.
  let 'return 0' instead of 'return (0)'
  also give a comment for 'break' to let readers notice it.

Signed-off-by: Chen Gang <gang.chen@asianux.com>
Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/isdn/divert/isdn_divert.c

index db432e6..50749a7 100644 (file)
@@ -441,8 +441,7 @@ static int isdn_divert_icall(isdn_ctrl *ic)
 
                switch (dv->rule.action) {
                case DEFLECT_IGNORE:
-                       return (0);
-                       break;
+                       return 0;
 
                case DEFLECT_ALERT:
                case DEFLECT_PROCEED:
@@ -510,10 +509,9 @@ static int isdn_divert_icall(isdn_ctrl *ic)
                        break;
 
                default:
-                       return (0); /* ignore call */
-                       break;
+                       return 0; /* ignore call */
                } /* switch action */
-               break;
+               break; /* will break the 'for' looping */
        } /* scan_table */
 
        if (cs) {