diff mbox series

[FFmpeg-devel,1/3] lavfi/formats: cosmetic: move negotiation stuff at the end

Message ID 20210819153057.149696-1-george@nsup.org
State Accepted
Commit 703c3c4151bf47a03ddf168f67a8b1a46834c791
Headers show
Series [FFmpeg-devel,1/3] lavfi/formats: cosmetic: move negotiation stuff at the end | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Nicolas George Aug. 19, 2021, 3:30 p.m. UTC
It was between AVFilterFormats and AVFilterChannelLayouts.

Signed-off-by: Nicolas George <george@nsup.org>
---
 libavfilter/formats.h | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)


The last commit is the one I sent earlier, just rebased on top of this
cosmetic change because it makes no sense to add almost a hundred lines
of documentation just to move it in the next commit.

Since these three patches are cosmetic and internal documentation, I
will push soon, probably as soon as I have something new to post on top
of them.

But please comment nonetheless.

Anton and Michael: you are the authors of the redundant similarity
logics in swap and pick_formats() (see the FIXME comments in the third
patche), I would appreciate if you had a look at it.

Michael, in particular, if you have test cases covering this logic that
are not part of FATE, please share them, or even better make them part
of FATE if possible.

Comments

Nicolas George Aug. 20, 2021, 8:48 a.m. UTC | #1
Nicolas George (12021-08-19):
> Since these three patches are cosmetic and internal documentation, I
> will push soon, probably as soon as I have something new to post on top
> of them.

Series pushed.

> Anton and Michael: you are the authors of the redundant similarity
> logics in swap and pick_formats() (see the FIXME comments in the third
> patche), I would appreciate if you had a look at it.
> 
> Michael, in particular, if you have test cases covering this logic that
> are not part of FATE, please share them, or even better make them part
> of FATE if possible.

Ping? I am about to start working on making this a score-based system.

Regards,
diff mbox series

Patch

diff --git a/libavfilter/formats.h b/libavfilter/formats.h
index ed513c265a..a9cfa14e95 100644
--- a/libavfilter/formats.h
+++ b/libavfilter/formats.h
@@ -69,21 +69,6 @@  struct AVFilterFormats {
     struct AVFilterFormats ***refs; ///< references to this list
 };
 
-typedef struct AVFilterFormatMerger {
-    unsigned offset;
-    int (*merge)(void *a, void *b);
-    int (*can_merge)(const void *a, const void *b);
-} AVFilterFormatsMerger;
-
-typedef struct AVFilterNegotiation {
-    unsigned nb;
-    const AVFilterFormatsMerger *mergers;
-    const char *conversion_filter;
-    unsigned conversion_opts_offset;
-} AVFilterNegotiation;
-
-const AVFilterNegotiation *ff_filter_get_negotiation(AVFilterLink *link);
-
 /**
  * A list of supported channel layouts.
  *
@@ -330,4 +315,19 @@  int ff_formats_check_sample_rates(void *log, const AVFilterFormats *fmts);
  */
 int ff_formats_check_channel_layouts(void *log, const AVFilterChannelLayouts *fmts);
 
+typedef struct AVFilterFormatMerger {
+    unsigned offset;
+    int (*merge)(void *a, void *b);
+    int (*can_merge)(const void *a, const void *b);
+} AVFilterFormatsMerger;
+
+typedef struct AVFilterNegotiation {
+    unsigned nb;
+    const AVFilterFormatsMerger *mergers;
+    const char *conversion_filter;
+    unsigned conversion_opts_offset;
+} AVFilterNegotiation;
+
+const AVFilterNegotiation *ff_filter_get_negotiation(AVFilterLink *link);
+
 #endif /* AVFILTER_FORMATS_H */