Исходное сообщение
"Леннарт Поттеринг представил демонстрационную библиотеку lib..." Отправлено xxx, 03-Ноя-11 16:54
Кто-нибудь по ссылкам ходил? А то смотрю все пульсаудио обсуждают. Почитал рекомендации, сделал вывод: ЛП (Лунар Потнинг) и есть КЭП. Хотя с autotools я не согласен. Поглядел код, видимо ЛП свои рекомендации не читал. >Function calls with 15 arguments are a bad idea. Ну да видимо, 6 аргументов функции + va_list args это всё таки не 15. >functions should return int and negative errors instead of NULL, return NULL in malloc() is fine, return NULL in fopen() is not! fopen ему не понравился, а чего тогда сам интересно сплош и рядом NULL возвращает? >Also, hacking is not a contest of finding the shortest possible     function name. And nobody cares about your 80ch line limit Да-да, пишите километровые шняги. А про читабельность кода он слышал что-нибудь? Правильно тут подметили: >а нет - в отношении его рекомендаций тут лучше наверно "Леннарт не читатель, Леннарт писатель!" Ну и отсебя добавил бы в рекомендации. Леннарт! Попробуй собери свою библиотеку с -Wall -Werror, хотябы просто ради интереса. 180 ABC_EXPORT struct abc_ctx *abc_unref(struct abc_ctx *ctx) 181 { 182         if (ctx == NULL) 183                 return; 184         ctx->refcount--; 185         if (ctx->refcount > 0) 186                 return ctx; 187         info(ctx, "context %p released\n", ctx); 188         free(ctx); 189         return NULL; 190 }

Ваше сообщение
Имя*:
EMail:	Для отправки ответов на email укажите знак ! перед адресом, например, !user@host.ru (!! - не показывать email). Более тонкая настройка отправки ответов производится в профиле зарегистрированного участника форума.
Заголовок*:
Сообщение*:	> Кто-нибудь по ссылкам ходил? А то смотрю все пульсаудио обсуждают. > Почитал рекомендации, сделал вывод: ЛП (Лунар Потнинг) и есть КЭП. Хотя с > autotools я не согласен. > Поглядел код, видимо ЛП свои рекомендации не читал. >>Function calls with 15 arguments are a bad idea. > Ну да видимо, 6 аргументов функции + va_list args это всё таки > не 15. >>functions should return int and negative errors instead of NULL, return NULL in malloc() is fine, return NULL in fopen() is not! > fopen ему не понравился, а чего тогда сам интересно сплош и рядом > NULL возвращает? >>Also, hacking is not a contest of finding the shortest possible > function name. And nobody cares about your 80ch > line limit > Да-да, пишите километровые шняги. А про читабельность кода он слышал что-нибудь? > Правильно тут подметили: >>а нет - в отношении его рекомендаций тут лучше наверно "Леннарт не читатель, Леннарт писатель!" > Ну и отсебя добавил бы в рекомендации. Леннарт! Попробуй собери свою библиотеку > с -Wall -Werror, хотябы просто ради интереса. > 180 ABC_EXPORT struct abc_ctx *abc_unref(struct abc_ctx *ctx) > 181 { > 182 if (ctx == > NULL) > 183 > return; > 184 ctx->refcount--; > 185 if (ctx->refcount > > 0) > 186 > return ctx; > 187 info(ctx, "context %p > released\n", ctx); > 188 free(ctx); > 189 return NULL; > 190 }
	Введите код, изображенный на картинке: