On 7 January 2014 20:03, Peter Maydell <peter.mayd...@linaro.org> wrote: > +#ifdef HOST_WORDS_BIGENDIAN > +#define offsetoflow32(S, M) (offsetof(S, M + sizeof(uint32_t))
Mismatched brackets, won't build on bigendian hosts. (I happened to randomly run cppcheck, or I'd not have spotted that.) Reroll #2 coming up later, though I'll just send the cover letter... thanks -- PMM