[cdev] Утилита Echo

Карабанова Елена ElenaKarabanova17 на yandex.ru
Пт Мар 19 14:20:54 MSK 2010


Учла замечания, просто  прошлый раз нажала на кнопку Переформатировать код вот все и сбилось

> Это не Linux Kernel Coding Style. Вы не смотрели LKCS.
> Скобки фигурные не так расставляются и пробелов не должно быть после `(' 
> и до `)'.
> После условия нельзя писать на той-же строке выражение.
> if (a) a += b; <-- не правильно
> if (a)
> a += b; <-- правильно
> Карабанова Елена пишет:
> > Развернуть
> > Здравствуйте
> > 
> > я не понимаю в чем моя первая ошибка, 
> Я в другой исходник смотрел и ошибся. Бывает.
> > Развернуть
> > > Развернуть
> > > Есть глобальная переменная write_newline и локальная переменная в 
> > > main с таким же именем. У вас из-за этого неправильно обрабатывается 
> > > опция -n.
> > моя глобальная переменная была задана один раз, в main я её второй раз не объявляла, опция -n работает
> Всё верно у вас.
> > Развернуть
> > подсказку исправила на стандартную
> Хорошо.
> > Развернуть
> > в функцию передаются только 2 параметра
> Есть ещё нюанс.
> И я потребую чтобы вы его тоже поправили. Если значение переменой нас 
> интересует как булево (true/false) то в операциях сравнения мы им и 
> пользуемся как логическим.
> У вас:
> if (put_newline == 1)
> а по идее надо бы:
> if (put_newline)
> Конечно, они тогда и с нулём не сравниваются а было бы:
> if (!put_newline)
> > Развернуть
> > при компиляции не выдается предупреждений
> Это так.
> Но в do_echo_with_escapes нет обработки случая, если после `\' 
> встретился неподдерживаемый символ, например '\z' не будет обработано. 
> Нет также обработки случая, когда после `\' вообще больше ничего нет, и 
> в argv[i] был только этот один символ всего. Я вас предупреждал, что 
> давал вам только скелет. Подумайте, как просто и красиво сделать 
> обработку этих возможных ситуаций.
> --
> Г.А.
> _______________________________________________
> cdev mailing list
> cdev на wiki.syktsu.ru
> http://wiki.syktsu.ru/cgi-bin/mailman/listinfo/cdev
> 

Здесь спама нет http://mail.yandex.ru/nospam/sign
----------- следущая часть -----------
A non-text attachment was scrubbed...
Name: main.c
Type: application/octet-stream
Size: 2007 bytes
Desc: отсутствует
URL: <http://amplab.syktsu.ru/pipermail/cdev/attachments/20100319/60f65106/attachment.obj>


Подробная информация о списке рассылки cdev