[CryoTheRenegade]

Follow-up to #3655, from xezon's "Are there more of these?" on #3580. DazzleRenderObjClass::Init_From_INI casts its const INIClass* to a DazzleINIClass* to call Get_Vector4. The object is a plain INIClass, never a DazzleINIClass, so calling a DazzleINIClass member through that pointer is undefined behavior.

DazzleINIClass only existed to add Get_Vector2, Get_Vector3 and Get_Vector4, and Init_From_INI only calls Get_Vector4. The class is replaced by a file-static Get_INI_Vector4 that takes the INIClass and uses its public Find_Entry, and the unused Get_Vector2 and Get_Vector3 are removed:

-			const DazzleINIClass* dini=reinterpret_cast<const DazzleINIClass*>(ini);
 			tmp_name.Format("%s%d",FLARE_UV_STRING,flare+1);
-			lic.flare_uv[flare]=dini->Get_Vector4(section_name,tmp_name,Vector4(0.0f,0.0f,1.0f,1.0f));
+			lic.flare_uv[flare]=Get_INI_Vector4(*ini,section_name,tmp_name,Vector4(0.0f,0.0f,1.0f,1.0f));

The parsing is unchanged. The same change is made in the Generals and Zero Hour copies of dazzle.cpp.

Evidence

• Disassembly: I compared dazzle.obj for both games against a build of main. The three DazzleINIClass members are gone, and nothing else references them.
  • VS 2026 (ClangCL) Release: Init_From_INI is identical. It already inlined Get_Vector4, and it inlines Get_INI_Vector4 the same way.
  • VC6 Release: Init_From_INI now inlines the helper. The call to Get_Vector4 is replaced by the same Find_Entry and sscanf calls that Get_Vector4 made. The float arithmetic is the same.

AI Disclosure

Claude Opus 5.5 was used to aid in implementation and to compare the disassembly.

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

The change replaces the DazzleINIClass parsing methods with a file-local Get_INI_Vector4 helper. Lensflare UV loading uses the helper directly and retains the (0, 0, 1, 1) fallback in both code trees.

Changes

Lensflare INI parsing

Layer / File(s)	Summary
Vector4 helper and lensflare integration Generals/Code/Libraries/Source/WWVegas/WW3D2/dazzle.cpp, GeneralsMD/Code/Libraries/Source/WWVegas/WW3D2/dazzle.cpp	Both code trees remove the DazzleINIClass parsing methods, add a file-local Get_INI_Vector4 helper, and use it to load lensflare UV values. The helper returns the supplied default when the entry is missing or does not contain four values.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Refactor

Suggested labels: Refactor

Suggested reviewers: bobtista, xezon

Merge Risk: Minimal · up to cd341

No merge-blocking issue is identified in this localized parsing refactor. The reported behavior remains unchanged in both game variants; merge after normal build checks.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title names the main change: it fixes undefined behavior in the dazzle code that reads flare UVs from the INI. The changeset removes the DazzleINIClass cast in Init_From_INI and replaces it with a…
Description check		The description explains the undefined behavior caused by casting a plain INIClass pointer to DazzleINIClass. It describes the replacement with Get_INI_Vector4, the removal of the unused Get_Vector2 a…
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.

Full details: Title check

Explanation

The title names the main change: it fixes undefined behavior in the dazzle code that reads flare UVs from the INI. The changeset removes the DazzleINIClass cast in Init_From_INI and replaces it with a file-static Get_INI_Vector4 helper that takes an INIClass. The title is specific and matches the primary change.

Full details: Description check

Explanation

The description explains the undefined behavior caused by casting a plain INIClass pointer to DazzleINIClass. It describes the replacement with Get_INI_Vector4, the removal of the unused Get_Vector2 and Get_Vector3 methods, the same change in the Generals and Zero Hour copies, and the disassembly comparison. It matches the changeset closely.

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

Recent review info

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 319ac21e-7d67-4d3a-bc1b-50adb58419fd

Commits

Reviewing files that changed from the base of the PR and between 116fbcf and cd3410f.

Files selected for processing: 2

• Generals/Code/Libraries/Source/WWVegas/WW3D2/dazzle.cpp
• GeneralsMD/Code/Libraries/Source/WWVegas/WW3D2/dazzle.cpp

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

[chatgpt-codex-connector]

Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review	Status	Commit	Review trigger
📝 Code Review	✅ Completed 2026-11-09T22:02:45.642382Z	cd3410f	Draft marked ready

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

[greptile-apps]

Confidence Score: 5/5

[Low impact] The PR appears safe to merge.

Summary

Replaces the invalid DazzleINIClass cast with file-static Get_INI_Vector4 in Generals and Zero Hour.

• Both games read flare UVs through the actual INI object.

_{Reviews (1) · Last reviewed commit: "style(ww3d2): Attach pointer and referen..." · Reviewed by Greptile}