LINUX.ORG.RU

История изменений

Исправление zendrz, (текущая версия) :

Ну конечно бывают. Например, какой-нибудь парсинг строк, особенно если это С, какую бы ты либу не использовал для этого, в комментарии хорошо бы примеры строк привести и что из них получается в итоге. (Но такие примеры надо в тесты переносить как можно скорее). Так же приветствуются комментарии с ответом на вопрос «зачем» это было сделано. Для оптимизации, для воркэраунда кривого API, как временная мера (тогда надо TODO или FIXME добавить) и тп.

(Также, на вопрос «зачем» рекомендуется отвечать в commit message. Для фикса такого-то тикета, для фикса падения при таком-то условии, для совместимости с обновленным API и тп. На вопрос «что сделано» должен хорошо отвечать сам кода. Конечно, надо проявлять гибкость и здравый смысл)

Исходная версия zendrz, :

Ну конечно бывают. Например, какой-нибудь парсинг строк, особенно если это С, какую бы ты либу не использовал для этого, в комментарии хорошо бы примеры строк привести и что из них получается в итоге. Так же приветствуются комментарии с ответом на вопрос «зачем» это было сделано. Для оптимизации, для воркэраунда кривого API, как временная мера (тогда надо TODO или FIXME добавить) и тп.

(Также, на вопрос «зачем» рекомендуется отвечать в commit message. Для фикса такого-то тикета, для фикса падения при таком-то условии, для совместимости с обновленным API и тп. На вопрос «что сделано» должен хорошо отвечать сам кода. Конечно, надо проявлять гибкость и здравый смысл)