On 07/09/13 20:31, Luca Barbato wrote:
> On 07/09/13 20:12, Yukinori Yamazoe wrote:
>> 2013/9/3 Luca Barbato <[email protected]>:
>>>
>>> The line
>>>  +        AVFrame *workframe = outsurf->Data.MemId;
>>> Does not compile for obvious reasons.
>>>
>>>
>>> The way you are doing was deemed impossible due the fact the system memory
>>> would be mapped somehow when using hw decoding, how did you test this 
>>> change?
>>>
>>> Maxim is on vacation till the next week so we can't ask him today =/
>>>
>>
>> I am bad. I did not notice error on gcc.
> 
> No problem, the review process is here to catch those issues =)
> 
>> I compile by MSVC2010+c99wrap and test on Windows 7 x64 SP1.
>> Because MinGW gcc/ld couldn't link mfx library on my environment.
>> I can not link, but I'll check that to be compiled by gcc.
> 
> https://github.com/lu-zero/mfx_dispatch/ builds with gcc just fine =)
> 

btw, how did you test your changes?

My current code works fine with avplay (on github the patches to force
decoding), your update seems to not provide the correct timestamps.

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

Reply via email to