[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