AllowAllConstructorInitializersOnNextLine and
ConstructorInitializerAllOnOneLineOrOnePerLine are both deprecated.
PackConstructorInitializers=Never is the most similar setting to
most code and the example in the policy document. E.g. this does not
get reformatted:
STRINGFORMATTER( int aReserve = 300 ) :
buffer( aReserve, '\0' )
{
}
Currently, the format enforces single lines when possible, but does
not enforce readable column-based alignment (and, moreover, *removes*
such manually added alignment:
switch( m_orientation )
{
case PIN_RIGHT: m_orientation = PIN_UP; break;
case PIN_UP: m_orientation = PIN_LEFT; break;
}
Change this to multi-line by default:
switch( m_orientation )
{
case PIN_RIGHT:
m_orientation = PIN_UP;
break;
case PIN_UP:
m_orientation = PIN_LEFT;
break;
}
If the developer wishes for column-aligned single-line cases, this
is permitted, but much be done manually:
switch( m_orientation )
{
case PIN_RIGHT: m_orientation = PIN_DOWN; break;
case PIN_UP: m_orientation = PIN_RIGHT; break;
}
CHANGE: the _clang-format file to reflect this, and add note about
manual override in the dev docs.
Enforce the following format:
template<>
void function<TYPE>()
Rather than:
template<> void function<TYPE>()
This is the more common format in KiCad (about 8:1) and agrees
with the uncrustify.cfg rules.
John Beard
Wayne Stambaugh
Marek Roszko
Jeff Young
Ian McInerney
Chris Pavlina