Pull to refresh
62
0
Алексей Кутумов @prograholic

Руководитель команды разработки

Send message
В этом и есть вся суть code review, потому что автор кода относится к свому коду предвзято (грубо говоря, он его считает правильным, идеальным), поэтому задача коллег — сделать этот код еще лучше: указать на опечатки, ошибки, подсказать более правильный путь решения.
Я рад что вы заметили)
Мы пробовали использовать статический анализатор кода pvs, но к сожалению msvc — не основная среда разработки (пишем время от времени), и поэтому как-то не прижался у нас этот инструмент, возможно с другими анализаторами больше повезет. Кстати, возможно прикрутить pvs studio к CI серверу?
Эта статья безусловно не претендует на полноценное описание всей практики, скорее это чисто субъективное мнение на основе небольшого опыта, упорядочивание собственных знаний.

Можете порекомендовать литературу по code review? Я могу расширить статью дополнительной информацией.
Спасибо всем за комментарии, я добавил в статью информацию о версии wine-1.4. Все успешно работает
Попробую на версии 1.4, просто когда я патчил wine, версия была 1.2.3. Спасибо за информацию
Насчет мух и котлет я с вами согласен. Однако багрепорт позволит не потерять эту особенность и донести ее до разработчиков.

p.s. В багтрекере boost такой проблемы не нашел, если что могу помочь автору с оформлением тикета
Тогда уж, надо было баг-репорт отправить разработчикам asio, будет больше пользы от данной статьи

Information

Rating
Does not participate
Location
Новосибирск, Новосибирская обл., Россия
Date of birth
Registered
Activity