[cdev] реализация утилиты find

[Grigoriy A. Sitkarev]

Для начала.

1. Приведите пожалуйста стиль в порядок. Я неоднократно вас
предупреждал, что у нас есть требование - соблюдать стиль. Если вы
забыли, где про это можно прочитать, напоминаю - Linux Kernel Coding
Style. Настройте свой текстовый редактор чтобы TAB был размером в 8
пробелов. Отступы в файлах с исходным текстом сохранять TAB-ами.

http://www.kernel.org/doc/Documentation/CodingStyle

Точку с запятой ставить после фигурных скобок не нужно везде. Например:

if (a == b) {

}; <--- здесь не надо точку с запятой, она лишняя.

Она нужна только в описании структуры или union.

struct my_struct {
        int a;
        int b;
}; <--- здесь нужна точка с запятой.

2. Вы могли бы сделать так  чтобы ваша утилита выводила подсказку по
её использованию, если ей передали неизвестные аргументы.

3. Я проверил вашу утилиту на большом каталоге со множеством
подкаталогов и она  у меня "сломалась" с сообщением:

не удалось открыть каталог : Too many open files

Программа не работает.

4. Почему у вас сообщения и на английском и на русском языках в куче?
Локаль у вас не используется (setlocale(3) и т.д.), поэтому должно
быть на каком-то одном языке.

Есть ещё множество замечаний, но для начала приведите пожалуйста в
порядок всё вышеописанное.

--
Г.А.

14 марта 2010 г. 9:39 пользователь Лена Довжко <lenad89 на list.ru> написал:
>
>
> _______________________________________________
> cdev mailing list
> cdev на wiki.syktsu.ru
> http://wiki.syktsu.ru/cgi-bin/mailman/listinfo/cdev
>
>