Re: [FFmpeg-devel] [PATCH] QSV : making three qsv routines
>public
>
>On Thu, Dec 10, 2015 at 10:16 AM, Sven Dueking wrote:
> >This patch expose 3 QSV functions as public.
> >This is needed because the VPP needs access to these functions too.
> >
> >Please review.
&g
> -Ursprüngliche Nachricht-
> Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag
> von Hendrik Leppkes
> Gesendet: Donnerstag, 10. Dezember 2015 10:19
> An: FFmpeg development discussions and patches
> Betreff: Re: [FFmpeg-devel] [PATCH] QSV : making
Hello Hendrik,
Thursday, December 10, 2015, 2:00:51 PM, you wrote:
>> What is wrong then?
HL> The public qsv.h uses config.h, thats not allowed.
ah, got it, thank.
--
Best regards,
Ivanmailto:ivan.us...@nablet.com
___
ffm
On Thu, Dec 10, 2015 at 11:20 AM, Ivan Uskov wrote:
> Hello Hendrik,
>
> Thursday, December 10, 2015, 1:10:59 PM, you wrote:
>
> This patch expose 3 QSV functions as public.
> This is needed because the VPP needs access to these functions too.
>
> Please review.
>
>>>
>>> HL> p
Hello Hendrik,
Thursday, December 10, 2015, 1:10:59 PM, you wrote:
This patch expose 3 QSV functions as public.
This is needed because the VPP needs access to these functions too.
Please review.
>>
>> HL> public API is not allowed to use config.h, it needs to be config an
On Thu, Dec 10, 2015 at 11:01 AM, Ivan Uskov wrote:
> Hello Hendrik,
>
> Thursday, December 10, 2015, 12:19:23 PM, you wrote:
>
> HL> On Thu, Dec 10, 2015 at 10:16 AM, Sven Dueking wrote:
>>> This patch expose 3 QSV functions as public.
>>> This is needed because the VPP needs access to these fun
Hello Hendrik,
Thursday, December 10, 2015, 12:19:23 PM, you wrote:
HL> On Thu, Dec 10, 2015 at 10:16 AM, Sven Dueking wrote:
>> This patch expose 3 QSV functions as public.
>> This is needed because the VPP needs access to these functions too.
>>
>> Please review.
>>
HL> public API is not allo
On Thu, Dec 10, 2015 at 10:16 AM, Sven Dueking wrote:
> This patch expose 3 QSV functions as public.
> This is needed because the VPP needs access to these functions too.
>
> Please review.
>
public API is not allowed to use config.h, it needs to be config and
OS agnostic.
- Hendrik
This patch expose 3 QSV functions as public.
This is needed because the VPP needs access to these functions too.
Please review.
Thanks,
Sven
0001-make-some-internal-QSV-routines-public.patch
Description: Binary data
___
ffmpeg-devel mailing list
ffmp
