[FFmpeg-devel] [PATCH] qsv.c: making qsv session routines common for libavcodec and libavfilter modules.

Ivan Uskov ivan.uskov at nablet.com
Fri Nov 13 11:22:49 CET 2015


Hello wm4,

Friday, November 13, 2015, 11:28:46 AM, you wrote:
>> HL> avutil is not a place we dump stuff that should be shared between
>> HL> avcodec and avfilter, or similar, avutil should generally be public
>> HL> useful API.  
>> I believe 3 of 4 functions here can be published as useful API.
>> If   you   do  not  like  this  way,  could  you  please  advise some another
>> constructive  way to avoid a duplication of more then 300 code lines? Or copy
>> paste between codecs and filters is really best solution for this case?
>> Thank!
>> 

w> Didn't we just write how we don't want this to be public API?
I  saw only mentions based on functions and headers names which were selected
arbitrary  long  time  ago. So I was forced to highlight that this api can be
useful  and  public.  Of  course  you  can  have another opposite opinion, no
problem.
But there are no constructive suggestion from your side how to solve the issue,
but just rejection, correct?


-- 
Best regards,
 Ivan                            mailto:ivan.uskov at nablet.com



More information about the ffmpeg-devel mailing list