On Tue, 3 Oct 2006 16:18:44 +0100 (BST) "Maciej W. Rozycki" <macro@xxxxxxxxxxxxxx> wrote: > + sbmac_state_t sbm_state; /* current state */ argh. The reader looks at this and doesn't know if it's an integer, a void*, a struct usb_ac_header_descriptor** or what. enum sbmac_state smb_state; is nicer. It has information.