[FFmpeg-devel] [PATCH] Generic part of frame multithreading

[Uoti Urpala]

On Fri, 2008-08-22 at 09:34 +0100, M?ns Rullg?rd wrote:
> Uoti Urpala <uoti.urpala at pp1.inet.fi> writes:
> > If you have any other data about the frame besides pts, so you can not
> > fit it all directly in the field. It shouldn't be too hard to think of
> 
> So cast a pointer or store an index into a table.

First is ugly and second can require extra complexity, and neither has
any real advantage over lavc supporting void * directly. If lavc has an
union of int64_t and double then adding void * as an alternative has
very little extra cost. If you agree that the user may need to store
such data do you have any reason why storing it in a field of the wrong
type with pointer casts would be preferable?

Ensuring that there is a robust way for the user to free any associated
resources when they're no longer needed may require more than completely
negligible work from the lavc side, but that issue exists independently
of whether void * is a possible type.