On 10/07/2012 02:01 AM, Ronald S. Bultje wrote:
> It often leads to smaller code. Instead of AVPixFmtDesc *d =
> getter(pix_fmt); if (!d) return -1; printf("%d\n", d->something);, you
> can just do printf("%d\n", array[pix_fmt].something);. Do we really
> need to babysit our users?

It can be _quite_ bad (e.g. pix_fmt > PIX_FMT_NB)...

>> It is the only case in all Libav where we export an array like this.
> 
> We never supported MSVC. So we shouldn't?
> 
>> It fixes the size of AVPixFmtDescriptor for no good reason.
> 
> Is there a need to extend it?

Yes =)

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

Reply via email to