phaneron/Kelsidavis-WoWee
1
0
Fork 0
mirror of https://github.com/Kelsidavis/WoWee.git synced 2026-05-08 18:13:52 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

fix(zone): scrub NaN/inf manifest floats at save time

Browse source 
Mirrors the load-side sanitize. nlohmann throws on NaN/inf, so
a corrupted in-memory baseHeight or volume would abort the manifest
save and lose all zone-level settings (flags, audio config, etc.).
This commit is contained in:
Kelsi 2026-05-06 07:05:13 -07:00
parent 49feb8b9f6
commit 1135e499d7
1 changed files with 7 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

10
tools/editor/zone_manifest.cpp
View file

@ -20,7 +20,9 @@ bool ZoneManifest::save(const std::string& path) const {
j["displayName"] = displayName;
j["mapId"] = mapId;
j["biome"] = biome;
j["baseHeight"] = baseHeight;
// Scrub before serialization — nlohmann throws on NaN/inf, which would
// wipe the entire manifest save and lose all zone settings.
j["baseHeight"] = std::isfinite(baseHeight) ? baseHeight : 100.0f;
j["hasCreatures"] = hasCreatures;
j["description"] = description;
j["editorVersion"] = "1.0.0";
@ -60,8 +62,10 @@ bool ZoneManifest::save(const std::string& path) const {
if (!musicTrack.empty()) audio["music"] = musicTrack;
if (!ambienceDay.empty()) audio["ambienceDay"] = ambienceDay;
if (!ambienceNight.empty()) audio["ambienceNight"] = ambienceNight;
audio["musicVolume"] = musicVolume;
audio["ambienceVolume"] = ambienceVolume;
audio["musicVolume"] = std::isfinite(musicVolume)
? std::clamp(musicVolume, 0.0f, 1.0f) : 0.7f;
audio["ambienceVolume"] = std::isfinite(ambienceVolume)
? std::clamp(ambienceVolume, 0.0f, 1.0f) : 0.5f;
j["audio"] = audio;
}