Define codestyle

[aol-nnov]

@fralx я попытался подобрать наиболее похожий на то, что было в репозитории стиль написания кода. Мне показалось, что на WebKit очень похоже и я к нему дописал некоторые визуально приятныее моему глазу добавки, плюс сортировка и группировка хедеров.

Отформатировал всё, кроме 3rdparty. Форматировал первым попавшимся под руку clang-format, сегодня у меня это версия 11. Есть подозрение, что другие версии будут давать немного иной результат. На работе недавно мигрировали с 7 на 14 и уже столкнулись с разницей в поведении..

Предлагаю на рассмотрение результат работы линтера.

Традиционно, работа основана на другом PR (#471) и если мёржить по порядку, ребейз не потребуется.

[aol-nnov]

посмотрел повнимательнее, запятые в начале строки - это какая-то фигня, на мой взгляд (в вебкитовском стиле так задано)

    : QWidget(parent)
    , m_designerWidget(NULL)
    , m_mainWindow(NULL)
    , m_changingItemSelection(false)
    , m_movingItem(false)

открутил.. но тогда список инициализации в одну строку получается - тоже не айс :/

ObjectBrowser::ObjectBrowser(QWidget* parent):
    QWidget(parent), m_designerWidget(NULL), m_mainWindow(NULL), m_changingItemSelection(false),
    m_movingItem(false)

О, накрутил, сейчас залью.

ObjectBrowser::ObjectBrowser(QWidget* parent):
    QWidget(parent),
    m_designerWidget(NULL),
    m_mainWindow(NULL),
    m_changingItemSelection(false),
    m_movingItem(false)
{

Вроде, прилично

[aol-nnov]

Остановился на заданном ранее стиле и clang-format версии 14, так как он по умолчанию в стабильном дебиане (bookworm)

[xakod]

Наверное надо будет подождать #470 и #471 потому что из-за 471 билд падает

[aol-nnov]

появилась мысль дописать в пайплайн проверку стиля, так что, это пока не готово, выходит.

[aol-nnov]

Ну, всё, ¡No pasarán! ))

Так будут выглядеть джобы в случае нарушения форматирования кода. В логах перечислены файлы, требующие внимания.

Как изучите ситуацию - дайте знать, я уберу демонстрационный коммит и можно будет мёржить, наверное. На appveyor ещё глянуть бы - он отстаёт.. )

[aol-nnov]

Ндэ.. В appveyor вообще cmd в качестве шелла, выходит? То есть, юниксовых команд нет или надо какие-то дополнительныне заклинания?

Идея была в том, чтобы реализовать раннее прекращение пайплайна в случае выявления нарушений стиля... Для гитхабовского сиая получилось, а как быть со сторонней интеграцией - пока не понятно..

@fralx @Jihadist что скажете?

[xakod]

Просто не добавляй в Appveyor и всё) сделай это отдельным шагом в github actions, который не включается в другие.

[aol-nnov]

Просто не добавляй в Appveyor и всё

но тогда он будет долго шуршать зазря... Изначальная идея была вообще не выполнять сборку, если стиль нарушен.

[fralx]

Да и пускай шуршит он железный :)

[aol-nnov]

Да и пускай шуршит

сокращение времени обратной связи, то-сё.... ))

Окей, выкину из аппвеера тогда. И, выходит, готово.