generic-library/ffmpeg
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

patches should be split into incremental changes if possible

Browse Source 
add patch split reasoning
suggest asking on ffmpeg-dev if theres doubt about spliting
(and like always if any developer disagrees say so and ill revert this and we can disscuss it on ffmpeg-dev)

Originally committed as revision 8412 to svn://svn.ffmpeg.org/ffmpeg/trunk
This commit is contained in:
Michael Niedermayer
2007-03-15 12:38:45 +00:00
parent 2c0c00cb14
commit 981990db4e
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
doc/ffmpeg-doc.texi
View File

@@ -1492,7 +1492,13 @@ please use av_log() instead.
reported and eventually fixed. reported and eventually fixed.
@item @item
Do not commit unrelated changes together, split them into self-contained Do not commit unrelated changes together, split them into self-contained
pieces. pieces. Also dont forget that if part B depends on part A but A doesnt
depend on B, then A can and should be commited first and seperately from B.
Keeping changes well split into self contained parts makes reviewing and
understanding them on svn log at the time of commit and later when
debugging a bug much easier.
Also if you have doubt about spliting or not spliting, dont hesitate to
ask/disscuss it on the developer mailing list.
@item @item
Do not change behavior of the program (renaming options etc) without Do not change behavior of the program (renaming options etc) without
first discussing it on the ffmpeg-devel mailing list. Do not remove first discussing it on the ffmpeg-devel mailing list. Do not remove