[FFmpeg-devel] [PATCH v4 18/18] tests/fate: Update subtitle tests

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Sat Sep 11 13:01:50 EEST 2021


Soft Works:
> Signed-off-by: softworkz <softworkz at hotmail.com>
> ---
>  tests/ref/fate/filter-overlay-dvdsub-2397 | 181 +++++++++++-----------
>  tests/ref/fate/sub-dvb                    | 162 ++++++++++---------
>  tests/ref/fate/sub2video                  | 178 ++++++++-------------
>  tests/ref/fate/sub2video_basic            |  93 ++++++-----
>  tests/ref/fate/sub2video_time_limited     |   6 +-
>  5 files changed, 293 insertions(+), 327 deletions(-)
>  

This is not how fate-updates are supposed to work: You should update the
fate-references in the commit that makes the adjustment necessary. That
way fate will (or rather: should) always be clean* (which is good for
bisecting). Furthermore, you should explain why the changes to the
references are necessary (i.e. what they actually mean). Increased
locality helps with this, too.
(This might of course lead to a bigger absolute diff, because you might
have to change lines multiple times. But that is not what really matters.)

- Andreas

*: In other words, if the mail headers were sent correctly, patchwork
would show lots of red.


More information about the ffmpeg-devel mailing list