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

Issue232 #237

Closed
wants to merge 3 commits into from
Closed

Issue232 #237

wants to merge 3 commits into from

Conversation

curtbinder
Copy link
Member

Just tested out my controller and fixed the 12 hour display bug that I reported (#232). Both midnight and lunch/noon display the 12 properly along with their respective AM/PM.

This branch was also created based off my fixing of the compiler warnings in the other pull request ( #236)

Also, my editor fixed a lot of other areas of the files by removing the additional whitespace from the lines.

Changed prog_uchar to unsigned char
changed prog_uint16_t to uint16_t
The case checking for displaying the 12 hour format was incorrect. It would display 12:00 PM (aka, noon) as 00:00 PM.

close reefangel#232
The previous attempt only fixed displaying 12:00 pm properly and did not handle midnight. This fix handles displaying midnight properly as well.

** The other changes in the file are my editor removing the trailing whitespace and blanks **
@curtbinder curtbinder closed this Aug 16, 2017
@curtbinder curtbinder deleted the issue232 branch August 16, 2017 17:31
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