Skip to content

Cleanup various things noticed while fixing rando generation / debug warp#6152

Merged
serprex merged 5 commits intoHarbourMasters:developfrom
serprex:cleanup
Jan 17, 2026
Merged

Cleanup various things noticed while fixing rando generation / debug warp#6152
serprex merged 5 commits intoHarbourMasters:developfrom
serprex:cleanup

Conversation

@serprex
Copy link
Contributor

@serprex serprex commented Jan 17, 2026

@mergify
Copy link
Contributor

mergify bot commented Jan 17, 2026

⚠️ The sha of the head commit of this PR conflicts with #6150. Mergify cannot evaluate rules on this PR. ⚠️

@Malkierian
Copy link
Contributor

I'm not comfortable with the loss of specificity in the source files where RAND_GET_OPTION was redefined. In my opinion, if anything, the places that use the .Get version should be converted to the more granular method, and the define in the header should lose the .Get. But that's a not insignificant effort.

@serprex serprex requested a review from Malkierian January 17, 2026 18:07
@serprex
Copy link
Contributor Author

serprex commented Jan 17, 2026

working through revising, but Is/IsNot don't offer more type safety since they're taking arg as int32_t

@Malkierian
Copy link
Contributor

I honestly just like the user-friendly reading of Is and IsNot, but maybe we need to revisit the whole Is and IsNot thing if that's the case.

@serprex
Copy link
Contributor Author

serprex commented Jan 17, 2026

@Malkierian updated to Is/IsNot

Recent change in init ordering had param hashmap built before static locations setup by hooks
OnGameLoad hook was not being triggered, which time savers rely on
@serprex serprex merged commit f474a86 into HarbourMasters:develop Jan 17, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants