[FFmpeg-cvslog] r11858 - trunk/doc/general.texi

diego subversion
Mon Feb 4 13:16:14 CET 2008


Author: diego
Date: Mon Feb  4 13:16:14 2008
New Revision: 11858

Log:
Update policy entry about compiler entries to reflect ml discussions.


Modified:
   trunk/doc/general.texi

Modified: trunk/doc/general.texi
==============================================================================
--- trunk/doc/general.texi	(original)
+++ trunk/doc/general.texi	Mon Feb  4 13:16:14 2008
@@ -825,8 +825,13 @@ should also be avoided if they don't mak
     it has a fourcc, add it to @file{libavformat/avienc.c}, even if it
     is only a decoder.
 @item
-    Do not change code to hide warnings without ensuring that the underlying
-    logic is correct and thus the warning was inappropriate.
+    Compiler warnings indicate bugs or code with bad style. If a type of
+    warning always points to correct and clean code, that warning should
+    be disabled, not the code changed.
+    Thus the remaining warnings can either be bugs or correct code.
+    If it is a bug, it has to be fixed. If it is not, the code should
+    be changed to not generate a warning unless that causes a slowdown
+    or obfuscates the code.
 @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.




More information about the ffmpeg-cvslog mailing list