На новую версию врядли тянет, но кое что поправил на свой взгляд. В частности со строк типа "FreaxNet 127.0.0.1" сразу кидает на run_error + добавил проверку на случай если при -p ничего не указано (хотя работает и без этого, но подумал пусть будет о.о). Дефолтом пауза 0.
Кстати, только сейчас заметил такую строку: argv[1]==NULL. Мне такой код кажется безумно кривым, хотя на C я не пишу уже 3 года. argc для чего тогда нужен?
Кстати, только сейчас заметил такую строку: argv[1]==NULL. Мне такой код кажется безумно кривым, хотя на C я не пишу уже 3 года. argc для чего тогда нужен?
вот тут согласен, я б заменил эту проверку на проверку что-то типо if(argc==1)... А по поводу FreaxNet -p -h 127.0.0.1 - вроде такое должно по задумке работать...
Кстати, только сейчас заметил такую строку: argv[1]==NULL. Мне такой код кажется безумно кривым, хотя на C я не пишу уже 3 года. argc для чего тогда нужен?
вот тут согласен, я б заменил эту проверку на проверку что-то типо if(argc==1)... А по поводу FreaxNet -p -h 127.0.0.1 - вроде такое должно по задумке работать...
Кстати, только сейчас заметил такую строку: argv[1]==NULL. Мне такой код кажется безумно кривым, хотя на C я не пишу уже 3 года. argc для чего тогда нужен?
вот тут согласен, я б заменил эту проверку на проверку что-то типо if(argc==1)... А по поводу FreaxNet -p -h 127.0.0.1 - вроде такое должно по задумке работать...
Почему? Этот ключ требует наличия операнда.
ну это опять же вопрос к автору... конечно можно и на хелп в таком случае переправлять справедливо.
Ну а command=argv[3] то зачем? Кстати, 0 возвращается если всё хорошо, 1 - иначе.
Что-то я раскритиковался smile.gif. Больше не буду лезть в процесс написания.
без той строки как будет работать -p 2 127.0.0.1 ? ну по поводу 0/1, да, но в этом случае не принципиально...
Не должно работать, т.к. пропущен ключ -h.
а если почитать тот самый run_error, то там такая комбинация в качестве примера приведена
Прошу прощения, возможно я не прав. Просто эта строка там действительно не нужна. Эту проблему нужно решать иначе. Это костыль, который работает в данном случае, но не будет работать в других (пример: 127.0.0.1 -p 2).
а если почитать тот самый run_error, то там такая комбинация в качестве примера приведена
+1
Глядя на ваше обсуждение ошибок и недочетов FreaxNet, я познал истинный метод вылавливания багов из кода. Очень эффективный метод, должен вам признаться Все замечания и вылавленные баги будут учтены в следующей версии!
Ну а command=argv[3] то зачем? Кстати, 0 возвращается если всё хорошо, 1 - иначе.
Что-то я раскритиковался smile.gif. Больше не буду лезть в процесс написания.
без той строки как будет работать -p 2 127.0.0.1 ? ну по поводу 0/1, да, но в этом случае не принципиально...
Не должно работать, т.к. пропущен ключ -h.
а если почитать тот самый run_error, то там такая комбинация в качестве примера приведена
Прошу прощения, возможно я не прав. Просто эта строка там действительно не нужна. Эту проблему нужно решать иначе. Это костыль, который работает в данном случае, но не будет работать в других (пример: 127.0.0.1 -p 2).
ну в таком случае должно кидать на run_error, т.к. по хелпу конструкция должна выглядеть как FreaxNet [Options] IP_Adrr. Именно в таком порядке. Другое дело что это не проверяется пока никак.
а если почитать тот самый run_error, то там такая комбинация в качестве примера приведена
+1
Глядя на ваше обсуждение ошибок и недочетов FreaxNet, я познал истинный метод вылавливания багов из кода. Очень эффективный метод, должен вам признаться
а если почитать тот самый run_error, то там такая комбинация в качестве примера приведена
+1
Глядя на ваше обсуждение ошибок и недочетов FreaxNet, я познал истинный метод вылавливания багов из кода. Очень эффективный метод, должен вам признаться
ну в таком случае должно кидать на run_error, т.к. по хелпу конструкция должна выглядеть как FreaxNet [Options] IP_Adrr. Именно в таком порядке. Другое дело что это не проверяется пока никак.
Взглянул сейчас на ман... Он тоже мягко говоря не доработан, там есть явные противоречия.
ну в таком случае должно кидать на run_error, т.к. по хелпу конструкция должна выглядеть как FreaxNet [Options] IP_Adrr. Именно в таком порядке. Другое дело что это не проверяется пока никак.
Взглянул сейчас на ман... Он тоже мягко говоря не доработан, там есть явные противоречия.
вы имеете ввиду сам ман или то что в процессе выполнения программы появляется в качестве хелпа?
ну в таком случае должно кидать на run_error, т.к. по хелпу конструкция должна выглядеть как FreaxNet [Options] IP_Adrr. Именно в таком порядке. Другое дело что это не проверяется пока никак.
Взглянул сейчас на ман... Он тоже мягко говоря не доработан, там есть явные противоречия.
вы имеете ввиду сам ман или то что в процессе выполнения программы появляется в качестве хелпа?
Отдаешь код двум (должно быть четное количество) энтузиастам-программистам на растерзание. Спустя совсем не продолжительное время всплывают такие нюансы и баги, о которых даже и подумать не мог, замечаешь совсем очевидные ошибки и просто опечатки.
Можно выпускать версию 0.063, просто учтя все, что вы тут написали, т.е. добавив проверки и т.д. На днях выложу.