Commit 814b3404 authored by michael's avatar michael

patches should be split into incremental changes if possible

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)



git-svn-id: file:///var/local/repositories/ffmpeg/trunk@8412 9553f0bf-9b14-0410-a0b8-cfaf0461ba5b
parent ab7b62b1
......@@ -1492,7 +1492,13 @@ please use av_log() instead.
reported and eventually fixed.
@item
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
Do not change behavior of the program (renaming options etc) without
first discussing it on the ffmpeg-devel mailing list. Do not remove
......
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