Do some pedantic spring cleaning

- Use const in more places where it makes sense - not absolutely
  everywhere because it can get a bit annoying

- Make all the instruction search loops a bit more readable by casting
  the function pointer into a temporary variable to loop over

- Add a few more doc comments and fix a typo or two

- Make that RTTI thing flexibly-sized, finally

- Don't include gamedata.h in vcall.h for no reason; consequently
  include gamedata.h in a bunch of places where it was implictly pulled
  in before

- Fix dbg_toghidra() and ent_getedict() having mismatched return types
  between their headers and respective source files

- Remove that one broken, hacky, secret Portal non-feature that probably
  nobody even ended up using; it can be implemented properly later if
  required

1 files changed, 1 insertions, 1 deletions

diff --git a/src/con_.h b/src/con_.h
index 6d67e51..dcba4d9 100644
--- a/src/con_.h
+++ b/src/con_.h
@@ -213,7 +213,7 @@ extern void *_con_vtab_cmd[];
 // a constant expression we have to use a macro.
 extern struct _con_vtab_var_wrap {
 #ifdef _WIN32
-	struct msvc_rtti_locator *rtti;
+	const struct msvc_rtti_locator *rtti;
 #else
 	// itanium ABI also has the top offset/"whole object" offset in libstdc++
 	ssize topoffset;