- `Conditionals.h` is included twice, setting LCD values the first time

Latest upstream commits

Fix for menus failing to scroll

by updating 'encoderPosition' in END_MENU() line 1.

Fix count_direction initializer

- Addressing #1625

Get upstream commits

Fix hiliting of edit items in LCD menus

Latest upstream changes

PID debug

Beautivy statusscreen

codepos not used in gcode_M28()

- Not sure why these are showing as all new

- Add `Conditionals.h` with calculated configuration values
- Add `SanityCheck.h` with checks for configuration errors
- Remove equivalent code from all configurations
- Move error checks from some sources to `SanityCheck.h` also
- Fix initialization of count_direction in stepper.cpp

Conflicts:
	Marlin/temperature.cpp

Cleaned merge conflict.

Codeposition had changed.
Variable pid_input had vanished.

Signed-off-by: AnHardt <github@kitelab.de>

Conflicts:
	Marlin/dogm_lcd_implementation.h

solved merge conflict.

Reduce size of menu code

by shifting things some pixels around

In the new method we pass the character that should be used for
selected state, not the character to print always.

- Pass selected state directly to lcd_implementation_mark_as_selected
- Rename sel function parameter
- Include a minor fix for SdBaseFile.h

Get upstream changes

When one hit "Stop Print" option in LCD menu, the command buffer was not
cleared. The printer keep moving until the buffer has been emptied.
Actually I could not clear the command buffer as well.. I don't know
why, it doesnt work as expected.
I need to implement a routine inside Stepper ISR to handle such
situation.

Saves one addition.
Needs one pointer on the stack.

Don't know if this is a go trade.

Remove 6x9 font

Decrease KILL_DELAY

Negative times are unlikly

optional parameters must be named when ignoring order

lcd_control_retract_menu() is conditional.

lcd_autostart_sd() has two conditions.

delta[3] not used when DELTA not defined.

G4 could wait for an undefined time

Shoot in the dark.

Got:
Marlin_main.cpp:2567: warning: unused variable 'codepos'

Needs revision.
Don't now how this should look like.

Got:
Marlin_main.cpp:1689: warning: 'codenum' may be used uninitialized in this function

And indeed codenum is undefined when no parameter is given.

got:
Marlin_main.cpp:387: warning: 'delta' defined but not used

Compiles cleaner when definition is removed.

got:
ultralcd.cpp:408: warning: 'void lcd_autostart_sd()' defined but not used

Got:
ultralcd.cpp:64: warning: 'void lcd_control_retract_menu()' declared 'static' but never defined

While bunting an other bug I stumbled across:
ultralcd.cpp:1250: warning: comparison between signed and unsigned
integer expressions
Changed to the type of lcd_next_update_millis.

Users hat to press kill-button for unexpexted lon times.
See #1593

fix macro