roboforum.ru

**Сергей** » 30 июл 2009, 18:03

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=** » 02 авг 2009, 18:17

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

**Сергей** » 02 авг 2009, 18:28

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

**=DeaD=** » 02 авг 2009, 18:45

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

**Vooon** » 03 авг 2009, 00:14

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

**=DeaD=** » 03 авг 2009, 06:09

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

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

**Vooon** » 03 авг 2009, 14:20

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

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

**=DeaD=** » 03 авг 2009, 14:24

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

**Vooon** » 03 авг 2009, 14:38

Учусь я, неужели не видно? Я под PC давно не писал на Си, вот и хочу вспомнить

Кстати парсер вполне готов и отлажен, ты можешь использовать его в своей либе для Qt.

**=DeaD=** » 03 авг 2009, 14:40

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

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

**Vooon** » 04 авг 2009, 17:02

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

В комплекте идет програмка 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** » 08 авг 2009, 01:14

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

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

**Vooon** » 12 авг 2009, 20:05

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

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

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

**Duhas** » 12 авг 2009, 21:31

пушки?

**=DeaD=** » 12 авг 2009, 21:34

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

roboforum.ru

[liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Re: [liborfa.so] Pure C API

Кто сейчас на конференции