display category name

Author: Goober5000

code/parse/sexp.cpp

@@ -32318,6 +32318,21 @@ int get_category(int sexp_id)
 	return category;
 }
 
+// Goober5000
+const char *get_category_name(int category_id)
+{
+	if (category_id == OP_CATEGORY_CHANGE2)
+		category_id = OP_CATEGORY_CHANGE;
+
+	for (auto &category : op_menu)
+	{
+		if (category.id == category_id)
+			return category.name.c_str();
+	}
+
+	return "<unknown>";
+}
+
 // Goober5000
 int category_of_subcategory(int subcategory_id)
 {

code/parse/sexp.h

@@ -1359,6 +1359,7 @@ void set_turret_secondary_ammo(ship_subsys *turret, int requested_bank, int requ
 extern int get_category(int sexp_id);
 extern int category_of_subcategory(int subcategory_id);
 extern int get_subcategory(int sexp_id);
+extern const char *get_category_name(int category_id);
 
 // Goober5000
 extern void sexp_music_close();

code/parse/sexp/sexp_lookup.cpp

@@ -181,12 +181,12 @@ int add_dynamic_sexp(std::unique_ptr<DynamicSEXP>&& sexp, int type)
 
 	int free_op_index = get_next_free_operator(sexp->getCategory());
 	if (free_op_index >= 256) {
-		Warning(LOCATION, "There are too many SEXPs in the category %d.  The %s SEXP will not be added.", sexp->getCategory(), sexp->getName().c_str());
+		Warning(LOCATION, "There are too many SEXPs in the %s category.  The SEXP %s will not be added.", get_category_name(sexp->getCategory()), sexp->getName().c_str());
 		return -1;
 	}
 
 	if (Operators.size() >= FIRST_OP) {
-		Warning(LOCATION, "There are too many total SEXPs.  The %s SEXP will not be added.", sexp->getName().c_str());
+		Warning(LOCATION, "There are too many total SEXPs.  The SEXP %s will not be added.", sexp->getName().c_str());
 		return -1;
 	}