Тахометр, нужна помощь.

[ew3ds]

akegor, Оставить как было? Как правильно?

[mike-y-k]

ew3ds, будете этот код таскать по разным компиляторам и разным чипам или просто остановитесь на достигнутом?
Если первое - правильней избежать неявных преобразований, если второе - без разницы…

[akegor]

Как правильно?

Правильно - не полагаться на "кефаль, которая думает за вас". Если загрузка должна производиться побайтно, значит так и делать.

будете этот код таскать по разным компиляторам и разным чипам или просто остановитесь...

Лучше сразу привыкать к непритягиванию ошибок, которые, бывает, не так-то и просто найти.

Говорят, правильный прогрАммер, ложась спать, ставит рядом стакан с водой, на случай, если ночью захочет пить. И пустой - если не захочет.
Другими словами - под контролем должно быть всё!

[ew3ds]

или просто остановитесь на достигнутом?

Именно так. Это будет один, максимум два экземпляра для себя.

[mike-y-k]

akegor, таки тогда нужно писать такие операции с учетом разрядности используемого железа… это уже к вопросу о правильных программистах…

ew3ds, тогда оставьте в коде комментарий о неявном присвоении, если будут проблемы - проверите. Или просто напишите операцию без #define прямым присвоением (те 2 операции присвоения из #define). Для 8-битного МК это будет максимально правильно…

[j-Roger]

Пару слов насчет правильности: переменную, которая используется и в прерывании и вне его (в данном проекте это переменная start в файле main.c), правильно объявлять с квалификатором volatile во избежание сюрпризов.
В частности, в данном проекте компилятор gcc при включенной оптимизации по размеру сгенерировал инструкцию бесконечного цикла, если у переменной start отсутствовал квалификатор volatile.

Во-о-он на картинке разница с 'volatile' и без оного:


Одним словом, это зависит. Компилятору доверяем, но проверяем.

[ew3ds]

правильно объявлять с квалификатором volatile во избежание сюрпризов.

...
#define NO_PULSES_INTERVAL 50000

bit
start;

volatile bit;

long
data;
...

...
interrupt [EXT_INT0] void ext_int0_isr(void)
{
static bit
process;

volatile ;

static unsigned char
...

Подскажите, так будет правильно?

[Signalshik2]

volatile bit start;

[ew3ds]

Signalshik2, вот так не ругается

...
#define NO_PULSES_INTERVAL    50000

//bit
//start;

//volatile bit;
 volatile bit start;

long 
data;
...

...
interrupt [EXT_INT0] void ext_int0_isr(void)
{
 static bit 
 process; 

 volatile; 
 //volatile bit start;
 
 static unsigned char 
 cnt;
...

[Signalshik2]

Тип и название переменной нужно в одну строку писать. Просто volatile; и точка с запятой - это нужно удалить, это бред.