clang_format

[yanovich]

Есть предложение добавить файл настроек для clang-format. Это позволит установить стандарт для форматирования. Сейчас единого подхода в коде нет, поэтому приходится код руками форматировать, от чего я отвык уже )

Я пробовал разные варианты, в итоге остановился на требованиях OpenSSL для форматирования. У них тоже нет своих настроек для clang-format, но есть требования.

Минусом вносимого предложения является большой первоначальный whitespace патч, но плюсом -- сокращение времени на "причёсывание" будущих патчей.

Если смысл есть, сделаю PR.

clang-format.txt

[beldmit]

Да, всячески одобряю. Если ещё и тест удастся сделать, будет вообще отлично

[vt-alt]

Есть же .editorconfig.

[yanovich]

Первоначальный вариант в #427

[yanovich]

Да, всячески одобряю. Если ещё и тест удастся сделать, будет вообще отлично

Точно можно сделать для clang-* вариантов тестов.

[beldmit]

Лучше бы отдельно

[phprus]

clang-format разных версий немного по разному воспринимает правила форматирования, в связи с этим автоматическая проверка в CI может приводить к сложностям, когда clang-format у автора отформатировал код одним способом, а clang-format в CI другим.

[vt-alt]

Да, всячески одобряю. Если ещё и тест удастся сделать, будет вообще отлично

К сожалению в результате этих изменений теряется git blame, например:

View file + Blame на один из файлов из #427: https://github.com/gost-engine/engine/blame/cf83bf3cb9108370bef19284278152e14acb115a/gost_ameth.c
у большинства строк с кодом: consistent formatting

ps. Это супер: https://github.com/gost-engine/engine/blame/cf83bf3cb9108370bef19284278152e14acb115a/test_ciphers.c#L213

pps. Может достаточно переформатировать только отступы в начале строк, но не менять внутри строк?

[phprus]

Для git blame есть штатное решение: https://michaelheap.com/git-ignore-rev/

Гитхаб его поддерживает: https://docs.github.com/en/repositories/working-with-files/using-files/viewing-a-file#ignore-commits-in-the-blame-view

[yanovich]

pps. Может достаточно переформатировать только отступы в начале строк, но не менять внутри строк?

Таких параметров я не нашёл, к сожалению.

Главный эффект в том, что если такое сделать, то дальнейшее форматирование будет одной кнопкой прямо из редактора.

Для git blame есть штатное решение: https://michaelheap.com/git-ignore-rev/

Гитхаб его поддерживает: https://docs.github.com/en/repositories/working-with-files/using-files/viewing-a-file#ignore-commits-in-the-blame-view

Бомба! Можно считать, что недостатка уже нет!

[yanovich]

Лучше бы отдельно

Сделал в #427, сработка в #428