[11442] Cleanup codestyle in WorldSocketMgr.cpp & WorldSocketMgr.h - Ru-MaNGOS

Den · 08.05.2011, 11:41

**Vladimir** · 08.05.2011, 18:06

Я добавлю вскоре. Небольшое замечание

Код:

ReactorRunnable():

Лучше

Код:

ReactorRunnable() :

Я добавил соответсвующите рекомендациии в https://github.com/mangos/mangos/wiki/CodingStandards. По поводу пробелов после имени/for/while у mangos team нет пока однозначности, может пока не стоит их трогать...

[added]После некоторого обсуждения в team https://github.com/mangos/mangos/wiki/CodingStandards обновлены для
if/for/switch/while -> пробел после
function_name() -> без пробела как у вас в пачте и сделано.

По поводу func(0) без пробела - это не сответствует GNU code style. НО главная проблем в GNU code style вариантов сложность быстрого чтения вложенных вызовов функций в выражении
func1 (func2 (0), func3 (func4 (1))) - аргументы вложенных вызовов приклеиваются к , а пробеды вносит визуально разбиение не в том месте и затрудняют быстрый разбор выражения взглядом на аргументы: func1(func2(0), func3(func4(1)))

[added2]
В [11442]

08.05.2011, 11:41	#1
Den Ученый Регистрация: 10.03.2010 Адрес: Бобруйск Сообщений: 284 Сказал(а) спасибо: 213 Поблагодарили 98 раз(а) в 84 сообщениях	[11442] Cleanup codestyle in WorldSocketMgr.cpp & WorldSocketMgr.h http://paste2.org/p/1404807

08.05.2011, 18:06	#2
Vladimir MaNGOS Dev Регистрация: 09.02.2010 Сообщений: 594 Сказал(а) спасибо: 315 Поблагодарили 438 раз(а) в 181 сообщениях	Я добавлю вскоре. Небольшое замечание Код: ReactorRunnable(): Лучше Код: ReactorRunnable() : Я добавил соответсвующите рекомендациии в https://github.com/mangos/mangos/wiki/CodingStandards. По поводу пробелов после имени/for/while у mangos team нет пока однозначности, может пока не стоит их трогать... [added]После некоторого обсуждения в team https://github.com/mangos/mangos/wiki/CodingStandards обновлены для if/for/switch/while -> пробел после function_name() -> без пробела как у вас в пачте и сделано. По поводу func(0) без пробела - это не сответствует GNU code style. НО главная проблем в GNU code style вариантов сложность быстрого чтения вложенных вызовов функций в выражении func1 (func2 (0), func3 (func4 (1))) - аргументы вложенных вызовов приклеиваются к , а пробеды вносит визуально разбиение не в том месте и затрудняют быстрый разбор выражения взглядом на аргументы: func1(func2(0), func3(func4(1))) [added2] В [11442] __________________ Так как устал объяснять знайте ICQ не пользуюсь Последний раз редактировалось Vladimir; 08.05.2011 в 19:21.

Похожие темы
Тема	Автор	Раздел	Ответов	Последнее сообщение
[11442] Cleanup codestyle in WorldSocketMgr.cpp & WorldSocketMgr.h	newsbot	CMaNGOS Commits	0	08.05.2011 19:20
[11439] Cleanup codestyle in Bag.cpp & Bag.h	newsbot	CMaNGOS Commits	0	08.05.2011 05:12
[11439] Cleanup codestyle in Bag.cpp & Bag.h	Den	Принятые патчи	1	08.05.2011 05:11