Comment 0 for bug 813819

Revision history for this message
Chase Douglas (chasedouglas) wrote :

The current definition of GeisSubscriptionFlags is:

typedef enum GeisSubscriptionFlags
{
  GEIS_SUBSCRIPTION_NONE = 0x0000,
  GEIS_SUBSCRIPTION_GRAB = 0x0001,
  GEIS_SUBSCRIPTION_CONT = 0x0002
} GeisSubscriptionFlags;

This prevents the following code from working:

GeisSubscriptionFlags flags = GEIS_SUBSCRIPTION_GRAB | GEIS_SUBSCRIPTION_CONT;

This is because the value of the result does not match any in the enum. Geis should use an anonymous enum for the flags definition, but a typedef of an int for GeisSubscriptionFlags.

A workaround is to define the flags variable as an int.