Skip to content

Add EmptyReload InfoType for digital display - #2347

Closed
NukeAtty wants to merge 3 commits into
Phobos-developers:developfrom
NukeAtty:feature/empty-reload-digital-display
Closed

Add EmptyReload InfoType for digital display#2347
NukeAtty wants to merge 3 commits into
Phobos-developers:developfrom
NukeAtty:feature/empty-reload-digital-display

Conversation

@NukeAtty

Copy link
Copy Markdown
Contributor

Adds a new InfoType=EmptyReload for digital display.

  • New DisplayInfoType::EmptyReload enumeration value (appended as 21 to preserve save-game compatibility).
  • New INI tag parsing for InfoType=EmptyReload.
  • Displays the empty reload countdown while the Techno has no ammo left, and 0 otherwise.
  • Falls back to Reload duration when EmptyReload is not positive.
  • Existing InfoType=Reload behavior is unchanged.

@github-actions

github-actions Bot commented Aug 16, 2026

Copy link
Copy Markdown

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

Comment thread docs/Whats-New.md Outdated
:open:

#### New:
- [New `InfoType=EmptyReload` for digital display](User-Interface.md#digital-display) (by NukeAtty)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest that after LLM helps complete the PR, human developers should review it independently first before opening the pull request.

@Coronia

Coronia commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

would it be better if this just use Reload infotype and handle EmptyReload specifically? Not sure if there's any case that only need to show EmptyReload seperatedly

@NukeAtty

Copy link
Copy Markdown
Contributor Author

would it be better if this just use Reload infotype and handle EmptyReload specifically? Not sure if there's any case that only need to show EmptyReload seperatedly

Oh I see, silly of me, now this PR is kinda meaningless. I'd close it now :)

@NukeAtty

Copy link
Copy Markdown
Contributor Author

Closing: EmptyReload behavior is already covered by the existing Reload InfoType.

@NukeAtty NukeAtty closed this Aug 17, 2026
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.

3 participants