[FFmpeg-devel] [PATCH 0/3] Fix /// comments that should be ///<

Stefano Sabatini stefasab at gmail.com
Mon Mar 4 19:00:05 EET 2024


On date Monday 2024-03-04 17:49:27 +0100, Andreas Rheinhardt wrote:
> Stefano Sabatini:
> > On date Wednesday 2024-02-28 17:09:38 +0000, Andrew Sayers wrote:
> >> This is the first in a planned series of patch series aimed at
> >> fixing documentation issues, especially doxygen rendering.
> >> I hope to post more as time allows in the coming days.
> >>
> >> The first patch reflects the output of a `sed` command,
> >> further patches fix edge cases not covered by `sed`.
> >> Splitting commits this way allows merge conflicts
> >> to be resolved by throwing the old commit away,
> >> then running the command in the commit message.
> >>
> >> Andrew Sayers (3):
> >>       all: Fix /// comments that should be ///<
> >>       avcodec/ivi: Make comments more Doxygen-friendly
> >>       avcodec/avcodec: Downgrade multi-member comment to non-Doxygen comment
> >>
> >>  libavcodec/avcodec.h     | 10 +++++-----
> >>  libavcodec/bsf/noise.c   | 24 ++++++++++++------------
> >>  libavcodec/ivi.h         | 15 +++++++--------
> >>  libavfilter/vf_deshake.c |  2 +-
> >>  libavformat/rmdec.c      |  4 ++--
> >>  5 files changed, 27 insertions(+), 28 deletions(-)
> > 
> > Will apply, thanks.
> 

> Please don't. Anton intends to remove the pts_correction fields with the
> bump and so this is unnecessary and will just cause a rebase conflict.

Too late, sorry, hopefully the rebase should not too hard to fix (also
there are more changes unrelated to pts_correction we want to keep).

Let me know if we should hold header changes to prevent rebase conflicts
at this stage.


More information about the ffmpeg-devel mailing list