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 building error 'did_pause_print' not declared in the scope #20869

Merged
merged 2 commits into from
Jan 25, 2021
Merged

Fix building error 'did_pause_print' not declared in the scope #20869

merged 2 commits into from
Jan 25, 2021

Conversation

bg-master
Copy link
Contributor

Description

The build failed due to 'did_pause_print' not declared in the scope in runout.h

In file included from Marlin\src\feature\runout.cpp:31:0:
Marlin\src\feature\runout.h: In static member function 'static void TFilamentMonitor<RESPONSE_T, SENSOR_T>::run()':
Marlin\src\feature\runout.h:121:66: error: 'did_pause_print' was not declared in this scope
       if (enabled && !filament_ran_out && (printingIsActive() || did_pause_print)) {
                                                                  ^~~~~~~~~~~~~~~
In file included from Marlin\src\feature\runout.cpp:31:0:
Marlin\src\feature\runout.h: In static member function 'static void RunoutResponseDelayed::block_completed(const block_t*)':
Marlin\src\feature\runout.h:344:55: error: 'did_pause_print' was not declared in this scope
         if (b->steps.x || b->steps.y || b->steps.z || did_pause_print) { // Allow pause purge move to re-trigger runout state
                                                       ^~~~~~~~~~~~~~~
*** [.pio\build\mks_robin_nano35\src\src\feature\runout.cpp.o] Error 1

@kad
Copy link
Contributor

kad commented Jan 24, 2021

Verified, works in my configs combination that I mentioned in #20832 (comment)

@thinkyhead thinkyhead merged commit 85d61de into MarlinFirmware:bugfix-2.0.x Jan 25, 2021
TyMi pushed a commit to TyMi/Marlin that referenced this pull request Jan 25, 2021
…_bugfix

* commit '876c2586b9146dd123af4c7b21138b8239ef5d39': (116 commits)
  Clean up MMU2 code (MarlinFirmware#20794)
  Init KILL, SUICIDE, PSU earlier (MarlinFirmware#20810)
  "Move … code" followup (MarlinFirmware#20869)
  Apply SEC_TO_MS and other fixes
  Reformat abortSDPrinting
  Fix sign warning (MarlinFirmware#20872)
  Cosmetic changes (2) (MarlinFirmware#20876)
  "Move … code" followup (MarlinFirmware#20868)
  "Move … code" followup (MarlinFirmware#20874)
  Graphical TFT fixes, cleanup (MarlinFirmware#20861)
  🧻 Cosmetic changes (MarlinFirmware#20859)
  🛠Fix deps script version regex
  Fix Ender 3 V2 DWIN manual move (MarlinFirmware#20837)
  [cron] Bump distribution date (2021-01-25)
  Fix LiquidCrystal CI failures (MarlinFirmware#20873)
  Revert "Adding custom move feedrate for G26 (MarlinFirmware#20729)" (MarlinFirmware#20870)
  MeatPack serial encoding (MarlinFirmware#20802)
  [cron] Bump distribution date (2021-01-24)
  lcd_put_wchar_max for COLOR_UI (MarlinFirmware#20838)
  Adding custom move feedrate for G26 (MarlinFirmware#20729)
  ...

# Conflicts:
#	Marlin/Configuration.h
#	Marlin/Configuration_adv.h
Jyers pushed a commit to Jyers/Marlin that referenced this pull request Feb 3, 2021
kpishere pushed a commit to kpishere/Marlin that referenced this pull request Feb 19, 2021
zillarob pushed a commit to zillarob/Marlin that referenced this pull request Feb 25, 2021
W4tel-BiDi pushed a commit to W4tel-BiDi/Marlin that referenced this pull request Apr 5, 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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants