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

FT_MOTION : Fix compile error #27367

Merged
merged 2 commits into from
Aug 26, 2024

Conversation

narno2202
Copy link
Contributor

Fix compile error

Description

As HAS_X_AXIS is true if FTM_SHAPING is defined, there is a missing comma in

#if HAS_FTM_SHAPING
  FTMotion::shaping_t FTMotion::shaping = {
    0,
    #if HAS_X_AXIS
      x:{ false, { 0.0f }, { 0.0f }, { 0 }, 0 } // ena, d_zi[], Ai[], Ni[], max_i
    #endif
    #if HAS_Y_AXIS
      y:{ false, { 0.0f }, { 0.0f }, { 0 }, 0 } // ena, d_zi[], Ai[], Ni[], max_i
    #endif
  };

Removing HAS_X_AXIS conditional and adding a comma after HAS_Y_AXIS solves the error

Requirements

Benefits

Configurations

Related Issues

Compilation fix
@narno2202 narno2202 changed the title FT_MOTION : Fix compile eeror FT_MOTION : Fix compile error Aug 24, 2024
@erril007
Copy link

erril007 commented Aug 24, 2024

For me it was enough to add the missing comma at the end of line 97 in ft_motion.cpp:

x:{ false, { 0.0f }, { 0.0f }, { 0 }, 0 }, // ena, d_zi[], Ai[], Ni[], max_i

i think it went missing in #27359

@thinkyhead thinkyhead merged commit 2cb252d into MarlinFirmware:bugfix-2.1.x Aug 26, 2024
63 checks passed
@narno2202 narno2202 deleted the FT_MOTION_fix branch August 26, 2024 20:25
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.

4 participants