Commit 64d32a57 authored by diego's avatar diego

Update policy entry about compiler entries to reflect ml discussions.


git-svn-id: file:///var/local/repositories/ffmpeg/trunk@11858 9553f0bf-9b14-0410-a0b8-cfaf0461ba5b
parent 611aa65d
...@@ -825,8 +825,13 @@ should also be avoided if they don't make the code easier to understand. ...@@ -825,8 +825,13 @@ should also be avoided if they don't make the code easier to understand.
it has a fourcc, add it to @file{libavformat/avienc.c}, even if it it has a fourcc, add it to @file{libavformat/avienc.c}, even if it
is only a decoder. is only a decoder.
@item @item
Do not change code to hide warnings without ensuring that the underlying Compiler warnings indicate bugs or code with bad style. If a type of
logic is correct and thus the warning was inappropriate. 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 @item
If you add a new file, give it a proper license header. Do not copy and 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. paste it from a random place, use an existing file as template.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment