From 77a99cc61d7a39745192354594086bb90ffb0b50 Mon Sep 17 00:00:00 2001
From: Max Kellermann <max@duempel.org>
Date: Fri, 4 Jan 2013 01:10:48 +0100
Subject: CommandListBuilder: rename attributes

---
 src/CommandListBuilder.hxx | 43 ++++++++++++++++++++++++++++---------------
 1 file changed, 28 insertions(+), 15 deletions(-)

(limited to 'src/CommandListBuilder.hxx')

diff --git a/src/CommandListBuilder.hxx b/src/CommandListBuilder.hxx
index dbcc15dc..cc9e7b15 100644
--- a/src/CommandListBuilder.hxx
+++ b/src/CommandListBuilder.hxx
@@ -25,23 +25,38 @@
 
 class CommandListBuilder {
 	/**
-	 * for when in list mode
+	 * print OK after each command execution
 	 */
-	GSList *cmd_list;
+	enum class Mode {
+		/**
+		 * Not active.
+		 */
+		DISABLED = -1,
+
+		/**
+		 * Enabled in normal list mode.
+		 */
+		ENABLED = false,
+
+		/**
+		 * Enabled in "list_OK" mode.
+		 */
+		OK = true,
+	} mode;
 
 	/**
-	 * print OK after each command execution
+	 * for when in list mode
 	 */
-	int cmd_list_OK;
+	GSList *list;
 
 	/**
-	 * mem cmd_list consumes
+	 * Memory consumed by the list.
 	 */
-	size_t cmd_list_size;
+	size_t size;
 
 public:
 	CommandListBuilder()
-		:cmd_list(nullptr), cmd_list_OK(-1), cmd_list_size(0) {}
+		:mode(Mode::DISABLED), list(nullptr), size(0) {}
 	~CommandListBuilder() {
 		Reset();
 	}
@@ -50,9 +65,7 @@ public:
 	 * Is a command list currently being built?
 	 */
 	bool IsActive() const {
-		assert(cmd_list_OK >= -1 && cmd_list_OK <= 1);
-
-		return cmd_list_OK >= 0;
+		return mode != Mode::DISABLED;
 	}
 
 	/**
@@ -61,7 +74,7 @@ public:
 	bool IsOKMode() const {
 		assert(IsActive());
 
-		return (bool)cmd_list_OK;
+		return (bool)mode;
 	}
 
 	/**
@@ -73,10 +86,10 @@ public:
 	 * Begin building a command list.
 	 */
 	void Begin(bool ok) {
-		assert(cmd_list == nullptr);
-		assert(cmd_list_OK == -1);
+		assert(list == nullptr);
+		assert(mode == Mode::DISABLED);
 
-		cmd_list_OK = (int)ok;
+		mode = (Mode)ok;
 	}
 
 	/**
@@ -93,7 +106,7 @@ public:
 		/* for scalability reasons, we have prepended each new
 		   command; now we have to reverse it to restore the
 		   correct order */
-		return cmd_list = g_slist_reverse(cmd_list);
+		return list = g_slist_reverse(list);
 	}
 };
 
-- 
cgit v1.2.3