Ru-MaNGOS

Вернуться   Ru-MaNGOS > Ядро > Патчи > Принятые патчи

Важная информация

Принятые патчи Иногда выкладывают патчи, которые потом в итоге все-таки принимают в ядро.

Повод для гордости.

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

http://paste2.org/p/1404807
Den вне форума  
Пользователь сказал cпасибо:
Vladimir (08.05.2011)
Старый 08.05.2011, 18:06   #2
Vladimir
MaNGOS Dev
 
Регистрация: 09.02.2010
Сообщений: 594
Сказал(а) спасибо: 315
Поблагодарили 438 раз(а) в 181 сообщениях
Vladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небесVladimir Как свет с небес
По умолчанию

Я добавлю вскоре. Небольшое замечание
Код:
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.
Vladimir вне форума  
Пользователь сказал cпасибо:
Den (08.05.2011)
 

Опции темы Поиск в этой теме
Поиск в этой теме:

Расширенный поиск
Опции просмотра

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

BB коды Вкл.
Смайлы Вкл.
[IMG] код Вкл.
HTML код Выкл.


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
[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


Текущее время: 19:51. Часовой пояс GMT +3.


ru-mangos.ru - Русское сообщество MaNGOS
Главная цель проекта MaNGOS - обучающая, поэтому разрешается использовать исходный код и собранную программу только для образовательных целей.
Вы не можете использовать MaNGOS в коммерческих целях, а также не разрешается устанавливать публичные серверы на базе MaNGOS.
Любое копирование материалов, информации в любом виде без указания источника - форума Ru-MaNGOS будет считаться нарушением авторских прав и нарушением Уголовного Кодекса РФ, ст. 146 ст. 147.
Перевод vBulletin: zCarot