[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