Re: [avr-gcc-list] generic queue library for AVR GCC?

[Richard Urwin]

On Tuesday 16 Nov 2004 9:36 am, David Brown wrote:
>     static inline unsigned char begin_critical_section(void) {
>         unsigned char s = SREG;
>         cli();
>         return s;
>     }
>

But you still have to declare a variable to hold the return value, and 
that was what broke the #define version.

Given an optimising compiler:

typedef enum {cs_start, cs_end} cs_enum;

static inline unsigned char critical_section(cs_enum dir)
{
        static unsigned char s;

 switch(dir)
 {
 case cs_start:
  s = SREG;
         cli();
  break;
 case cs_end:
  SREG = s;
 }
}

Not much bug protection there though.


-- 
Richard Urwin