X-Git-Url: https://git.lttng.org/?a=blobdiff_plain;f=src%2Fcommon%2Fspawn-viewer.cpp;h=e3aeb8314858badb81ab71684489025e481d6475;hb=28ab034a2c3582d07d3423d2d746731f87d3969f;hp=288e3a2ae6b60beb404bf6248e509d9370d6aef1;hpb=64803277bbdbe0a943360d918298a48157d9da55;p=lttng-tools.git

diff --git a/src/common/spawn-viewer.cpp b/src/common/spawn-viewer.cpp
index 288e3a2ae..e3aeb8314 100644
--- a/src/common/spawn-viewer.cpp
+++ b/src/common/spawn-viewer.cpp
@@ -7,39 +7,40 @@
  *
  */
 
-#include <stdbool.h>
-#include <sys/stat.h>
-#include <sys/types.h>
-#include <unistd.h>
-
-#include <lttng/constant.h>
-
-#include <common/compat/errno.hpp>
 #include "error.hpp"
 #include "macros.hpp"
 #include "spawn-viewer.hpp"
 
+#include <common/compat/errno.hpp>
 
-static const char *babeltrace_bin = CONFIG_BABELTRACE_BIN;
-static const char *babeltrace2_bin = CONFIG_BABELTRACE2_BIN;
+#include <lttng/constant.h>
 
-/*
- * This is needed for each viewer since we are using execvp().
- */
-static const char *babeltrace_opts[] = { "babeltrace" };
-static const char *babeltrace2_opts[] = { "babeltrace2" };
+#include <stdbool.h>
+#include <sys/stat.h>
+#include <sys/types.h>
+#include <unistd.h>
 
 /*
  * Type is also use as the index in the viewers array. So please, make sure
  * your enum value is in the right order in the array below.
  */
 enum viewer_type {
-	VIEWER_BABELTRACE    = 0,
-	VIEWER_BABELTRACE2   = 1,
-	VIEWER_USER_DEFINED  = 2,
+	VIEWER_BABELTRACE = 0,
+	VIEWER_BABELTRACE2 = 1,
+	VIEWER_USER_DEFINED = 2,
 };
 
-static const struct viewer {
+namespace {
+const char *babeltrace_bin = CONFIG_BABELTRACE_BIN;
+const char *babeltrace2_bin = CONFIG_BABELTRACE2_BIN;
+
+/*
+ * This is needed for each viewer since we are using execvp().
+ */
+const char *babeltrace_opts[] = { "babeltrace" };
+const char *babeltrace2_opts[] = { "babeltrace2" };
+
+const struct viewer {
 	const char *exec_name;
 	enum viewer_type type;
 } viewers[] = {
@@ -47,6 +48,7 @@ static const struct viewer {
 	{ "babeltrace2", VIEWER_BABELTRACE2 },
 	{ NULL, VIEWER_USER_DEFINED },
 };
+} /* namespace */
 
 static const struct viewer *parse_viewer_option(const char *opt_viewer)
 {
@@ -122,8 +124,10 @@ error:
  *
  * The returning pointer is ready to be passed to execvp().
  */
-static char **alloc_argv_from_local_opts(const char **opts, size_t opts_len,
-		const char *trace_path, bool opt_live_mode)
+static char **alloc_argv_from_local_opts(const char **opts,
+					 size_t opts_len,
+					 const char *trace_path,
+					 bool opt_live_mode)
 {
 	char **argv;
 	size_t mem_len;
@@ -159,7 +163,6 @@ error:
 	return argv;
 }
 
-
 /*
  * Spawn viewer with the trace directory path.
  */
@@ -186,9 +189,8 @@ retry_viewer:
 		} else {
 			viewer_bin = viewer->exec_name;
 		}
-		argv = alloc_argv_from_local_opts(babeltrace2_opts,
-				ARRAY_SIZE(babeltrace2_opts), trace_path,
-				opt_live_mode);
+		argv = alloc_argv_from_local_opts(
+			babeltrace2_opts, ARRAY_SIZE(babeltrace2_opts), trace_path, opt_live_mode);
 		break;
 	case VIEWER_BABELTRACE:
 		if (stat(babeltrace_bin, &status) == 0) {
@@ -196,9 +198,8 @@ retry_viewer:
 		} else {
 			viewer_bin = viewer->exec_name;
 		}
-		argv = alloc_argv_from_local_opts(babeltrace_opts,
-				ARRAY_SIZE(babeltrace_opts), trace_path,
-				opt_live_mode);
+		argv = alloc_argv_from_local_opts(
+			babeltrace_opts, ARRAY_SIZE(babeltrace_opts), trace_path, opt_live_mode);
 		break;
 	case VIEWER_USER_DEFINED:
 		argv = alloc_argv_from_user_opts(opt_viewer, trace_path);
@@ -223,16 +224,16 @@ retry_viewer:
 			if (viewer->type == VIEWER_BABELTRACE2) {
 				/* Fallback to legacy babeltrace. */
 				DBG("Default viewer \"%s\" not installed on the system, falling back to \"%s\"",
-						viewers[VIEWER_BABELTRACE2].exec_name,
-						viewers[VIEWER_BABELTRACE].exec_name);
+				    viewers[VIEWER_BABELTRACE2].exec_name,
+				    viewers[VIEWER_BABELTRACE].exec_name);
 				viewer = &viewers[VIEWER_BABELTRACE];
 				free(argv);
 				argv = NULL;
 				goto retry_viewer;
 			} else {
 				ERR("Default viewer \"%s\" (and fallback \"%s\") not found on the system",
-						viewers[VIEWER_BABELTRACE2].exec_name,
-						viewers[VIEWER_BABELTRACE].exec_name);
+				    viewers[VIEWER_BABELTRACE2].exec_name,
+				    viewers[VIEWER_BABELTRACE].exec_name);
 			}
 		} else {
 			PERROR("Failed to launch \"%s\" viewer", viewer_bin);