diff options
author | Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2019-05-10 19:51:43 +0200 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2019-05-12 12:33:55 -0700 |
commit | 75767c456a2889d213621866a9f7fbb108f3366d (patch) | |
tree | 1f941f73a58718affe6c199c510e08e0eec75f95 /core/applicationstate.h | |
parent | 114b3d9d470e65c25bcb5711daf053478f30e319 (diff) | |
download | subsurface-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.h | 21 |
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 |