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

Fix compiler warning in Calculator app #1952

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

NotherNgineer
Copy link
Contributor

Fixed compiler warning on this line. The memmove() function supports overlapping copy operations, whereas the memcpy() function might or might not.

I can't figure out how to use the Calculator app either, but I didn't like seeing the compiler warning. :-)

@zxkmm zxkmm requested a review from bernd-herzog March 7, 2024 16:00
Copy link
Member

@gullradriel gullradriel left a comment

Choose a reason for hiding this comment

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

Good to go !
Thanks, I also do not like warnings

@gullradriel gullradriel merged commit 908c0e2 into portapack-mayhem:next Mar 7, 2024
3 checks passed
@NotherNgineer NotherNgineer deleted the calculator branch March 13, 2024 17:45
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