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 compilation errors for UI on Native build (simulator) #20092

Merged

Conversation

rhapsodyv
Copy link
Member

@rhapsodyv rhapsodyv commented Nov 10, 2020

Description

TFT UI (color, classic and lvgl) fails to build on simulator because of the issues fixed here. Same are code that aren't used (but compiled), other a null pointer where it can't be.
ARM gcc just drops the unused references. But native don't. I prefer to fix it here, that just hiding the problem making native gcc drop it too.

@rhapsodyv rhapsodyv changed the title Fix compilation error for Color UI on Native build (simulator) Fix compilation errors for UI on Native build (simulator) Nov 10, 2020
@thinkyhead thinkyhead merged commit f8ee6f8 into MarlinFirmware:bugfix-2.0.x Nov 11, 2020
@rhapsodyv rhapsodyv deleted the color-ui-compilation-fix branch November 11, 2020 01:43
ustsv pushed a commit to ustsv/Marlin that referenced this pull request Nov 12, 2020
FhlostonParadise pushed a commit to FhlostonParadise/Marlin that referenced this pull request Nov 21, 2020
Kannix2005 pushed a commit to Kannix2005/Marlin-1 that referenced this pull request Dec 7, 2020
vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request Dec 9, 2020
tharts pushed a commit to tharts/Marlin that referenced this pull request Jan 6, 2021
dpreed pushed a commit to dpreed/Marlin_2.0.x that referenced this pull request Feb 5, 2021
kpishere pushed a commit to kpishere/Marlin that referenced this pull request Feb 19, 2021
W4tel-BiDi pushed a commit to W4tel-BiDi/Marlin that referenced this pull request Apr 5, 2021
chrisjenda pushed a commit to chrisjenda/Marlin that referenced this pull request Apr 5, 2021
chrisjenda pushed a commit to chrisjenda/Marlin that referenced this pull request Apr 8, 2021
chrisjenda pushed a commit to chrisjenda/Marlin that referenced this pull request Apr 11, 2021
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Apr 28, 2021
thinkyhead pushed a commit to thinkyhead/Marlin that referenced this pull request Apr 29, 2021
thinkyhead pushed a commit that referenced this pull request Apr 30, 2021
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