Addresses GitHub issues themuffinator#78, themuffinator#79, themuffinator#82, themuffinator#87, themuffinator#88 and themuffinator#89. Startup failures exited silently (themuffinator#79). idCommonLocal::Error's recursive fatal branch called Sys_Quit(), which reports EXIT_SUCCESS and prints nothing, so the original message never reached the log and the shell saw a clean exit. What tripped it: idVertexCache::PurgeAll ran before Init() linked its list sentinels and handed ActuallyFree() a NULL block, raising a second error while the first was still unwinding. Both are fixed, and the glConfig.isInitialized guard at the call site is gone - it was set before vertexCache.Init() ran and cleared by ShutdownOpenGL() without freeing the cache, so it was wrong in both directions. The four startup content checks now print the resolved paths and the pk4 files actually found, which is what identifies the common Linux cause of a case-mismatched directory or pk4 name copied from a Windows install. Accented text rendered as two wrong glyphs (themuffinator#89). The string tables are UTF-8 while the stock fonts are a fixed 256-glyph atlas indexed by a raw byte, so "MENUS" drew as "MENA S". idLangDict::Load now converts a UTF-8 table to the font codepage, folding the typographic characters that land on a zero-advance .notdef cell to ASCII - a non-breaking space would otherwise delete the word gap rather than merely draw nothing. CP1252 and ASCII tables are left byte-identical, so retail localized packs are unaffected. This also fixes mojibake apostrophes in English subtitles. DDS replacements were rejected without explanation (themuffinator#82). ImageTools_SetCompressionCaps had a single caller, and imagetools is a static library linked separately into the engine and each renderer module, so depending on the active renderer the capability block stayed zero-initialized and every DDS was refused. Caps are now published from both backends and mirrored into the engine's copy. User replacements are probed ahead of the retail progimg/ tree instead of behind it, BC7 normal maps use the ordinary XYZ-in-RGB layout rather than being forced into the DXT5nm alpha convention, the ultra preset no longer zeroes the cvar and discards an archived setting, and a miss now logs every candidate path with the reason it was rejected. Sound lookup failures were invisible (themuffinator#87). A sample that resolved to nothing was replaced by the default with no warning. The engine now reports the sample and every path probed, voice-over falls back to English when a language ships text but no voice track as the original game did, and s_showVoices prints its table instead of building one every frame and discarding it. The reported symptom is not yet root-caused; these are the confirmed defects on that path. Scripted characters could stall a map permanently (themuffinator#88). idAASFileLocal::FinishAreas never derived aasArea_t::ceiling, so AreaCeiling() returned 0 for every area on every map and consumers computing "ceiling - bounds height" clamped every flying goal below ground. Movement failures were also silent: with no AAS, MoveToEntity() still returns true and the AI wanders, so the map script waits on scriptedDone() forever. Both failure modes now warn, and ai_scriptedMoveTimeout (30s, 0 to disable) abandons a move that has made no progress so the script can continue. Loading a savegame over a live map left stale state (themuffinator#89). UnloadMap() did not stop the game sound world before destroying the entities that own its emitters, and a freed emitter is only recycled once its channel list empties, so looping sounds from the outgoing session survived. Stop() already did this, which is why quitting to the menu first was clean. Hardened the arm64 paths behind themuffinator#78. The savegame root cause is already fixed; a regression guard now pins it. The binary lexer's stored-size defaults fail closed instead of asserting - release builds compile asserts out, leaving unreadSize at 0 and desynchronizing the rest of the stream - an unexpected BC7 image degrades to uncompressed instead of ending the session mid-load, and -ffp-contract=off keeps ARM64 and x86-64 evaluating the same float math. Requires the companion change in openQ4-game for the AI and AAS reporting, the idlib string-table mirror, and the matching build flag. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

Uh oh!

progression bug #88

Description

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions