[MPlayer-cvslog] r22609 - trunk/DOCS/tech/new_policy_proposal.txt

michael subversion at mplayerhq.hu
Thu Mar 15 13:45:04 CET 2007


Author: michael
Date: Thu Mar 15 13:45:04 2007
New Revision: 22609

Modified:
   trunk/DOCS/tech/new_policy_proposal.txt

Log:
update spliting rule to what i just added to ffmpeg
comments welcome


Modified: trunk/DOCS/tech/new_policy_proposal.txt
==============================================================================
--- trunk/DOCS/tech/new_policy_proposal.txt	(original)
+++ trunk/DOCS/tech/new_policy_proposal.txt	Thu Mar 15 13:45:04 2007
@@ -105,13 +105,13 @@ Testing code
 
 Splitting changes
    Do not commit unrelated changes together, split them into self-contained
-   pieces.
-   if you have any doubt discuss it on the developer mailing list before
-   committing, also when in doubt more splitting is better then less, changes
-   which are larger then 10kbyte generally should be split into several
-   incremental changes if possible even if you think they are all related
-   keeping changes well split makes reviewing and understanding them on 
-   svn log at the time of commit and later when debugging a bug much easier
+   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.
 
 4. Do not change behavior of the program (renaming options etc) or
    remove functionality from the code without approval in a discussion on



More information about the MPlayer-cvslog mailing list