Optimized parsing in interfaceCmd.c and parseACK.c + configurable M109/M190

[digant73]

IMPROVEMENTS:

• Optimized parsing code in interfaceCmd: Reduced the use of array indexing and used length of the key for a fast check on the buffer and matching/not matching conditions

• Optimized parsing code in parseAck: Reduced conditions in the parsing functions to avoid as much as possible continue checks on the validity of the array elements (e.g. != 0). Used length L2 cache and key for a fast check on matching/not matching conditions. Major benefits are present when searching a key in the middle of L2 cache

• Code reduction: Due to optimizations above

• Configurable feature "emulated M109/M190": New param "emulated_m109_m190" added in config.ini to enable/disable M109/M190 emulated with M104/M140

• Added rotary encoder support on List menu (list/icon menu): Rotary encoder can be used to navigate (page up/down) on any List menu without the need to use the page up/down buttons.

  NOTE: As for all the other menus supporting rotary encoder, the button on rotary encoder MUST/CAN NOT be used to apply an action on a list item (e.g. select the file to print). That's due to the HW shared with mainboard (Rotary encoder is physically connected to the mainboard and any press on rotary button will have effects also on Marlin menu).
  To apply an action on the item, you MUST continue to apply a touch on the screen

• Minor cleanup in config.ini: Forgot to apply the following param rename in my prevoius PR:
  rotate_ui -> rotated_ui
  block_touch_on_idle -> lcd_lock_on_idle

• Minor cleanup

BUGFIXES / FR:
fixes #2153
resolves #1974
resolves #2151

[bttguy]

Wow @digant73 you already implemented the encoder scroll?

[digant73]

rotary encoder is now supported in both List/Icon modes

[bttguy]

rotary encoder is now supported in both List/Icon modes

Awesome! Huge thanks to you!!

[Tannoo]

In TFT/src/User/API/config.c LN#557:

    case C_INDEX_EMULATED_M600:
    case C_INDEX_EMULATED_M109_M190:
      SET_BIT_VALUE(infoSettings.general_settings, (index - C_INDEX_EMULATED_M600), getOnOff());
      break;

Is this intentional or is something missing?

I am trying to implement this now and this doesn't make sense.

[digant73]

yes, it is correct