Is this a common coding style to embed defines in the struct definition like this?  Also maybe make them an enum where they are used - it would make the gdb output cleaner