From: Francis Deslauriers <francis.deslauriers@efficios.com>
Date: Thu, 2 Apr 2020 20:16:59 +0000 (-0400)
Subject: lttng-view: clean-up: rename `parse_options()` -> `parse_viewer_option()`
X-Git-Tag: v2.12.0~12
X-Git-Url: https://git.lttng.org./?a=commitdiff_plain;h=6651a61fbc1dd70c9893a3bd1c963afebf1db563;p=lttng-tools.git

lttng-view: clean-up: rename `parse_options()` -> `parse_viewer_option()`

This seems more representative of what this function does.

Signed-off-by: Francis Deslauriers <francis.deslauriers@efficios.com>
Change-Id: I78e4282a13048f4a35df4ddeae91b89969b73941
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
---

diff --git a/src/bin/lttng/commands/view.c b/src/bin/lttng/commands/view.c
index 232638963..3ba5766ea 100644
--- a/src/bin/lttng/commands/view.c
+++ b/src/bin/lttng/commands/view.c
@@ -70,10 +70,10 @@ static const struct viewer {
 /* Is the session we are trying to view is in live mode. */
 static int session_live_mode;
 
-static const struct viewer *parse_options(void)
+static const struct viewer *parse_viewer_option(void)
 {
 	if (opt_viewer == NULL) {
-		/* Default is babeltrace */
+		/* Default is babeltrace2 */
 		return &(viewers[VIEWER_BABELTRACE2]);
 	}
 
@@ -195,8 +195,8 @@ static int spawn_viewer(const char *trace_path)
 	const struct viewer *viewer;
 	char **argv = NULL;
 
-	/* Check for --viewer options */
-	viewer = parse_options();
+	/* Check for --viewer option. */
+	viewer = parse_viewer_option();
 	if (viewer == NULL) {
 		ret = CMD_ERROR;
 		goto error;