roboforum.ru

[Сергей]

Код: Выделить всё • Развернуть

void *pointer = malloc( ... );

free( pointer ); 

...

if( pointer ) // pointer может быть не равен NULL после очистки, либо нужно вручную pointer = NULL после free
{
...

Код: Выделить всё • Развернуть

void *pointer = malloc( ... );

my_free( &pointer ); 

...

if( pointer ) // pointer равен NULL, тк в методе my_free мы его почистили
{
...

Добавлено спустя 8 минут 52 секунды:

Код: Выделить всё • Развернуть

static bool parse(struct parser_vars *self, uint8_t c, struct orfareply *reply)

Вставь проверку на валидность указателей

Код: Выделить всё • Развернуть

struct orfadev *orfa_open(char *port, int baudrate)

так же проверка на указатель.
Сначала проверь возможность открытия open(port, O_RDWR|O_NOCTTY|O_NDELAY) а потом уже делай malloc(sizeof(struct orfadev))

Код: Выделить всё • Развернуть

orfa_close(struct orfadev *device)

Вставь проверку на валидность указателя
Оберни возвратное значение close, чтобы в консоль выводилось сообщение что проблема при закрытии, иногда очень помогает.

Вставь проверку на валидность указателя во всех остальных функциях

[=DeaD=]

2Vooon: А какая задача этой библиотеки? Может кстати сразу её писать как обертку Qt-библиотеки для COM-порта?

[Сергей]

Qt это С++, а здесь чистый Си

[=DeaD=]

Я понимаю, поэтому и спрашиваю - какую задачу решаем, зачем чистый Си?

[Vooon]

С си можно легко сделать привязку к куче языков, а с крестами сложенее.
И я хочу попробовать розиксовый последовательный порт, без лишних прослоек.

[=DeaD=]

2Vooon: Ну если оно аналогично будет и там и там работать - тогда ок, но вроде даже в Qt-шной либе им пришлось для линуха и для винды делать отдельные субклассы и прописывать и там и там индивидуально всё.

Кстати, ты чисто синхронный вариант делать хочешь?

[Vooon]

Я не ставлю перед собой задачу чтобы оно _сразу_ работало под виндой, и поэтому сейчас пишу под POSIX системы.

Нет, отдельный поток чтения.

[=DeaD=]

Ну ладно, посмотрим чего получится, а пока для меня загадка, зачем это делается

[Vooon]

Учусь я, неужели не видно? Я под PC давно не писал на Си, вот и хочу вспомнить
Кстати парсер вполне готов и отлажен, ты можешь использовать его в своей либе для Qt.

[=DeaD=]

Ну если как учебная задача - тогда ясно

Сначала интерфейс восстановлю (завтра-послезавтра надеюсь) и тогда посмотрим чего куда прикрутить

[Vooon]

Отказался от отдельного потока чтения, скинул на на буфер ОС.
Уже работает, но пока только под линуксом.

В комплекте идет програмка lsorfa,
аналогична orintro.py, но менее информативна, т.к. не использует бд драйверов.

Код: Выделить всё • Развернуть

vovan@vovan-laptop:~/src/liborfa$ ./lsorfa /dev/rfcomm1
Drivers:
UID     Version Start   Count
0x0000  1.0     0x00    1
0x0020  1.0     0x02    8
0x0001  1.0     0x0A    2
0x0060  1.0     0x0C    4
0x0031  1.0     0x10    2
0x0040  1.0     0x12    2
vovan@vovan-laptop:~/src/liborfa$

Для сборки нужен CMake.
Правда правила установки я еще не описывал...
cmake .
make

Код: Выделить всё • Развернуть

vovan@vovan-laptop:~/src/liborfa$ ls
bin/  CMakeLists.txt  include/  LICENSE  Makefile  README  src/  tests/
vovan@vovan-laptop:~/src/liborfa$ cmake .
-- The C compiler identification is GNU
-- The CXX compiler identification is GNU
-- Check for working C compiler: /usr/bin/gcc
-- Check for working C compiler: /usr/bin/gcc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Configuring done
-- Generating done
-- Build files have been written to: /home/vovan/src/liborfa
vovan@vovan-laptop:~/src/liborfa$ make
Scanning dependencies of target orfa
[ 33%] Building C object CMakeFiles/orfa.dir/src/orfa.c.o
/home/vovan/src/liborfa/src/orfa.c: In function ‘parse’:
/home/vovan/src/liborfa/src/orfa.c:137: warning: implicit declaration of function ‘toupper’
[ 66%] Building C object CMakeFiles/orfa.dir/src/orfaintro.c.o
Linking C shared library liborfa.so
[ 66%] Built target orfa
Scanning dependencies of target lsorfa
[100%] Building C object CMakeFiles/lsorfa.dir/bin/lsorfa.c.o
/home/vovan/src/liborfa/bin/lsorfa.c:59:10: warning: unknown escape sequence '\s'
Linking C executable lsorfa
[100%] Built target lsorfa
vovan@vovan-laptop:~/src/liborfa$ ls
bin/  CMakeCache.txt  CMakeFiles/  cmake_install.cmake  CMakeLists.txt  include/  liborfa.so*  LICENSE  lsorfa*  Makefile  README  src/  tests/
vovan@vovan-laptop:~/src/liborfa$

[Vooon]

Из qextserialport тиснул работу с виндовым ком-портом, если быть точнее то посмотрел как работать.
И добавил возможность сборки для WIN32, но не проверял т.к. у меня нет инструментария под виндой.
(я сейчас на даче)

Для сборки нужен CMake (очень понравился, удобнее сконса).

[Vooon]

Сегодня разобрался с работой на винде.
Есть особенность, для ком-портов старше 9 нужно добавлять \\.\ ("\\\\.\\" на си)

Обновил первый пост, загрузил pdf созданный доксигеном.

Добавлено спустя 2 минуты 34 секунды:
Да, на юид 0xff00 пока не обращайте внимание, я начал драйвер пушки писать...

[Duhas]

пушки?

[=DeaD=]

Следующим будет драйвер постановки дымовых заграждений и включения приборов ночного видения