Цитата(Rst7 @ Mar 24 2009, 22:16)

Я же специально обратил внимание, что не в форматировании дело. Табуляции, скобочки и прочее - это достаточно интимные пристрастия, при необходимости - переформатируется.
а это чудо:
Код
putc(*p, out), p++;
зачем так писать, ведь можно
putc(*p++, out);
while ( p != endfn ) putc(*p, out), p++;
Несколько разношерстных операторов в одной строке - обычно пишут люди которые не знают, что такое пошаговая отладка. Вот сами скажите, как при использовании такого стиля поставить точку останова на
putc, а не на
while( p != endfn)ну и ...
Код
GLOBAL(void)
jpeg_start_compress (j_compress_ptr cinfo, boolean write_all_tables)
{
тип с маленькой буквы, и не в начале а в конце пишется, что этот тип - указатель. Плохо читаемо, не видно где типы, а где переменные.
Природа переменных не понятна по их имени - т.е. сам указатель, тобиш cinfo в данном случае, никак не показывает, что он указатель, а не экземпляр....
Докучи в исходниках еще и все коментарии с "/*" - клиника.
короче стиль imho очень далек от идеала.
Я бы предпочел хотя бы так:
Код
void JPEG_StartCompress( PJPEG_COMPRESS pCompressInfo, BOOL write_all_tables)
{
По крайней мере так легко можно сразу отличить функции от типов, типы от переменных, структуры от указателей на структуры.