engine
engine copied to clipboard
gost-engine
clang_format
Есть предложение добавить файл настроек для clang-format. Это позволит установить стандарт для форматирования. Сейчас единого подхода в коде нет, поэтому приходится код руками форматировать, от чего я отвык уже )
Я пробовал разные варианты, в итоге остановился на требованиях OpenSSL для форматирования. У них тоже нет своих настроек для clang-format, но есть требования.
Минусом вносимого предложения является большой первоначальный whitespace патч, но плюсом -- сокращение времени на "причёсывание" будущих патчей.
Если смысл есть, сделаю PR.
Да, всячески одобряю. Если ещё и тест удастся сделать, будет вообще отлично
Да, всячески одобряю. Если ещё и тест удастся сделать, будет вообще отлично
Точно можно сделать для clang-* вариантов тестов.
clang-format разных версий немного по разному воспринимает правила форматирования, в связи с этим автоматическая проверка в CI может приводить к сложностям, когда clang-format у автора отформатировал код одним способом, а clang-format в CI другим.
Да, всячески одобряю. Если ещё и тест удастся сделать, будет вообще отлично
К сожалению в результате этих изменений теряется 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. Может достаточно переформатировать только отступы в начале строк, но не менять внутри строк?
Для 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
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
Бомба! Можно считать, что недостатка уже нет!