diff options
| author | Michael Niedermayer <michaelni@gmx.at> | 2007-03-15 12:38:45 +0000 |
|---|---|---|
| committer | Michael Niedermayer <michaelni@gmx.at> | 2007-03-15 12:38:45 +0000 |
| commit | 981990db4e05494a6a4f930e42078082ac527dfc (patch) | |
| tree | c6df06151e167ba255b49f1e43c7a0c8b16bad64 /doc | |
| parent | 2c0c00cb143c48b0e7979b04496c87390aa277fc (diff) | |
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)
Originally committed as revision 8412 to svn://svn.ffmpeg.org/ffmpeg/trunk
Diffstat (limited to 'doc')
| -rw-r--r-- | doc/ffmpeg-doc.texi | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/doc/ffmpeg-doc.texi b/doc/ffmpeg-doc.texi index babf35e810..71b94b83bb 100644 --- a/doc/ffmpeg-doc.texi +++ b/doc/ffmpeg-doc.texi @@ -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 |