Commit b0fee7c5 authored by Diego Biurrun's avatar Diego Biurrun

Group benchmark items in the patch submission checklist together.

Originally committed as revision 8887 to svn://svn.ffmpeg.org/ffmpeg/trunk
parent 58683d27
...@@ -1681,6 +1681,8 @@ do not attach several unrelated patches to the same mail. ...@@ -1681,6 +1681,8 @@ do not attach several unrelated patches to the same mail.
achieved with a smaller patch and/or simpler final code? achieved with a smaller patch and/or simpler final code?
@item @item
If the change is to speed critical code, did you benchmark it? If the change is to speed critical code, did you benchmark it?
@item
If you did any benchmarks, did you provide them in the mail?
@item @item
Have you checked that the patch does not introduce buffer overflows or Have you checked that the patch does not introduce buffer overflows or
other security issues? other security issues?
...@@ -1709,8 +1711,6 @@ do not attach several unrelated patches to the same mail. ...@@ -1709,8 +1711,6 @@ do not attach several unrelated patches to the same mail.
@item @item
Did you provide an example so we can verify the new feature added by the Did you provide an example so we can verify the new feature added by the
patch easily? patch easily?
@item
If you did any benchmarks, did you provide them in the mail?
@item @item
If you added a new file, did you insert a license header? It should be If you added a new file, did you insert a license header? It should be
taken from FFmpeg, not randomly copied and pasted from somewhere else. taken from FFmpeg, not randomly copied and pasted from somewhere else.
......
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