[FFmpeg-devel] [PATCH] avformat/imf: s/++i/i++/g

Pierre-Anthony Lemieux pal at sandflow.com
Fri Feb 4 19:29:30 EET 2022


On Thu, Feb 3, 2022 at 11:58 PM Paul B Mahol <onemda at gmail.com> wrote:
>
> Never apply this.

I am not a maintainer and so cannot apply the patch.

There is another minor patch that has also been outstanding for some time now:

https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=5700

> Very sorry state of project.

Can you be more specific?

Are you referring to code style issues? If so, there is a draft patch at [1].

[1] https://github.com/sandflow/ffmpeg-imf/pull/85

I have been reluctant to propose it until:

(a) I am more confident in the expected coding style; and
(b) issues logged in trac are fixed.

Regarding (a), I asked the following question, which I never received
feedback on:

http://ffmpeg.org/pipermail/ffmpeg-devel/2022-January/290990.html

This prompted me to work on expanding the code style documentation,
which I mentioned on IRC and plan to proposed shortly:

https://github.com/sandflow/ffmpeg-imf/pull/95

Regarding (b), the following patch is up for review:

https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=5873

I can prioritize (a) if folks feel this is the right approach.

There is also an outstanding proposal [2] to refactor uuid
functionality. Have you had a chance to review?

[2] http://ffmpeg.org/pipermail/ffmpeg-devel/2022-January/291917.html

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list