[FFmpeg-devel] Collection Ping (softworkz)

[Marton Balint]

On Wed, 10 Nov 2021, Soft Works wrote:

> Hi,
>
> I thought it might be more friendly to ping for my pending patches
> in a single message.
>
>
> Here’s the list of my pending (mostly unresponded) patch sets:
>
>
> [FFmpeg-devel,v2] DXVA2: Add ARGB format
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=4491

Applied.

>
> [FFmpeg-devel,v3] doc/developer: Add description about safely sending patches via E-Mail clients
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=4494

Applied with some minor changes.

>
> [FFmpeg-devel,v6,01/11] libavformat/asf: Fix handling of byte array length values
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=5030

You should ping this series alone, it is not trivial.

>
> [FFmpeg-devel,v2,1/2] fftools/ffmpeg: Remove redundant loop
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=5059

Applied.

>
> [FFmpeg-devel,v3,1/1] avutils/hwcontext: When deriving a hwdevice, search for existing device in bo…
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=5063

Can you ping this one more time? I am not familiar with hwcontext enough 
to decide, and it adds some public stuff as well.

>
> [FFmpeg-devel,v2,1/1] avutil/frame: Document the possibility of negative line sizes
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=5080

This patch is corrupt, please resend. Otherwise seems fine, although I 
might remove the comment about 0 linesize, I have never seen it, maybe it 
should be kept undocumented... :)

>
> [FFmpeg-devel,1/5] avfilter/avfilter: Add avfilter_alloc() and avfilter_query_formats() for initial…
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=5082

Can't decide and there were some people against it.

>
> [FFmpeg-devel,1/1] libavutil/opt: fix mis-alignment of option and constant values for filter help
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=5084

Applied.

>
> [FFmpeg-devel,1/1] fftools/ffmpeg: Output log message when interactive q command is received
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=5086

Applied.

>
> [FFmpeg-devel,v3,1/1] fftools/cmdutils: Print bit depths when executing 'ffmpeg -pix_fmts'
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=5089

Applied.

>
> [FFmpeg-devel,v2,1/1] avformat/dump: Print codec level and stream start offsets for input streams
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=5088

Level unfortunately is not always readable based on the number, and 
preferably it should be written correctly and after the profile, e.g. base 
profile level 3.0.

>
> [FFmpeg-devel,1/1] swscale/swscale: check SWS_PRINT_INFO flag for printing alignment warnings
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=5101

Applied.

>
> [FFmpeg-devel,1/1] fftools/ffprobe: print size of attachment streams
> https://patchwork.ffmpeg.org/bundle/softworkz/Pending_Nov_2021/?series=5113

I suggest you dump extradata_size instead always. Seems more generic.

>
> [FFmpeg-devel,v14,01/14] Subtitle Filtering
> https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=5125

Can't decide.

Thanks,
Marton