Add 'make checkheaders' to developer guidelines and patch checklist.
authorDiego Biurrun <diego@biurrun.de>
Wed, 29 Oct 2008 06:48:59 +0000 (06:48 +0000)
committerDiego Biurrun <diego@biurrun.de>
Wed, 29 Oct 2008 06:48:59 +0000 (06:48 +0000)
Originally committed as revision 15741 to svn://svn.ffmpeg.org/ffmpeg/trunk

doc/general.texi

index 78372f6..9a823d4 100644 (file)
@@ -951,6 +951,9 @@ should also be avoided if they don't make the code easier to understand.
 @item
     If you add a new file, give it a proper license header. Do not copy and
     paste it from a random place, use an existing file as template.
+@item
+    If you add a new header or change an existing one, make sure it passes
+    @code{make checkheaders}.
 @end enumerate
 
 We think our rules are not too hard. If you have comments, contact us.
@@ -1026,6 +1029,8 @@ do not attach several unrelated patches to the same mail.
 @item
     Do the regression tests pass with the patch applied?
 @item
+    Does @code{make checkheaders} pass with the patch applied?
+@item
     Is the patch a unified diff?
 @item
     Is the patch against latest FFmpeg SVN?