Re: [FFmpeg-devel] [PATCH 2/3] avcodec/nvdec: Use av_buffer_replace() where appropriate

2022-08-07 Thread Timo Rothenpieler
First two patches LGTM ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH 3/3] avcodec/nvdec: Redo handling of failure in nvdec_retrieve_data()

2022-08-07 Thread Timo Rothenpieler
I'm not immediately convinced one is better than the other, but no strong opinion there, both approached work just fine. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link a

[FFmpeg-devel] [PATCHv3] avcodec: WBMP (Wireless Application Protocol Bitmap) image format

2022-08-07 Thread Peter Ross
Reviewed-by: Andreas Rheinhardt Signed-off-by: Peter Ross --- flipped fate ref file order, due to 4fb8741c4670965eaf1f78d6122c6bdfdf1f3358 submitting to patchwork before push Changelog | 1 + doc/general_contents.texi | 2 + libavcodec/Makefile | 2 + libavcodec/allcod

Re: [FFmpeg-devel] ffmpeg.org issue today

2022-08-07 Thread Michael Niedermayer
Hi everyone as some may have noticed there was a 2nd hang happening yesterday 5th/6th august so heres a update about what i/we know/guess the 2nd hang seems to have looked like the first. ive spend some time yesterday looking over logs, there are some showers of slightly odd entries in the apache

Re: [FFmpeg-devel] ffmpeg.org issue today

2022-08-07 Thread Timo Rothenpieler
I had similar issues with Apache on my own server, and never found a concise reason as to what was going on. It seemed like Apache sometimes felt like stopping operation until restarting it. Stopped happening after some Apache updates. Most likely candidate in my case is the DoS attack that was

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/raw: add VUYA pixel format to raw_pix_fmt_tags

2022-08-07 Thread Michael Niedermayer
On Sat, Aug 06, 2022 at 09:34:55PM -0300, James Almer wrote: > Signed-off-by: James Almer > --- > libavcodec/raw.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavcodec/raw.c b/libavcodec/raw.c > index a371bb36c4..1e5b48d1e0 100644 > --- a/libavcodec/raw.c > +++ b/libavcodec/raw.c

Re: [FFmpeg-devel] [PATCH 2/3] avcodec/raw: add VUYA pixel format to raw_pix_fmt_tags

2022-08-07 Thread James Almer
On 8/7/2022 8:39 AM, Michael Niedermayer wrote: On Sat, Aug 06, 2022 at 09:34:55PM -0300, James Almer wrote: Signed-off-by: James Almer --- libavcodec/raw.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/raw.c b/libavcodec/raw.c index a371bb36c4..1e5b48d1e0 100644 --- a/libav

[FFmpeg-devel] [PATCH] avcodec/mpegpicture: Move mb_var, mc_mb_var and mb_mean to MpegEncCtx

2022-08-07 Thread Andreas Rheinhardt
These tables are only used by encoders and only for the current picture; ergo they need not be put into the picture at all, but rather into the encoder's context. They also don't need to be refcounted, because there is only one owner. In contrast to this, the earlier code refcounts them which incu

[FFmpeg-devel] [PATCH 1/3] configure: always enable gnu_windres if available

2022-08-07 Thread Timo Rothenpieler
Use the appropiate Makefile variable to ensure it's only built into shared libraries instead. --- configure | 4 ++-- libavdevice/Makefile | 2 +- libavfilter/Makefile | 2 +- libavformat/Makefile | 2 +- libavutil/Makefile | 2 +- libpostproc/Makefile | 2 +- libswresampl

[FFmpeg-devel] [PATCH 2/3] fftools: add DPI awareness manifest

2022-08-07 Thread Timo Rothenpieler
Some filters, like gdigrab, rely on this to be set to see and report proper dimensions. --- fftools/Makefile | 3 +++ fftools/fftools.manifest | 9 + fftools/fftoolsres.rc| 2 ++ 3 files changed, 14 insertions(+) create mode 100644 fftools/fftools.manifest create mode 100644

[FFmpeg-devel] [PATCH 3/3] compat: add msvc windres wrapper

2022-08-07 Thread Timo Rothenpieler
This is by no means a complete wrapper. It's only designed to fit the usecase ffmpegs build system has. --- compat/windows/mswindres | 32 configure| 2 ++ 2 files changed, 34 insertions(+) create mode 100755 compat/windows/mswindres diff --git a

Re: [FFmpeg-devel] [PATCH 1/3] configure: always enable gnu_windres if available

2022-08-07 Thread Andreas Rheinhardt
Timo Rothenpieler: > Use the appropiate Makefile variable to ensure it's only built into > shared libraries instead. > --- > configure | 4 ++-- > libavdevice/Makefile | 2 +- > libavfilter/Makefile | 2 +- > libavformat/Makefile | 2 +- > libavutil/Makefile | 2 +- > libpos

Re: [FFmpeg-devel] [PATCH 1/3] configure: always enable gnu_windres if available

2022-08-07 Thread Andreas Rheinhardt
Timo Rothenpieler: > Use the appropiate Makefile variable to ensure it's only built into > shared libraries instead. > --- > configure | 4 ++-- > libavdevice/Makefile | 2 +- > libavfilter/Makefile | 2 +- > libavformat/Makefile | 2 +- > libavutil/Makefile | 2 +- > libpos

Re: [FFmpeg-devel] [PATCH] avcodec/jpeg2000: Fast fail if HTJ2K codeblocks are present.

2022-08-07 Thread Caleb Etemesi
caleb is fine. On Sat, May 7, 2022 at 11:35 AM Michael Niedermayer wrote: > Hi > > before applying this > > Author: caleb > > If you want your full name there please resend this with your full name > so its in git with your full name. It cannot be > changed later if its pushed like this > > th

Re: [FFmpeg-devel] [PATCH 1/3] configure: always enable gnu_windres if available

2022-08-07 Thread Timo Rothenpieler
On 07.08.2022 17:51, Andreas Rheinhardt wrote: If the resource files are only for dlls, you can remove SLIBOBJS, because it is now empty. I wasn't sure what those 3 variables are for, compared to the other OBJS variables. In any case I'd be inclined to leave it in, just in case it turns ou

Re: [FFmpeg-devel] [PATCH 1/3] configure: always enable gnu_windres if available

2022-08-07 Thread Timo Rothenpieler
On 07.08.2022 18:04, Andreas Rheinhardt wrote: Timo Rothenpieler: Use the appropiate Makefile variable to ensure it's only built into shared libraries instead. --- configure | 4 ++-- libavdevice/Makefile | 2 +- libavfilter/Makefile | 2 +- libavformat/Makefile | 2 +-

[FFmpeg-devel] [PATCH] avcodec: deprecate AYUV codec id, decoder and encoder

2022-08-07 Thread James Almer
The vuya pixel format was recently added, so this lavc workaround is no longer needed. Signed-off-by: James Almer --- doc/APIchanges | 4 libavcodec/allcodecs.c | 2 ++ libavcodec/codec_desc.c| 2 ++ libavcodec/codec_id.h | 4 libavcodec/v408dec.c | 1

Re: [FFmpeg-devel] [PATCH] avcodec/jpeg2000: Fast fail if HTJ2K codeblocks are present.

2022-08-07 Thread Michael Niedermayer
On Sun, Aug 07, 2022 at 07:46:51PM +0300, Caleb Etemesi wrote: > caleb is fine. ok will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Old school: Use the lowest level language in which you can solve the problem conveniently. New school:

Re: [FFmpeg-devel] [PATCH v2] avformat/mxfdec: SMPTE RDD 48:2018 Amd 1:2022 support

2022-08-07 Thread Michael Niedermayer
On Sat, Aug 06, 2022 at 10:42:02PM +0200, Tomas Härdin wrote: > lör 2022-07-30 klockan 18:42 +0200 skrev Michael Niedermayer: > > Signed-off-by: Michael Niedermayer > > --- > >  libavformat/mxf.c    |  3 +++ > >  libavformat/mxf.h    |  1 + > >  libavformat/mxfdec.c | 48 > > ++

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: Split project server admin list

2022-08-07 Thread Michael Niedermayer
On Sat, Aug 06, 2022 at 04:32:37PM +0200, Michael Niedermayer wrote: > This updates the list closer to reality. > Iam not a professional server admin, iam happy to help maintain the box as i > have > done in the past. But iam not qualified nor volunteering to fix sudden > problems > nor do i do m

[FFmpeg-devel] [PATCH] lavu/pixfmt: add packed RGBA float16 format

2022-08-07 Thread Timo Rothenpieler
This is the default format of the Windows compositor and what DXGI Desktop Duplication will give you for any kind of HDR output. --- libavutil/pixdesc.c | 28 libavutil/pixfmt.h | 5 + libavutil/version.h | 4 ++-- tests/re

Re: [FFmpeg-devel] [PATCH] avcodec/jpeg2000: Fast fail if HTJ2K codeblocks are present.

2022-08-07 Thread Caleb Etemesi
Also apologies for a really late reply, Still learning about mail threads Kind regards. Caleb On Sun, 7 Aug 2022, 22:51 Michael Niedermayer, wrote: > On Sun, Aug 07, 2022 at 07:46:51PM +0300, Caleb Etemesi wrote: > > caleb is fine. > > ok will apply > > thx > > [...] > -- > Michael GnuPG

Re: [FFmpeg-devel] [PATCH] lavu/pixfmt: add packed RGBA float16 format

2022-08-07 Thread Timo Rothenpieler
On 08.08.2022 00:25, Thomas Volkert wrote: Hi Timo, Am 07.08.2022 um 22:14 schrieb Timo Rothenpieler: This is the default format of the Windows compositor and what DXGI Desktop Duplication will give you for any kind of HDR output. --- libavutil/pixdesc.c | 28 +++

Re: [FFmpeg-devel] [PATCH] lavu/pixfmt: add packed RGBA float16 format

2022-08-07 Thread Thomas Volkert
Am 08.08.2022 um 00:26 schrieb Timo Rothenpieler: On 08.08.2022 00:25, Thomas Volkert wrote: Hi Timo, Am 07.08.2022 um 22:14 schrieb Timo Rothenpieler: This is the default format of the Windows compositor and what DXGI Desktop Duplication will give you for any kind of HDR output. ---   libav

Re: [FFmpeg-devel] [PATCH] lavu/pixfmt: add packed RGBA float16 format

2022-08-07 Thread Thomas Volkert
Am 07.08.2022 um 22:14 schrieb Timo Rothenpieler: This is the default format of the Windows compositor and what DXGI Desktop Duplication will give you for any kind of HDR output. --- libavutil/pixdesc.c | 28 libavutil/pixfmt.h | 5 +++

[FFmpeg-devel] [PATCH] avfilter/framesync: add a new option to set how to sync streams based on secondary input timestamps

2022-08-07 Thread James Almer
Include two values for it, a default one that sets/keeps the current behavior, where the frame event generated by the primary input will have a timestamp equal or higher than frames in secondary input, plus a new one where the secondary input frame will be that with the absolute closest timestamp t

[FFmpeg-devel] [PATCH 0/3] avisynth: miscellaneous fixes

2022-08-07 Thread Stephen Hutchinson
Stephen Hutchinson (3): avformat/avisynth: add missing avs_release_video_frame avformat/avisynth: use ch_layout.nb_channels for channel count avformat/avisynth: cosmetics after nb_channels change libavformat/avisynth.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) -- 2.34

[FFmpeg-devel] [PATCH 1/3] avformat/avisynth: add missing avs_release_video_frame

2022-08-07 Thread Stephen Hutchinson
The AviSynth C API requires using avs_release_video_frame whenever avs_get_frame has been used, but the recent addition of frameprop reading to the demuxer was missing this in avisynth_create_stream_video. --- libavformat/avisynth.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/

[FFmpeg-devel] [PATCH 2/3] avformat/avisynth: use ch_layout.nb_channels for channel count

2022-08-07 Thread Stephen Hutchinson
Fixes deprecation warning --- libavformat/avisynth.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/avisynth.c b/libavformat/avisynth.c index 98b4d68a57..5605fbefc4 100644 --- a/libavformat/avisynth.c +++ b/libavformat/avisynth.c @@ -753,7 +753,7 @@ static int avis

[FFmpeg-devel] [PATCH 3/3] avformat/avisynth: cosmetics after nb_channels change

2022-08-07 Thread Stephen Hutchinson
--- libavformat/avisynth.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavformat/avisynth.c b/libavformat/avisynth.c index 5605fbefc4..3d9fa2be50 100644 --- a/libavformat/avisynth.c +++ b/libavformat/avisynth.c @@ -751,10 +751,10 @@ static int avisynth_create_stre

Re: [FFmpeg-devel] [PATCH 2/3] avformat/avisynth: use ch_layout.nb_channels for channel count

2022-08-07 Thread James Almer
On 8/7/2022 10:25 PM, Stephen Hutchinson wrote: Fixes deprecation warning --- libavformat/avisynth.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/avisynth.c b/libavformat/avisynth.c index 98b4d68a57..5605fbefc4 100644 --- a/libavformat/avisynth.c +++ b/libavf