Make ClientInterface::statenames consistent with the state enum again

est31 · est31 · commit 2c637ce1c261 · 2015-08-31T13:31:29.000+02:00
Fixes minetest.get_player_information segfault due
to out of bounds access problems, when compiled as debug build.
diff --git a/src/clientiface.cpp b/src/clientiface.cpp
@@ -38,10 +38,12 @@ const char *ClientInterface::statenames[] = {
 	"Disconnecting",
 	"Denied",
 	"Created",
-	"InitSent",
+	"AwaitingInit2",
+	"HelloSent",
 	"InitDone",
 	"DefinitionsSent",
-	"Active"
+	"Active",
+	"SudoMode",
 };
 
 
diff --git a/src/clientiface.h b/src/clientiface.h
@@ -167,6 +167,9 @@ namespace con {
 
 #define CI_ARRAYSIZE(a) (sizeof(a) / sizeof((a)[0]))
 
+// Also make sure to update the ClientInterface::statenames
+// array when modifying these enums
+
 enum ClientState
 {
 	CS_Invalid,

Original file line number	Diff line number	Diff line change
`@@ -167,6 +167,9 @@ namespace con {`
`167`	`167`
`168`	`168`	`#define CI_ARRAYSIZE(a) (sizeof(a) / sizeof((a)[0]))`
`169`	`169`
	`170`	`+// Also make sure to update the ClientInterface::statenames`
	`171`	`+// array when modifying these enums`
	`172`	`+`
`170`	`173`	`enum ClientState`
`171`	`174`	`{`
`172`	`175`	`CS_Invalid,`