ulidtko 11.02.2013 17:22

Вот когда на код ревью ноют про unnecessary whitespace changes that added extra "noise" — это признак пидораса, или я всё же тупой?

dev
#tinhoo
re 25 comments !recommend ~bookmark Ssubscribe
1. Crazy_Owl 11.02.2013 17:23 Azatoth

по ебалу им бей с ноги. И шелухой от семок харкни в лицо потом, чтоб знали кто тут девелопер, а кто петух-ревьюер.

#tinhoo/1
rereply
2. generatorglukoff 11.02.2013 17:23 Воркота

у вас чо, кодестайла нету?

#tinhoo/2
rereply
3. hirthwork 11.02.2013 17:24 mcabber

ты тупой. после того как файл написан и закоммичен, любое изменение в нём должно проходить через бритву Оккама. Исключения составляют только большие коммиты, которые ничего кроме правки стиля под утверждённых свыше не делают, ну и те коммиты, которые и так больше половины файла затрагивают.

#tinhoo/3
rereply
4. ojab 11.02.2013 17:29 YGG!

ты тупой, unnecessary whitespace changes должны отдельным патчем идти

#tinhoo/4
rereply
5. 238328 11.02.2013 18:09

japh?

#tinhoo/5
rereply
6. 238328 → ojab /4 11.02.2013 18:09

нет ты

#tinhoo/6
rereply
7. ulidtko → hirthwork /3 11.02.2013 19:03

через твою тупую бритву Оккама у меня всё проходит, я изменений «от балды» не делаю

#tinhoo/7
rereply
8. hirthwork → ulidtko /7 11.02.2013 19:04 mcabber

это как это она у тебя проходит? ты что, в коммит мессажде напишешь «Убрал пробел, ибо противоречит корпоративному стайлгайду»? Сдаётся мне, что ты напишешь «Починил такую-то багу» и ни слова про пробелы

#tinhoo/8
rereply
9. gelraen 11.02.2013 20:48 work-laptop

ты тупой и whitespace cleanup надо делать отдельным коммитом. А вообще все тупые и надо сразу нормально whitespace расставлять.

#tinhoo/9
rereply
10. ulidtko → gelraen /9 12.02.2013 11:07

ебать вы умные канеш, здесь (в аккуреве) нет КОММИТОВ

#tinhoo/10
rereply
11. generatorglukoff → ulidtko /10 12.02.2013 11:09 Воркота

што? блядь, ответь, у вас кодестайл есть? или один ревьювер юзает 4 пробела, а другой — таб, который выставлен на 8 позиций?

#tinhoo/11
rereply
12. ulidtko → generatorglukoff /11 12.02.2013 11:09

да

#tinhoo/12
rereply
13. hirthwork → ulidtko /12 12.02.2013 11:11 mcabber

ебать у вас пиздец там

#tinhoo/13
rereply
14. generatorglukoff → ulidtko /12 12.02.2013 11:15 Воркота

пиши согласно кодестайлу. посылай нахуй. кодестайл правь в отдельном коммите.

#tinhoo/14
rereply
15. ojab → generatorglukoff /14 12.02.2013 11:15 YGG!

у них нет коммитов, мудило

#tinhoo/15
rereply
16. ulidtko → hirthwork /13 12.02.2013 11:19

оперативно // никому здесь ваши табы с пробелами в хуй не впёрлись, здесь ВАЙТСПЕЙС вида пробелов вокруг =, выравнивания съехавших мемберов вроде
virtual AObject* NthItem(intptr_t n);
- virtual intptr_t FindIndex(AObject *theObject);
+ virtual intptr_t FindIndex(AObject *theObject);
virtual AObject* FirstSuccess(TestFunc testFunc);

этц

#tinhoo/16
rereply
17. ulidtko → generatorglukoff /14 12.02.2013 11:20

а на отсутствие/хуевую проработанность и заэнфоршеность кодестайла кому, мамке твоей жаловаться?

#tinhoo/17
rereply
18. generatorglukoff → ulidtko /17 12.02.2013 11:21 Воркота

твоей // тимлиду ёба

#tinhoo/18
rereply
19. ulidtko → generatorglukoff /18 12.02.2013 11:22

и да, у нас нет коммитов (есть кипы и промоуты, две ваще разных хуйни, если вкратце)

#tinhoo/19
rereply
20. ulidtko → generatorglukoff /18 12.02.2013 11:22

тимлиду похуй лол

#tinhoo/20
rereply
21. generatorglukoff → ulidtko /19 12.02.2013 11:22 Воркота

што блядь? это что еще за хипстерская хуита?

#tinhoo/21
rereply
22. generatorglukoff → ulidtko /20 12.02.2013 11:23 Воркота

а вот это — признак пидораса

#tinhoo/22
rereply
23. ulidtko → generatorglukoff /21 12.02.2013 11:23

accurev

#tinhoo/23
rereply
24. generatorglukoff → ulidtko /23 12.02.2013 11:30

это какой-то пиздец

#tinhoo/24
rereply
25. ulidtko → generatorglukoff /24 12.02.2013 15:22

йеп

#tinhoo/25
rereply
Do you really want to delete ?