[SCM] libav/experimental: patch submission checklist

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Jun 30 15:58:57 UTC 2013


The following commit has been merged in the experimental branch:
commit dd597cd7b0f81b36be8c7f65af125f5247755198
Author: Michael Niedermayer <michaelni at gmx.at>
Date:   Tue Apr 17 13:07:31 2007 +0000

    patch submission checklist
    
    Originally committed as revision 8745 to svn://svn.ffmpeg.org/ffmpeg/trunk

diff --git a/doc/ffmpeg-doc.texi b/doc/ffmpeg-doc.texi
index 8825874..5246049 100644
--- a/doc/ffmpeg-doc.texi
+++ b/doc/ffmpeg-doc.texi
@@ -1623,6 +1623,53 @@ and has no lrint()')
 Also please if you send several patches, send each patch as seperate mail,
 dont attach several unrelated patches to the same mail.
 
+ at section patch submission checklist
+
+ at enumerate
+ at item
+    Do the regression tests pass with the patch applied?
+ at item
+    Is the patch a unified diff?
+ at item
+    Is the patch against latest ffmpeg SVN?
+ at item
+    Are you subscribed to ffmpeg-dev?
+    (the list is subscribers only due to spam)
+ at item
+    Have you checked that the changes are minimal, so that the same cannot be
+    achived with a smaller patch and/or simpler final code?
+ at item
+    If the change is to speed critical code did you benchmark it?
+ at item
+    Have you checked that the patch does not intruduce buffer overflows or
+    other security issues?
+ at item
+    Is the patch made from the root of the source, so it can be applied with -p0?
+ at item
+    Does the patch not mix functional and cosmetic changes?
+ at item
+    Is the patch attached to the email you send?
+ at item
+    Is the mime type of the patch correct? (not application/octet-stream)
+ at item
+    If the patch fixes a bug did you provide a verbose analysis of the bug?
+ at item
+    If the patch fixes a bug did you provide enough information including
+    a sample, so the bug can be reproduced and the fix can be verified?
+ at item
+    Did you provide a verbose summary about what the patch does change?
+ at item
+    Did you provide a verbose explanation why it changes things like it does?
+ at item
+    Did you provide a verbose summary of the user vissible advantages and
+    disadvantages if the patch is applied?
+ at item
+    Did you provide an example so we can verify the new feature added by the
+    patch easily?
+ at item
+    If you did any benchmarks, did you provide them in the mail?
+ at end enumerate
+
 @section Patch review process
 
 All patches posted to ffmpeg-devel will be reviewed, unless they contain a

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list