On Fri, Nov 1, 2013 at 12:05 AM, Stephen Hutchinson <[email protected]> wrote:
> On Thu, Oct 31, 2013 at 11:43 AM, Diego Biurrun <[email protected]> wrote:
>> On Mon, Oct 28, 2013 at 04:52:09PM -0400, Stephen Hutchinson wrote:
>>> --- a/libavformat/avisynth.c
>>> +++ b/libavformat/avisynth.c
>>> @@ -1,6 +1,6 @@
>>>  /*
>>> - * AviSynth support
>>> - * Copyright (c) 2006 DivX, Inc.
>>> + * Avi/AvxSynth support
>>
>> I'd say "AviSynth/AvxSynth" here.
>
> Agreed.  To be honest I've always thought 'Avi/Avx' looked a bit
> weird.  Will fix.
>
>>
>> Otherwise this looks acceptable to me now, only the author name remains
>> fishy.  Can somebody test the patch before pushing?
>
> There's also a \n that's missing in the av_log that had the useless
> %s, which I've fixed (it was the only fix) in a newer version of the
> patch on Github.
>
> I still didn't get a response through Github about the author name.

I'll post the (hopefully final) version of the patch shortly.  The
fixes mentioned above are included, but still no response on the name.

Technically, I've been running tests on it throughout the entire
vetting process, but some specific charts below based on the latest
version of the patch:

Windows XP SP3, with AviSynth
video+audio = works
video-only = works
audio-only = works
ConvertToRGB24 = works
ConvertToYV12 = works
ConvertToYUY2 = works
ConvertToRGB32 = works
ConvertToYV24 = works
ConvertToYV16 = works
ConvertToYV411 = works
ConvertToY8 = works
ConvertAudioTo8bit = works
ConvertAudioTo16bit = works
ConvertAudioTo24bit = works
ConvertAudioTo32bit = works
ConvertAudioToFloat = works
AviSynth 2.5.8 = 'not supported' error ; expected behavior


OS X 10.7.5, with AvxSynth
video+audio = works
video-only = works
audio-only = works
ConvertToRGB24 = works
ConvertToYV12 = works
ConvertToYUY2 = works
ConvertToRGB32 = works
ConvertToYV24 = script error ; expected behavior
ConvertAudioTo8bit = works
ConvertAudioTo16bit = works
ConvertAudioTo24bit = works
ConvertAudioTo32bit = works
ConvertAudioToFloat = works


Ubuntu 13.10 32-bit, with AvxSynth
video+audio = works
video-only = works
audio-only = works
ConvertToRGB24 = works
ConvertToYV12 = works
ConvertToYUY2 = works
ConvertToRGB32 = works
ConvertToYV24 = script error ; expected behavior
ConvertAudioTo8bit = works
ConvertAudioTo16bit = works
ConvertAudioTo24bit = works
ConvertAudioTo32bit = works
ConvertAudioToFloat = works

Ubuntu 13.10 64-bit, with AvxSynth
video+audio = works
video-only = works
audio-only = works
ConvertToRGB24 = works
ConvertToYV12 = works
ConvertToYUY2 = works
ConvertToRGB32 = works
ConvertToYV24 = script error ; expected behavior
ConvertAudioTo8bit = works
ConvertAudioTo16bit = works
ConvertAudioTo24bit = works
ConvertAudioTo32bit = works
ConvertAudioToFloat = works
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to