[FFmpeg-devel] [PATCH]opencl: compile kernels separately

Reimar Döffinger Reimar.Doeffinger at gmx.de
Wed Oct 30 17:41:40 CET 2013


On Wed, Oct 30, 2013 at 12:12:20PM -0500, Lenny Wang wrote:
> Thanks for your comments.
> 
> I can keep the ABI intact without modifying
> av_opencl_register_kernel_code() but still make this patch work as
> intended.
> 
> I can also keep the old API but since their functionality will break,
> I can do the "return AVERROR(ENOSYS) hack with a message" based on
> Michael's suggestion.

Also, maybe you could go through your new API and check that you can
extend it without breaking old applications in as many places as
possible.
Things like making sure that users will use an allocation function
for structures for example, that means that you an always add additional
members at the end.
At least whenever it isn't much effort to do.
We should really avoid keeping APIs in this "experimental" state,
it's ok for a short while, but after a certain time it starts saying
"we can't be bothered to make sure our stuff actually works properly".


More information about the ffmpeg-devel mailing list