Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow resetting timer more than once without starting it #2249

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JustScott
Copy link
Contributor

Previously the timer only allows resetting the hour and minute values right when the app opens, and after the timer has ran once.

broken_reset_button

My fix just ignores whatever DisplayTime() would return, setting the minute and second value to 0 instead. There may be a better approach that someone could suggest below, otherwise this one works fine.

reset_fixed

Copy link

Build size and comparison to main:

Section Size Difference
text 373040B 16B
data 948B 0B
bss 22536B 0B

Run in InfiniEmu

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.

1 participant