summaryrefslogtreecommitdiffstats
path: root/core/applicationstate.h
diff options
context:
space:
mode:
authorGravatar Berthold Stoeger <bstoeger@mail.tuwien.ac.at>2019-05-10 19:51:43 +0200
committerGravatar Dirk Hohndel <dirk@hohndel.org>2019-05-12 12:33:55 -0700
commit75767c456a2889d213621866a9f7fbb108f3366d (patch)
tree1f941f73a58718affe6c199c510e08e0eec75f95 /core/applicationstate.h
parent114b3d9d470e65c25bcb5711daf053478f30e319 (diff)
downloadsubsurface-75767c456a2889d213621866a9f7fbb108f3366d.tar.gz
Turn application state into enum
The application state was encoded in a QByteArray. Thus, there was no compile-time checking. Typos would lead to silent failures. Turn the application state into an enum. Use the enum-class construct, so that the values don't polute the global namespace. Moreover, this makes them strongly typed, i.e. they don't auto-convert to integers. A disadvantage is that the enums now have to be cast to int explicitly when used to index an array. Replace two hash-maps in MainWindow to arrays of fixed sizes. Move the application-state details into their own files. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'core/applicationstate.h')
-rw-r--r--core/applicationstate.h21
1 files changed, 21 insertions, 0 deletions
diff --git a/core/applicationstate.h b/core/applicationstate.h
new file mode 100644
index 000000000..fbaf3e829
--- /dev/null
+++ b/core/applicationstate.h
@@ -0,0 +1,21 @@
+// SPDX-License-Identifier: GPL-2.0
+#ifndef APPLICATIONSTATE_H
+#define APPLICATIONSTATE_H
+
+// By using an enum class, the enum entries don't polute the global namespace.
+// Moreover, they are strongly typed. This means that they are not auto-converted
+// to integer types if e.g. used as array-indices.
+enum class ApplicationState {
+ Default,
+ EditDive,
+ PlanDive,
+ EditPlannedDive,
+ EditDiveSite,
+ FilterDive,
+ Count
+};
+
+ApplicationState getAppState();
+void setAppState(ApplicationState state);
+
+#endif