Firstly evaluated that assert, it's benign as far as I can tell. A hangover from an earlier way of getting data to/from the patch when I used to pass the GeoPatches pointer around with the request.
It did lead me to a couple of cases though where things weren't being cleaned up if finding a GeoSphere or GeoPatch should fail.
Also I've added some more sanity checking.
replace NULL with plain old 0 as the C++ gods intended
pull a bunch of game constants out of libs.h and give them proper definitions
SIZE_FMT is not a build option
remove DEG/RAD constants, only use the conversion functions
move PIONEER_DATA_DIR down to the only place where its actually used
remove pi_load_lua and CurrentDirectory
remove RefList, its unused
remove BezierCurve, also unused
initialise rmb flag
silence faction debug
Better internationalization of the joystick binding string
Provide a translation of JOY_AXIS to Spanish
Delete AXIS (now unused) from all languages
Update Czech.txt

* new token; SHA: c03a5ca3e97067f82d7cbc71072f298674f6d38b
Merge remote-tracking branch 'robn/minor-cleanups'
Factions.cpp: Silence unused variable warning
LuaTable: Add a m_clean attribute to autoclean the table in ~LuaTable
PropertyMap: Just wrap Get/Set around LuaTable calls
Merge remote-tracking branch 'valin/czech_lang'