[Grbl_Esp32 PR#590] Suppress one-line case statements and fix atari custom

未分类 bolang 4个月前 (10-14) 35次浏览

Issue #590 | 状态: 已关闭 | 作者: odaki | 创建时间: 2020-09-10


AllowShortCaseLabelsOnASingleLine: 'true'
It works for simplifying break-only cases, but to me it feels like over-connecting lines.
This configuration runs the following format.
From:
C++
default:
grblmsgsendf(CLIENT_SERIAL, MsgLevel::Info, "Unknown Switch %d", index);
break;
`
To:
`C++
default: grblmsgsendf(CLIENT_SERIAL, MsgLevel::Info, "Unknown Switch %d", index); break;

There seems to be a little bit of a change forgotten in the changes you put in to accommodate the enum class State.
We can keep as much of the original code as possible by adding this fix and then adding a small modification to Atari custom.

原始Issue: https://github.com/bdring/Grbl_Esp32/pull/590

喜欢 (0)