codelab: reformat

Author: vmaffione

.clang-format

@@ -0,0 +1,112 @@
+---
+Language:        Cpp
+# BasedOnStyle:  Google
+AccessModifierOffset: -4
+AlignAfterOpenBracket: Align
+AlignConsecutiveAssignments: true
+AlignConsecutiveDeclarations: false
+AlignEscapedNewlines: Right
+AlignOperands:   true
+AlignTrailingComments: true
+AllowAllParametersOfDeclarationOnNextLine: true
+AllowShortBlocksOnASingleLine: false
+AllowShortCaseLabelsOnASingleLine: false
+AllowShortFunctionsOnASingleLine: All
+AllowShortIfStatementsOnASingleLine: false
+AllowShortLoopsOnASingleLine: false
+AlwaysBreakAfterDefinitionReturnType: None
+AlwaysBreakAfterReturnType: TopLevelDefinitions
+AlwaysBreakBeforeMultilineStrings: false
+AlwaysBreakTemplateDeclarations: true
+BinPackArguments: true
+BinPackParameters: true
+BraceWrapping:   
+  AfterClass:      false
+  AfterControlStatement: false
+  AfterEnum:       false
+  AfterFunction:   true
+  AfterNamespace:  false
+  AfterObjCDeclaration: false
+  AfterStruct:     false
+  AfterUnion:      false
+  BeforeCatch:     false
+  BeforeElse:      false
+  IndentBraces:    false
+  SplitEmptyFunction: true
+  SplitEmptyRecord: true
+  SplitEmptyNamespace: true
+BreakBeforeBinaryOperators: None
+BreakBeforeBraces: Custom
+BreakBeforeInheritanceComma: false
+BreakBeforeTernaryOperators: true
+BreakConstructorInitializersBeforeComma: false
+BreakConstructorInitializers: BeforeColon
+BreakAfterJavaFieldAnnotations: false
+BreakStringLiterals: true
+ColumnLimit:     80
+CommentPragmas:  '^ IWYU pragma:'
+CompactNamespaces: false
+ConstructorInitializerAllOnOneLineOrOnePerLine: true
+ConstructorInitializerIndentWidth: 4
+ContinuationIndentWidth: 4
+Cpp11BracedListStyle: true
+DerivePointerAlignment: false
+DisableFormat:   false
+ExperimentalAutoDetectBinPacking: false
+FixNamespaceComments: true
+ForEachMacros:   
+  - foreach
+  - Q_FOREACH
+  - BOOST_FOREACH
+  - list_for_each_entry
+  - list_for_each_entry_safe
+  - hlist_for_each_entry
+  - rb_list_foreach
+  - rb_list_foreach_safe
+IncludeCategories: 
+  - Regex:           '^<.*\.h>'
+    Priority:        1
+  - Regex:           '^<.*'
+    Priority:        2
+  - Regex:           '.*'
+    Priority:        3
+IncludeIsMainRegex: '([-_](test|unittest))?$'
+IndentCaseLabels: false
+IndentWidth:     4
+IndentWrappedFunctionNames: false
+JavaScriptQuotes: Leave
+JavaScriptWrapImports: true
+KeepEmptyLinesAtTheStartOfBlocks: false
+MacroBlockBegin: ''
+MacroBlockEnd:   ''
+MaxEmptyLinesToKeep: 1
+NamespaceIndentation: None
+ObjCBlockIndentWidth: 4
+ObjCSpaceAfterProperty: false
+ObjCSpaceBeforeProtocolList: false
+PenaltyBreakAssignment: 2
+PenaltyBreakBeforeFirstCallParameter: 1
+PenaltyBreakComment: 300
+PenaltyBreakFirstLessLess: 120
+PenaltyBreakString: 1000
+PenaltyExcessCharacter: 1000000
+PenaltyReturnTypeOnItsOwnLine: 200
+PointerAlignment: Right
+ReflowComments:  true
+SortIncludes:    false
+SortUsingDeclarations: true
+SpaceAfterCStyleCast: false
+SpaceAfterTemplateKeyword: true
+SpaceBeforeAssignmentOperators: true
+SpaceBeforeParens: ControlStatements
+SpaceInEmptyParentheses: false
+SpacesBeforeTrailingComments: 1
+SpacesInAngles:  false
+SpacesInContainerLiterals: false
+SpacesInCStyleCastParentheses: false
+SpacesInParentheses: false
+SpacesInSquareBrackets: false
+Standard:        Auto
+TabWidth:        4
+UseTab:          Never
+...

codelab/fe.c

@@ -23,11 +23,11 @@
 #include <netinet/udp.h>
 #include <netinet/tcp.h>
 
-static int stop = 0;
+static int stop                   = 0;
 static unsigned long long fwdback = 0;
-static unsigned long long fwda = 0;
-static unsigned long long fwdb = 0;
-static unsigned long long tot = 0;
+static unsigned long long fwda    = 0;
+static unsigned long long fwdb    = 0;
+static unsigned long long tot     = 0;
 
 static void
 sigint_handler(int signum)
@@ -40,13 +40,13 @@ rx_ready(struct nm_desc *nmd)
 {
     unsigned int ri;
 
-    for (ri = nmd->first_rx_ring; ri <= nmd->last_rx_ring; ri ++) {
-            struct netmap_ring *ring;
+    for (ri = nmd->first_rx_ring; ri <= nmd->last_rx_ring; ri++) {
+        struct netmap_ring *ring;
 
-            ring = NETMAP_RXRING(nmd->nifp, ri);
-            if (nm_ring_space(ring)) {
-                return 1; /* there is something to read */
-            }
+        ring = NETMAP_RXRING(nmd->nifp, ri);
+        if (nm_ring_space(ring)) {
+            return 1; /* there is something to read */
+        }
     }
 
     return 0;
@@ -75,7 +75,6 @@ pkt_get_udp_port(const char *buf)
     return ntohs(udph->dest);
 }
 
-
 static void
 forward_pkts(struct nm_desc *src, struct nm_desc *dst)
 {
@@ -90,8 +89,8 @@ forward_pkts(struct nm_desc *src, struct nm_desc *dst)
 
         rxring = NETMAP_RXRING(src->nifp, si);
         txring = NETMAP_TXRING(dst->nifp, di);
-        nrx = nm_ring_space(rxring);
-        ntx = nm_ring_space(txring);
+        nrx    = nm_ring_space(rxring);
+        ntx    = nm_ring_space(txring);
         if (nrx == 0) {
             si++;
             continue;
@@ -101,25 +100,25 @@ forward_pkts(struct nm_desc *src, struct nm_desc *dst)
             continue;
         }
 
-	rxhead = rxring->head;
-	txhead = txring->head;
-	for (; nrx > 0 && ntx > 0;
-               nrx --, rxhead = nm_ring_next(rxring, rxhead), tot ++) {
+        rxhead = rxring->head;
+        txhead = txring->head;
+        for (; nrx > 0 && ntx > 0;
+             nrx--, rxhead = nm_ring_next(rxring, rxhead), tot++) {
             struct netmap_slot *rs = &rxring->slot[rxhead];
             struct netmap_slot *ts = &txring->slot[txhead];
-            char *rxbuf = NETMAP_BUF(rxring, rs->buf_idx);
-            char *txbuf = NETMAP_BUF(txring, ts->buf_idx);
+            char *rxbuf            = NETMAP_BUF(rxring, rs->buf_idx);
+            char *txbuf            = NETMAP_BUF(txring, ts->buf_idx);
 
             ts->len = rs->len;
             memcpy(txbuf, rxbuf, ts->len);
             txhead = nm_ring_next(txring, txhead);
-            ntx --;
-            fwdback ++;
-            tot ++;
+            ntx--;
+            fwdback++;
+            tot++;
         }
         /* Update state of netmap ring. */
-	rxring->head = rxring->cur = rxhead;
-	txring->head = txring->cur = txhead;
+        rxring->head = rxring->cur = rxhead;
+        txring->head = txring->cur = txhead;
     }
 }
 
@@ -134,11 +133,11 @@ main_loop(const char *netmap_port_one, const char *netmap_port_two,
     nmd_one = nm_open(netmap_port_one, NULL, 0, NULL);
     if (nmd_one == NULL) {
         if (!errno) {
-          printf("Failed to nm_open(%s): not a netmap port\n",
-                 netmap_port_one);
+            printf("Failed to nm_open(%s): not a netmap port\n",
+                   netmap_port_one);
         } else {
-          printf("Failed to nm_open(%s): %s\n", netmap_port_one,
-                 strerror(errno));
+            printf("Failed to nm_open(%s): %s\n", netmap_port_one,
+                   strerror(errno));
         }
         return -1;
     }
@@ -159,7 +158,7 @@ main_loop(const char *netmap_port_one, const char *netmap_port_two,
     if (nmd_three == NULL) {
         if (!errno) {
             printf("Failed to nm_open(%s): not a netmap port\n",
-                    netmap_port_three);
+                   netmap_port_three);
         } else {
             printf("Failed to nm_open(%s): %s\n", netmap_port_three,
                    strerror(errno));
@@ -168,7 +167,6 @@ main_loop(const char *netmap_port_one, const char *netmap_port_two,
     }
 
     while (!stop) {
-
         /* Forward traffic from ports two and three back to port one. */
         forward_pkts(nmd_two, nmd_one);
         forward_pkts(nmd_three, nmd_one);
@@ -191,61 +189,62 @@ usage(char **argv)
 {
     printf("usage: %s [-h] [-i NETMAP_PORT_ONE] "
            "[-i NETMAP_PORT_TWO] [-i NETMAP_PORT_THREE] "
-           "[-p UDP_PORT_A] [-p UDP_PORT_B]\n", argv[0]);
+           "[-p UDP_PORT_A] [-p UDP_PORT_B]\n",
+           argv[0]);
     exit(EXIT_SUCCESS);
 }
 
 int
 main(int argc, char **argv)
 {
-    const char *netmap_port_one = NULL;
-    const char *netmap_port_two = NULL;
+    const char *netmap_port_one   = NULL;
+    const char *netmap_port_two   = NULL;
     const char *netmap_port_three = NULL;
     int udp_port;
-    int udp_port_a = 8000;
-    int udp_port_b = 8001;
+    int udp_port_a    = 8000;
+    int udp_port_b    = 8001;
     int udp_port_args = 0;
     struct sigaction sa;
     int opt;
     int ret;
 
     while ((opt = getopt(argc, argv, "hi:p:")) != -1) {
         switch (opt) {
-            case 'h':
+        case 'h':
+            usage(argv);
+            return 0;
+
+        case 'i':
+            if (netmap_port_one == NULL) {
+                netmap_port_one = optarg;
+            } else if (netmap_port_two == NULL) {
+                netmap_port_two = optarg;
+            } else if (netmap_port_three == NULL) {
+                netmap_port_three = optarg;
+            }
+            break;
+
+        case 'p':
+            udp_port = atoi(optarg);
+            if (udp_port <= 0 || udp_port >= 65535) {
+                printf("    invalid UDP port %s\n", optarg);
                 usage(argv);
-                return 0;
-
-            case 'i':
-                if (netmap_port_one == NULL) {
-                    netmap_port_one = optarg;
-                } else if (netmap_port_two == NULL) {
-                    netmap_port_two = optarg;
-                } else if (netmap_port_three == NULL) {
-                    netmap_port_three = optarg;
-                }
+            }
+            switch (udp_port_args) {
+            case 0:
+                udp_port_a = udp_port;
                 break;
-
-            case 'p':
-                udp_port = atoi(optarg);
-                if (udp_port <= 0 || udp_port >= 65535) {
-                    printf("    invalid UDP port %s\n", optarg);
-                    usage(argv);
-                }
-                switch (udp_port_args) {
-                case 0:
-                    udp_port_a = udp_port;
-                    break;
-                case 1:
-                    udp_port_b = udp_port;
-                    break;
-                }
-                udp_port_args ++;
+            case 1:
+                udp_port_b = udp_port;
                 break;
+            }
+            udp_port_args++;
+            break;
 
-            default:
-                printf("    unrecognized option '-%c'\n", opt);
-                usage(argv);
-                return -1;
+        default:
+            printf("    unrecognized option '-%c'\n", opt);
+            usage(argv);
+            return -1;
         }
     }
 
@@ -263,7 +262,7 @@ main(int argc, char **argv)
     sa.sa_handler = sigint_handler;
     sigemptyset(&sa.sa_mask);
     sa.sa_flags = SA_RESTART;
-    ret = sigaction(SIGINT, &sa, NULL);
+    ret         = sigaction(SIGINT, &sa, NULL);
     if (ret) {
         perror("sigaction(SIGINT)");
         exit(EXIT_FAILURE);
@@ -276,8 +275,8 @@ main(int argc, char **argv)
     printf("UDP port A: %d\n", udp_port_a);
     printf("UDP port B: %d\n", udp_port_b);
 
-    main_loop(netmap_port_one, netmap_port_two, netmap_port_three,
-              udp_port_a, udp_port_b);
+    main_loop(netmap_port_one, netmap_port_two, netmap_port_three, udp_port_a,
+              udp_port_b);
 
     return 0;
 }