[FFmpeg-devel] [PATCH] doc/developer: patches should not cross libs or mix MAINTAINER changes in

Michael Niedermayer michael at niedermayer.cc
Tue Jan 4 22:06:26 EET 2022


maintainer part inspired by IRC comments by lynne

Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
---
 doc/developer.texi | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/doc/developer.texi b/doc/developer.texi
index addee0d8263..eae3ade48a7 100644
--- a/doc/developer.texi
+++ b/doc/developer.texi
@@ -683,6 +683,10 @@ are notoriously left unchecked, which is a serious problem.
 @item
 Test your code with valgrind and or Address Sanitizer to ensure it's free
 of leaks, out of array accesses, etc.
+
+ at item
+Changes to 2 libs which change both versions should be in 2 seperate patches.
+Non cosmetic changes to @file{MAINTAINERS} should not be in a patch with other changes.
 @end enumerate
 
 @chapter Patch review process
-- 
2.17.1



More information about the ffmpeg-devel mailing list