Le lundi 15 septembre 2014, 23:00:40 Katerina Barone-Adesi a écrit :
> The -1/unsigned approach is a bit ugly, but that's a minor nit. I do
> not object to merging it as-is; I don't think it needs to be changed.

Well, pick the ugliness you prefer:
- use int32_t (or larger) even though the API expects uint32_t,
- use zero as invalid value instead of -1,
- keep it as is.

I can change it if libav has a coding policy on the topic, but meanwhile I 
think the current approach is the least ugly.

-- 
Rémi Denis-Courmont
http://www.remlab.net/

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to