On 09/05/2014 12:30 PM, Savolainen, Petri (NSN - FI/Espoo) wrote:
Why not put this to enum?  I have:

typedef enum {
          ODP_SHM_THREAD = 1,        /**< Memory accessible by threads.  */
          ODP_SHM_PROC = 2,
}

It might be ODP_SHM_THREAD is not the good name. And we can change it to
ODP_SHM_DEFAULT. Which is more readable then 0 flag.
Or we can go with enum and bind default to 0.

Maxim.
I used uint32_t since there can be multiple flags you can enable 
simultaneously. One example is: ODP_SHM_SW_ONLY|ODP_SHM_PROC, and we may come 
up with more such flags in the future.

Enum is int, so you cannot tell how many flags are supported in total.

-Petri

Ok, if everybody agree, I'm ok to have it int. Please send v2, I will update IPC path on top of yours.

Thanks,
Maxim.

_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to