Вопрос по макросу в С, Сброс/установка битов в двух разных портах Опции

[mempfis_]

Помогите пожалуйста с макросом
Необходимо с помощью макроса записать установку бита в одном порту и сброс в другом т.е. чтобы в макросе выполнялись такие строчки:
PORTA |= (1<<BIT1)
PORTB &= ~(1<<BIT2)

В IAR нашёл макрос
#define input_block_dec(PORT,ADDRESS,COUNT)\
{ \
unsigned char i;\
unsigned char *addr=(ADDRESS);\
for(i=0;i<(COUNT);i++)\
*addr--=(PORT);\
}

Пробовал подобным образом переделать под свой:
#define Set1()\
{ \
PORTA |= (1<<BIT1);\
PORTB &= ~(1<<BIT2);\
}

Пока не использую его компилятор не ругается, только вставляю где-нибудь пишет что нужна ;
Компилятор IAR 5.10a

[Сергей Борщ]

Пока не использую его компилятор не ругается, только вставляю где-нибудь пишет что нужна ;

Покажите, как используете.

[mempfis_]

Покажите, как используете.

собственно макрос:

Код

#define Set1()\
{ \
PORTA |= (1<<BIT1);\
PORTB &= ~(1<<BIT2);\
}

использую в таком виде:

Код

...код..........
//пытаюсь установить бит одного порта и сбросить в другом
Set1();
...код......

макрос из одной строчки работает:

Код

#define Set1() PORTA |= (1<<BIT1)

А из двух не хочет
И ещё попутно вопрос каково назначение \ ?

[Палыч]

Например, можно записать так
#define Set1() ( PORTA |= (1<<BIT1), PORTB &= ~(1<<BIT2) )
Это, если всегда для одних и тех же битов одних и тех же портов. Иначе - добавить параметры
#define Set1(Port1, Bit1, Port2, Bit2) ( Port1 |= (1<<Bit1), Port2 &= ~(1<<Bit2) )
Вызывать
Set1();
или
Set1(PORTA, BIT1, PORTB, BIT2);

[mempfis_]

Например, можно записать так
#define Set1() ( PORTA |= (1<<BIT1), PORTB &= ~(1<<BIT2) )
Это, если всегда для одних и тех же битов одних и тех же портов. Иначе - добавить параметры
#define Set1(Port1, Bit1, Port2, Bit2) ( Port1 |= (1<<Bit1), Port2 &= ~(1<<Bit2) )
Вызывать
Set1();
или
Set1(PORTA, BIT1, PORTB, BIT2);

Спасибо Ваш вариант работает
Кстати и мой тоже (там была ошибка - случайно не удалил одну скобочку )
Но вопрос про "\" всёже остался

[Палыч]

И ещё попутно вопрос каково назначение \ ?

Обратная косая означает: следующая строка - продолжение предыдущей (т.е. исключить возврат каретки). Обычно используется для записи в несколько строк элементов, которые по правилам должны распологаться в одной строке (например, #define)

[mempfis_]

Обратная косая означает: следующая строка - продолжение предыдущей (т.е. исключить возврат каретки). Обычно используется для записи в несколько строк элементов, которые по правилам должны распологаться в одной строке (например, #define)

Всем огромное спасибо
Проблема решена

[mdmitry]

Посмотрите вывод препроцессора. Сильно помогает разобраться с макросами, ошибки сразу видны после препроцессора.

[Сергей Борщ]

Проблема решена

Не, вы еще не на все грабли наступили . Попробуйте так:

Код

if(var) Set1(); else var = 1;

[mdmitry]

Не, вы еще не на все грабли наступили . Попробуйте так:

Код

if(var) Set1(); else var = 1;

+1. Без препроцесора не разобраться

[mempfis_]

Не, вы еще не на все грабли наступили . Попробуйте так:

Код

if(var) Set1(); else var = 1;

Если не затруднит, расскажите в чём могут быть грабли?
И что Вы имели ввиду под этой строчкой

Код

if(var) Set1(); else var = 1;

Вызоа макроса в операторе if-else в программе?
Проверил у себя в вроде всё работает.

[mdmitry]

Определение блока (фигурные скобки) может сыграть злую шутку:

Код

#define Set1()/
{ /
PORTA |= (1<<BIT1);/
PORTB &= ~(1<<BIT2);/
}

if(var) Set1(); else var = 1;

Разбираем подстановку:

Код

if(var)
{
PORTA |= (1<<BIT1);
PORTB &= ~(1<<BIT2);
}
;
else var = 1;

Посмотрите как сделаны макросы в GCC. Заложены очень хорошие идеи.

[kurtis]

Может проще отказаться от использования макросов и использовать встраиваемые функции (inline functions)???

[mempfis_]

Определение блока (фигурные скобки) может сыграть злую шутку:

Код

#define Set1()/
{ /
PORTA |= (1<<BIT1);/
PORTB &= ~(1<<BIT2);/
}

if(var) Set1(); else var = 1;

Разбираем подстановку:

Код

if(var)
{
PORTA |= (1<<BIT1);
PORTB &= ~(1<<BIT2);
}
;
else var = 1;

Посмотрите как сделаны макросы в GCC. Заложены очень хорошие идеи.

Спасибо, понял в чём может быть проблема.
К счастью эти макросы не будут применятся в условных операторах (только чтобы установить/сбросить нужные пины контроллера). Если что то просто перепишу их без фигурных скобок

[mdmitry]

Пример макроса (идея из GCC)

Код

#define Set1() \
do { \
PORTA |= (1<<BIT1);\
PORTB &= ~(1<<BIT2);\
} while (0)

Сергей Борщ в форуме по AVR комментировал используемую здесь идею. Оптимизатор компилятора выкинет цикл, но оставит тело цикла. Безопасное использование гарантировано. Сам пишу сложные макросы на основе этой идеи.