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

Quick fixes for warnings #27507

Merged

Conversation

classicrocker883
Copy link
Contributor

Description

These are fixes for the following warnings

powerloss.cpp

'void* memset(void*, int, size_t)' clearing an object of non-trivial type 'struct job_recovery_info_t'; use assignment or value-initialization instead [-Wclass-memaccess]
void PrintJobRecovery::init() 
-{ memset(&info, 0, sizeof(info)); }

+{ info = {}; }

// Possible 2nd option for fix //
+{ info = job_recovery_info_t(); }

limits.h

narrowing conversion from 'int' to 'short unsigned int' [-Wnarrowing]
// Homing Feedrate limits
EDITABLE_HOMING_FEEDRATE
-xyz_uint_t
+xyz_ulong_t

Finally, there is a typo in proui/dwin.cpp, and also removed static from editable homing feedrate menu item

Requirements

Benefits

Configurations

Related Issues

@thinkyhead thinkyhead merged commit 4b6109f into MarlinFirmware:bugfix-2.1.x Nov 12, 2024
64 checks passed
@classicrocker883 classicrocker883 deleted the bugfix-2.1.x-October8 branch November 13, 2024 09:35
bsdnomad pushed a commit to bsdnomad/Marlin that referenced this pull request Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants