This patch makes the internal `clock-snapshot-set.h` require what it
needs instead of using the macros only if they exist. This makes it
impossible to skip logging in this header.
Signed-off-by: Philippe Proulx <eeppeliteloop@gmail.com>
Change-Id: Ib8b3def2869ab214a3d38e3a1cea7b9954fc056c
Reviewed-on: https://review.lttng.org/c/babeltrace/+/1578
Tested-by: jenkins <jenkins@lttng.org>
Reviewed-by: Simon Marchi <simon.marchi@efficios.com>
#include "clock-snapshot.h"
#include "clock-class.h"
#include "clock-snapshot.h"
#include "clock-class.h"
+/* Protection: this file uses BT_LIB_LOG*() macros directly */
+#ifndef BT_LIB_LOG_SUPPORTED
+# error Please include "lib/logging.h" before including this file.
+#endif
+
struct bt_clock_snapshot_set {
/* Unique objects owned by this */
GPtrArray *clock_snapshots;
struct bt_clock_snapshot_set {
/* Unique objects owned by this */
GPtrArray *clock_snapshots;
cs_set->clock_snapshots = g_ptr_array_sized_new(1);
if (!cs_set->clock_snapshots) {
cs_set->clock_snapshots = g_ptr_array_sized_new(1);
if (!cs_set->clock_snapshots) {
BT_LOGE_STR("Failed to allocate one GPtrArray.");
BT_LOGE_STR("Failed to allocate one GPtrArray.");
if (!clock_snapshot) {
clock_snapshot = bt_clock_snapshot_create(cc);
if (!clock_snapshot) {
if (!clock_snapshot) {
clock_snapshot = bt_clock_snapshot_create(cc);
if (!clock_snapshot) {
BT_LIB_LOGE("Cannot create a clock snapshot from a clock class: "
"%![cc-]+K", cc);
BT_LIB_LOGE("Cannot create a clock snapshot from a clock class: "
"%![cc-]+K", cc);