fix: R_FindPlane aborts on visplane pool exhaustion

[VedantMadane]

Summary

R_FindPlane aborts on visplane pool exhaustion

Changes

• R_FindPlane calls i_error! on visplane pool exhaustion instead of returning null

Fixes #117

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)

• room/src/doom/r_plane.rs

Previous Review Summaries (2 snapshots, latest commit 6ea8c18)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 6ea8c18)

Status: No Issues Found | Recommendation: Merge

Previous WARNING on room/src/doom/r_plane.rs line 550 (stale doc comment) has been resolved.

Files Reviewed (1 file)

• room/src/doom/r_plane.rs

Previous review (commit 15c8a44)

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity	Count
CRITICAL	0
WARNING	1
SUGGESTION	0

Issue Details (click to expand)

WARNING

File	Line	Issue
room/src/doom/r_plane.rs	550	Stale doc comment — function no longer returns null on exhaustion

Files Reviewed (1 file)

• room/src/doom/r_plane.rs — 1 issue

Fix these issues in Kilo Cloud

---

_{Reviewed by kimi-k2.6 · Input: 64.6K · Output: 9.9K · Cached: 499.7K}

[VedantMadane]

Doc comment already updated in 6ea8c18 to state abort via i_error! on visplane pool exhaustion (no null return). Leaving the thread for maintainers; no further code change needed.

[VedantMadane]

Resolved 1 review thread that is addressed on current head:

• R_FindPlane doc comment now states it aborts via i_error! when the visplane pool is exhausted (no longer claims null return)

[VedantMadane]

Soft bump — no further code change.

Doc on R_FindPlane already states it aborts via crate::i_error! when the visplane pool (128 entries) is exhausted (fixed in 6ea8c18). Kilo’s later review also marked the stale-null doc as resolved. Ready for merge.