----- Original Message ----- > Looks good to me. > > Funny that we have: > #define XvNumReasons (XvLastReason + 1) > and a few lines below it: > #define XvAnyReasonMask ((1L<<XvNumReasons) - 1) Ah stoopid me... /me clears glasses. Sorry for the noise. > Anyway, > > Reviewed-by: Olivier Fourdan <ofourdan at redhat.com> >