search

FWGS / xash3d-fwgs

Xash3D FWGS engine
1.55k stars 234 forks source link

Code style and clang-format #844

Open vklachkov opened 2 years ago

vklachkov commented 2 years ago

Немного ковыряясь в xash3d-fwgs, обратил внимание на отсутствие единого стиля кода и декларативного файла clang-format. Я сделал черновик clang-format.txt, который выдаёт после форматирования стилистически-схожий код. Для удобства пользовался конфигуратором, можете загрузить в него конфиг и покрутить крутилочки.

К обсуждению призываю @a1batross, @mittorn, @SNMetamorph

a1batross commented 2 years ago

У нас был в старом движке, но там под clang-format пятилетней давности. Скрасноглазил напрямую с документации.

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

vklachkov commented 2 years ago

У нас был в старом движке

Что такое старый движок? Не первый раз вижу упоминание и не понимаю, про что речь

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

На весь код может и не следует, но на новые PR -- нужен, чтобы не вносить анархию в движок

a1batross commented 2 years ago

FWGS/xash3d

vklachkov commented 2 years ago

А, етот, понял. С тех пор прошло несколько лет, синхронизации с другой версией уже не происходит, препятствия более нет

a1batross commented 2 years ago

Короче, я посмотрел снова, оно всегда умело вставлять пробелы между скобками, но не умеет игнорировать пробел когда скобки идет друг за дружкой: Q_memcpy( dest, src, sizeof( *dest )); оно справедливо отформатирует в Q_memcpy( dest, src, sizeof( *dest ) );

В остальном же привести к наименьшему делителю среди всего зоопарка сишного кода его вполне возможно. Фичу для скобок хоть сам контрибьють, что кстати вполне неплохая идея. :)

vklachkov commented 2 years ago

В остальном же привести к наименьшему делителю среди всего зоопарка сишного кода его вполне возможно

Что нужно добавить, убрать, исправить в вышеприкреплённых правилах?

Фичу для скобок хоть сам контрибьють

Ещё погружаться в шланг-формат, чтобы у скобочки не было лишнего пробела... Расточительно. Нужно больше сайд проектов :)