bugfix(w3d): Fix flat terrain tiles leaking on device reset

[tintinhamans]

FlatHeightMapRenderObjClass::ReleaseResources never frees m_tiles. ReAcquireResources allocates a new array over it, so every device reset leaks the old tiles with their vertex and index buffers, textures and map reference.

ReleaseResources now calls releaseTiles first, same as initHeightData does when the tile count changes.

Only reachable with TerrainLOD = AUTOMATIC, retail data uses DISABLE. Relevant for #3237.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 35a1b8fe-2292-41df-b5b7-60f7d9a6a595

---

📥 Commits

Reviewing files that changed from the base of the PR and between fa486c0 and 6539b7a.

---

📒 Files selected for processing (1)

• Core/GameEngineDevice/Source/W3DDevice/GameClient/FlatHeightMap.cpp

---

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 2 remain after this review.

---

---

Walkthrough

FlatHeightMapRenderObjClass::ReleaseResources now releases the terrain tile array before calling the base resource-release method.

Changes

Terrain resource release

Layer / File(s)	Summary
Release terrain tiles Core/GameEngineDevice/Source/W3DDevice/GameClient/FlatHeightMap.cpp	ReleaseResources calls releaseTiles() before the base resource-release method.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Suggested reviewers: cryotherenegade

Merge Risk: ⚪ Minimal · up to 6539b

Resource release now frees terrain tiles before cleanup, and reacquisition can rebuild them. No actionable merge-blocking risk remains.

Pre-merge checks · 4

Passed checks (4 passed)

Check name	Status	Explanation
Title check		The title clearly identifies a bug fix for leaking flat terrain tiles during device reset. It matches the main change.
Description check		The description explains the resource leak, the device-reset scenario, the fix, and its applicability. It is directly related to the changeset.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

---

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

---

_{Comment @coderabbitai help to get the list of available commands.}

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge; the added cleanup fits the existing terrain rebuild path.

Summary

Adds releaseTiles() before base cleanup in FlatHeightMapRenderObjClass::ReleaseResources().

• Flat terrain tiles are cleared before reset rebuilds them.

_{Reviews (2) · Last reviewed commit: "bugfix(w3d): Fix flat terrain tiles leak..." · Reviewed by Greptile}

[Mauller]

Looks clear