On 04/09/2011 03:59 PM, Kostya Shishkov wrote:
> This implementation should work fine with multiple side information types
> present.

> +    uint8_t **side_data;
> +    int      *side_data_sizes;
> +    int      *side_data_types;
> +    int       side_data_elems;

Undecided if a

AVSideData {
    void *data
    int data_type
}

AVSideData *side_data;
int side_data_elems;

Is nicer since everything is ordered even if looking up is slower.
Probably your solution is overall better. Patch Ok.

lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero

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

Reply via email to