typedef struct
{
    unsigned long thang;
}
MYSTRUCT;

void raw_to_mystruct(unsigned char *raw, MYSTRUCT * x)
{
    x->thang = 0L;
    x->thang |= raw[3];
    x->thang <<= 8;
    x->thang |= raw[2];
    x->thang <<= 8;
    x->thang |= raw[1];
    x->thang <<= 8;
    x->thang |= raw[0];
}

int main(void)
{
    unsigned char raw[4];
    MYSTRUCT mystruct;
    raw_to_mystruct(raw, &mystruct);
    return 0;
}

Anybody know how to simplify (or more specifically optimise) the code in
the raw_to_mystruct function?  I was thinking along the lines of

  x->thang = (raw[3] << 8) | (raw[2] << 8) | (raw[1]
<< 8) | raw[0];

but am unsure if that's a correct translation.  Thanks. 

--- Msged/2 3.40ac alpha 2