classicrocker883 / MRiscoCProUI

This is optimized firmware for Voxelab Aquila & Ender3 V2/S1 3D printers.
https://classicrocker883.github.io/
Other
74 stars 17 forks source link

[BUG] Warning with NOZZLE_PARK_FEATURE disabled #130

Closed wlkmanist closed 2 weeks ago

wlkmanist commented 3 months ago

Did you test the latest release build?

No, but I will try it!

Bug Description

May be it is precompiled binary issue. I don't know.

How to reproduce: Disable in config: NOZZLE_PARK_FEATURE FILAMENT_RUNOUT_SENSOR ADVANCED_PAUSE_FEATURE

Then I have this warning: .pio/build/GD32F103RC_voxelab_maple/src/src/lcd/e3v2/proui/toolbar.cpp.o:(.rodata.TBItemA+0xa4): warning: undefined reference to 'ParkHead()'

I am not actually flashed that build (with warning) into my board. Would be cool if there is any solution to fix that because disabling that features in config can save a lot of memory.

Printer Model

Voxelab Aquila

Model Type

No response

Your Mainboard

Aquila GD32

Other Mainboard Type

No response

Add-ons that could be involved

No response

ProUI?

ProUI

Bed Leveling

MM - Manual Mesh Bed Leveling

Did you include your own configuration files?

Additional information & file uploads

No response

classicrocker883 commented 2 months ago

in Marlin\src\lcd\e3v2\proui\toolbar_def.h

replace the line with ParkHead with this, and remove the , after WriteEeprom):

TB_ITEM(ICON_WriteEEPROM, MSG_STORE_EEPROM, WriteEeprom)
OPTARG(NOZZLE_PARK_FEATURE, TB_ITEM(ICON_Park, MSG_FILAMENT_PARK_ENABLED, ParkHead))
github-actions[bot] commented 2 weeks ago

This issue has had no activity in the last 60 days. Please add a reply if you want to keep this issue active, otherwise it will be automatically closed within 10 days.

classicrocker883 commented 2 weeks ago

fixed in a commit a while back