From: Paul B Mahol <onemda@gmail.com>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter/framepool: fix alignment requirements for audio and video filters
Date: Sun, 10 Apr 2022 23:59:10 +0200
Message-ID: <CAPYw7P4TafD2J00a8pvCRsnnfxFVHc88mn+4DL=ghHhwFrC-Bw@mail.gmail.com> (raw)
In-Reply-To: <CAPYw7P7osHQ-aeyfYyA14_kdF+v3B9Ez9iOOxBxegeuoe2WJBQ@mail.gmail.com>
On Sun, Apr 10, 2022 at 11:53 PM Paul B Mahol <onemda@gmail.com> wrote:
>
>
> On Sun, Apr 10, 2022 at 11:41 PM Michael Niedermayer <
> michael@niedermayer.cc> wrote:
>
>> On Tue, Feb 22, 2022 at 08:22:55AM +0000, Paul B Mahol wrote:
>> > ffmpeg | branch: master | Paul B Mahol <onemda@gmail.com> | Sun Feb 20
>> 19:06:52 2022 +0100| [17a59a634c39b00a680c6ebbaea58db95594d13d] |
>> committer: Paul B Mahol
>> >
>> > avfilter/framepool: fix alignment requirements for audio and video
>> filters
>> >
>> > >
>> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=17a59a634c39b00a680c6ebbaea58db95594d13d
>> > ---
>> >
>> > libavfilter/audio.c | 11 +++++------
>> > libavfilter/framepool.c | 18 ++++++++----------
>> > libavfilter/video.c | 11 +++++------
>> > 3 files changed, 18 insertions(+), 22 deletions(-)
>>
>> This change broke:
>> ./ffmpeg -i matrixbench_mpeg2.mpg -an -vframes 5 -vf
>> "frei0r=perspective:0.2/0.2|0.8/0.2" -qscale 1 perspective.avi
>>
>> The resulting file looks like it has a messed up linesize
>>
>
> Are you sure this is not encoder bug?
>
> I'm not willing to invest debugging horrible code of mpeg encoders.
>
Actually, frei0r should be removed completely.
It does not handle linesize at all, and never have no support for it.
>
>
>>
>> thx
>>
>> [...]
>> --
>> Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>>
>> Does the universe only have a finite lifespan? No, its going to go on
>> forever, its just that you wont like living in it. -- Hiranya Peiri
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel@ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
>>
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
prev parent reply other threads:[~2022-04-10 21:57 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20220222082256.7991341055D@natalya.videolan.org>
2022-04-10 21:41 ` Michael Niedermayer
2022-04-10 21:53 ` Paul B Mahol
2022-04-10 21:57 ` Michael Niedermayer
2022-04-10 21:59 ` Paul B Mahol [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAPYw7P4TafD2J00a8pvCRsnnfxFVHc88mn+4DL=ghHhwFrC-Bw@mail.gmail.com' \
--to=onemda@gmail.com \
--cc=ffmpeg-devel@ffmpeg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git