Remove unused StatisticsStatus values
authorAndrey Alekseenko <al42and@gmail.com>
Tue, 18 May 2021 11:41:49 +0000 (14:41 +0300)
committerAndrey Alekseenko <al42and@gmail.com>
Tue, 18 May 2021 11:41:49 +0000 (14:41 +0300)
The `enumValueToString(StatisticsStatus)` had the wrong order of
strings. But in fact most of these statuses were not used anyway, so we
can just remove the corresponding enum values altogether.

src/gromacs/statistics/statistics.cpp
src/gromacs/statistics/statistics.h

index 38ccc006e770863d3591c69aa0c8d029efd62dc2..6f5fc87e2e0e3a348b0f7cb7336f40371b9a42fb 100644 (file)
@@ -326,8 +326,7 @@ StatisticsStatus gmx_stats_get_ase(gmx_stats_t gstats, real* aver, real* sigma,
 static const char* enumValueToString(StatisticsStatus enumValue)
 {
     constexpr gmx::EnumerationArray<StatisticsStatus, const char*> statisticsStatusNames = {
-        "All well in STATS land",  "No points",     "Not enough memory",
-        "Invalid histogram input", "Unknown error", "Not implemented yet"
+        "All well in STATS land", "No points"
     };
     return statisticsStatusNames[enumValue];
 }
index bfe0354af94d9b3c1319ecabf2db072997f15d59..8cf2db947d17a58135bd5672c1bc57e88eac4082 100644 (file)
@@ -56,10 +56,6 @@ enum class StatisticsStatus : int
 {
     Ok,
     NoPoints,
-    NoMemory,
-    Error,
-    InvalidInput,
-    NotImplemented,
     Count
 };