On 3/14/2019 5:27 PM, Martin Vignali wrote:
>>
>> This seems to have broken all qtrle tests on sparc64.
>>
>> See http://fate.ffmpeg.org/history.cgi?slot=sparc64-linux-gcc-4.9 and
>> others.
>>
>
> Thanks for reporting this.
> Like i can't test on sparc64, doesn't know what line create the bus err
>
> This seems to have broken all qtrle tests on sparc64.
>
> See http://fate.ffmpeg.org/history.cgi?slot=sparc64-linux-gcc-4.9 and
> others.
>
Thanks for reporting this.
Like i can't test on sparc64, doesn't know what line create the bus error.
Patch in attach (untested on sparc64), can probably
On 3/4/2019 9:06 AM, Martin Vignali wrote:
> Pushed. Thanks
>
> Martin
This seems to have broken all qtrle tests on sparc64.
See http://fate.ffmpeg.org/history.cgi?slot=sparc64-linux-gcc-4.9 and
others.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg
Pushed. Thanks
Martin
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> I know we forgot before but this definitely needs a micro version bump.
>
>
New patchs in attach
001, 003, 004 : unchanged
002 : add avcodec micro version bump
Martin
0001-fate-qtrle-change-32b-test-to-output-bgra-instead-of.patch
Description: Binary data
0003-avcodec-qtrle-32bpp-dec-copy-tw
> Am 25.02.2019 um 20:59 schrieb Martin Vignali :
>
> Hello,
>
> thanks for testing and comments.
> New patch in attach
> 001 : unchanged
> 002 : use ARGB for little and big endian, and change the commit msg
I know we forgot before but this definitely needs a micro version bump.
Carl Eugen
_
Hello,
thanks for testing and comments.
New patch in attach
001 : unchanged
002 : use ARGB for little and big endian, and change the commit msg
003 and 004 : mention test platform in commit msg
I can't compile anymore for x86_32, so can't test speed in X86_32.
Martin
0004-avcodec-qtrle-improve
2019-02-23 16:02 GMT+01:00, Martin Vignali :
> Hello,
>
> Patch in attach, improve decoding speed of qtrle (also known as Mov
> Animation)
>
> Can't test on big endian. Test on big endian is welcome.
Patch 1 works on BE and lgtm.
Patch 2 does not work, correct change for case 32 on le and be is:
