Style fixes (#505)

Ran `clang-format` on all module and core files using our
`.clang-format` file.

Author: vivi

core/bessctl.cc

@@ -1593,7 +1593,7 @@ class BESSControlImpl final : public BESSControl::Service {
 
 bool ApiServer::grpc_cb_set_ = false;
 
-void ApiServer::Listen(const std::string &host, int port) {
+void ApiServer::Listen(const std::string& host, int port) {
   if (!builder_) {
     builder_ = new grpc::ServerBuilder();
   }

core/bessctl.h

@@ -4,7 +4,7 @@
 #include <string>
 
 namespace grpc {
-  class ServerBuilder;
+class ServerBuilder;
 }  // namespace grpc
 
 // gRPC server encapsulation. Usage:
@@ -17,8 +17,8 @@ class ApiServer {
   ApiServer() : builder_(nullptr) {}
 
   // This class is neither copyable nor movable.
-  ApiServer(const ApiServer&) = delete;
-  ApiServer& operator=(const ApiServer&) = delete;
+  ApiServer(const ApiServer &) = delete;
+  ApiServer &operator=(const ApiServer &) = delete;
 
   // Adds a host:port pair. host can be an ipv6 address.
   void Listen(const std::string &host, int port);

core/dpdk.cc

@@ -157,7 +157,7 @@ static int determine_default_core() {
   int ret = pthread_getaffinity_np(pthread_self(), sizeof(set), &set);
   if (ret < 0) {
     PLOG(WARNING) << "pthread_getaffinity_np()";
-    return 0; // Core 0 as a fallback
+    return 0;  // Core 0 as a fallback
   }
 
   // Choose the last core available

core/module.h

@@ -29,7 +29,6 @@ struct task_result {
 typedef uint16_t task_id_t;
 typedef uint64_t placement_constraint;
 
-
 using module_cmd_func_t =
     pb_func_t<CommandResponse, Module, google::protobuf::Any>;
 using module_init_func_t =

core/module_bench.cc

@@ -33,7 +33,7 @@ class DummySourceModule : public Module {
 
   RunNextModule(&batch);
 
-  return { .block = false, .packets = batch_size, .bits = 0 };
+  return {.block = false, .packets = batch_size, .bits = 0};
 }
 
 class DummyRelayModule : public Module {

core/modules/arp_responder.h

@@ -16,7 +16,7 @@ using bess::utils::be32_t;
 struct arp_entry {
   Ethernet::Address mac_addr;
   be32_t ip_addr;
-  uint64_t time; // timestamp used to expire cache entries (in milliseconds)
+  uint64_t time;  // timestamp used to expire cache entries (in milliseconds)
 };
 
 // ARP Responder module

core/modules/drr.cc

@@ -131,7 +131,7 @@ struct task_result DRR::RunTask(void*) {
   // the number of bits inserted into the packet batch
   uint32_t cnt = batch.cnt();
   uint64_t bits_retrieved = (total_bytes + cnt * kPacketOverhead) * 8;
-  return { .block = (cnt == 0), .packets = cnt, .bits = bits_retrieved };
+  return {.block = (cnt == 0), .packets = cnt, .bits = bits_retrieved};
 }
 
 uint32_t DRR::GetNextBatch(bess::PacketBatch* batch, int* err) {

core/modules/flowgen.cc

@@ -486,11 +486,9 @@ struct task_result FlowGen::RunTask(void *) {
   RunNextModule(&batch);
 
   uint32_t cnt = batch.cnt();
-  return {
-    .block = (cnt == 0),
-    .packets = cnt,
-    .bits = ((template_size_ + pkt_overhead) * cnt) * 8
-  };
+  return {.block = (cnt == 0),
+          .packets = cnt,
+          .bits = ((template_size_ + pkt_overhead) * cnt) * 8};
 }
 
 std::string FlowGen::GetDesc() const {

core/modules/nat.cc

@@ -125,7 +125,7 @@ NAT::HashTable::Entry *NAT::CreateNewEntry(const Endpoint &src_internal,
     src_external.port = be16_t(port);
     auto *hash_reverse = map_.Find(src_external);
     if (hash_reverse == nullptr) {
-found:
+    found:
       // Found an available src_internal <-> src_external mapping
       NatEntry forward_entry;
       NatEntry reverse_entry;

core/modules/noop.cc

@@ -11,7 +11,7 @@ CommandResponse NoOP::Init(const bess::pb::EmptyArg &) {
 }
 
 struct task_result NoOP::RunTask(void *) {
-  return { .block = false, .packets = 0, .bits = 0 };
+  return {.block = false, .packets = 0, .bits = 0};
 }
 
 ADD_MODULE(NoOP, "noop", "creates a task that does nothing")