.clang-tidy: enable cppcoreguidelines-avoid-const-or-ref-data-members

[babeltrace.git] / src / plugins / ctf / common / metadata / ctf-meta-update-stream-class-config.cpp

diff --git a/src/plugins/ctf/common/metadata/ctf-meta-update-stream-class-config.cpp b/src/plugins/ctf/common/metadata/ctf-meta-update-stream-class-config.cpp
index b3e0d6e06158f8d013f3939bdf423d66e04b7a4e..a8f5add6dbd989b8e3bf8d18fc279e5c0c0a571e 100644 (file)
--- a/src/plugins/ctf/common/metadata/ctf-meta-update-stream-class-config.cpp
+++ b/src/plugins/ctf/common/metadata/ctf-meta-update-stream-class-config.cpp
@@ -4,69 +4,55 @@
  * Copyright 2019 Philippe Proulx <pproulx@efficios.com>
  */
 
-#include <babeltrace2/babeltrace.h>
-#include "common/macros.h"
-#include "common/assert.h"
-#include <glib.h>
 #include <stdint.h>
-#include <string.h>
-#include <inttypes.h>
 
 #include "ctf-meta-visitors.hpp"
 
-BT_HIDDEN
 int ctf_trace_class_update_stream_class_config(struct ctf_trace_class *ctf_tc)
 {
-       struct ctf_field_class_int *int_fc;
-       uint64_t i;
-
-       for (i = 0; i < ctf_tc->stream_classes->len; i++) {
-               struct ctf_stream_class *sc =
-                       (ctf_stream_class *) ctf_tc->stream_classes->pdata[i];
-
-               if (sc->is_translated) {
-                       continue;
-               }
-
-               if (!sc->packet_context_fc) {
-                       continue;
-               }
-
-               int_fc = ctf_field_class_struct_borrow_member_int_field_class_by_name(
-                       ctf_field_class_as_struct(sc->packet_context_fc), "timestamp_begin");
-               if (int_fc && int_fc->meaning ==
-                               CTF_FIELD_CLASS_MEANING_PACKET_BEGINNING_TIME) {
-                       sc->packets_have_ts_begin = true;
-               }
-
-               int_fc = ctf_field_class_struct_borrow_member_int_field_class_by_name(
-                       ctf_field_class_as_struct(sc->packet_context_fc), "timestamp_end");
-               if (int_fc && int_fc->meaning ==
-                               CTF_FIELD_CLASS_MEANING_PACKET_END_TIME) {
-                       sc->packets_have_ts_end = true;
-               }
-
-               int_fc = ctf_field_class_struct_borrow_member_int_field_class_by_name(
-                       ctf_field_class_as_struct(sc->packet_context_fc), "events_discarded");
-               if (int_fc && int_fc->meaning ==
-                               CTF_FIELD_CLASS_MEANING_DISC_EV_REC_COUNTER_SNAPSHOT) {
-                       sc->has_discarded_events = true;
-               }
-
-               sc->discarded_events_have_default_cs =
-                       sc->has_discarded_events && sc->packets_have_ts_begin &&
-                       sc->packets_have_ts_end;
-               int_fc = ctf_field_class_struct_borrow_member_int_field_class_by_name(
-                       ctf_field_class_as_struct(sc->packet_context_fc), "packet_seq_num");
-               if (int_fc && int_fc->meaning ==
-                               CTF_FIELD_CLASS_MEANING_PACKET_COUNTER_SNAPSHOT) {
-                       sc->has_discarded_packets = true;
-               }
-
-               sc->discarded_packets_have_default_cs =
-                       sc->has_discarded_packets &&
-                       sc->packets_have_ts_begin && sc->packets_have_ts_end;
-       }
-
-       return 0;
+    struct ctf_field_class_int *int_fc;
+    uint64_t i;
+
+    for (i = 0; i < ctf_tc->stream_classes->len; i++) {
+        struct ctf_stream_class *sc = (ctf_stream_class *) ctf_tc->stream_classes->pdata[i];
+
+        if (sc->is_translated) {
+            continue;
+        }
+
+        if (!sc->packet_context_fc) {
+            continue;
+        }
+
+        int_fc = ctf_field_class_struct_borrow_member_int_field_class_by_name(
+            ctf_field_class_as_struct(sc->packet_context_fc), "timestamp_begin");
+        if (int_fc && int_fc->meaning == CTF_FIELD_CLASS_MEANING_PACKET_BEGINNING_TIME) {
+            sc->packets_have_ts_begin = true;
+        }
+
+        int_fc = ctf_field_class_struct_borrow_member_int_field_class_by_name(
+            ctf_field_class_as_struct(sc->packet_context_fc), "timestamp_end");
+        if (int_fc && int_fc->meaning == CTF_FIELD_CLASS_MEANING_PACKET_END_TIME) {
+            sc->packets_have_ts_end = true;
+        }
+
+        int_fc = ctf_field_class_struct_borrow_member_int_field_class_by_name(
+            ctf_field_class_as_struct(sc->packet_context_fc), "events_discarded");
+        if (int_fc && int_fc->meaning == CTF_FIELD_CLASS_MEANING_DISC_EV_REC_COUNTER_SNAPSHOT) {
+            sc->has_discarded_events = true;
+        }
+
+        sc->discarded_events_have_default_cs =
+            sc->has_discarded_events && sc->packets_have_ts_begin && sc->packets_have_ts_end;
+        int_fc = ctf_field_class_struct_borrow_member_int_field_class_by_name(
+            ctf_field_class_as_struct(sc->packet_context_fc), "packet_seq_num");
+        if (int_fc && int_fc->meaning == CTF_FIELD_CLASS_MEANING_PACKET_COUNTER_SNAPSHOT) {
+            sc->has_discarded_packets = true;
+        }
+
+        sc->discarded_packets_have_default_cs =
+            sc->has_discarded_packets && sc->packets_have_ts_begin && sc->packets_have_ts_end;
+    }
+
+    return 0;
 }