diff options
| author | Josh de Kock <josh@itanimul.li> | 2016-10-05 01:10:00 +0300 |
|---|---|---|
| committer | James Almer <jamrial@gmail.com> | 2016-10-05 04:55:44 +0300 |
| commit | 5173ffb27f6e3ad25c1b947cda2fb9e196a7918a (patch) | |
| tree | add08d620f5d8c55770a4d2061a9f7f9edc131bd /doc/developer.texi | |
| parent | b33369b6128a7c99fa7fa4686712ce8b8b5fef05 (diff) | |
doc/developer: remove duplicate policies and fix error
Fixes regression as of ee72b6d1
Reviewed-by: Michael Niedermayer <michael@niedermayer.cc>
Signed-off-by: Josh de Kock <josh@itanimul.li>
Signed-off-by: James Almer <jamrial@gmail.com>
Diffstat (limited to 'doc/developer.texi')
| -rw-r--r-- | doc/developer.texi | 18 |
1 files changed, 2 insertions, 16 deletions
diff --git a/doc/developer.texi b/doc/developer.texi index 3fcfe86c33..cf809b9255 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -293,12 +293,7 @@ later on. Also if you have doubts about splitting or not splitting, do not hesitate to ask/discuss it on the developer mailing list. -@item API/ABI changes should be discussed before they are made. -Do not change behavior of the programs (renaming options etc) or public -API or ABI without first discussing it on the ffmpeg-devel mailing list. -Do not remove widely used functionality or features (redundant code can be removed). - -@item Ask before you change the build system (configure, etc). +@subheading Ask before you change the build system (configure, etc). Do not commit changes to the build system (Makefiles, configure script) which change behavior, defaults etc, without asking first. The same applies to compiler warning fixes, trivial looking fixes and to code @@ -307,7 +302,7 @@ the way we do. Send your changes as patches to the ffmpeg-devel mailing list, and if the code maintainers say OK, you may commit. This does not apply to files you wrote and/or maintain. -@item Cosmetic changes should be kept in separate patches. +@subheading Cosmetic changes should be kept in separate patches. We refuse source indentation and other cosmetic changes if they are mixed with functional changes, such commits will be rejected and removed. Every developer has his own indentation style, you should not change it. Of course @@ -356,15 +351,6 @@ Do not change behavior of the programs (renaming options etc) or public API or ABI without first discussing it on the ffmpeg-devel mailing list. Do not remove widely used functionality or features (redundant code can be removed). -@subheading Ask before you change the build system (configure, etc). -Do not commit changes to the build system (Makefiles, configure script) -which change behavior, defaults etc, without asking first. The same -applies to compiler warning fixes, trivial looking fixes and to code -maintained by other developers. We usually have a reason for doing things -the way we do. Send your changes as patches to the ffmpeg-devel mailing -list, and if the code maintainers say OK, you may commit. This does not -apply to files you wrote and/or maintain. - @subheading Remember to check if you need to bump versions for libav*. Depending on the change, you may need to change the version integer. Incrementing the first component means no backward compatibility to |