[Mplayer-cvslog] CVS: main/DOCS/tech patches.txt,1.16,1.17
Diego Biurrun CVS
syncmail at mplayerhq.hu
Tue May 4 16:50:19 CEST 2004
CVS change done by Diego Biurrun CVS
Update of /cvsroot/mplayer/main/DOCS/tech
In directory mail:/var2/tmp/cvs-serv4156
Modified Files:
patches.txt
Log Message:
Hint at diff options useful for avoiding cosmetic changes, patch by Reimar.
Index: patches.txt
===================================================================
RCS file: /cvsroot/mplayer/main/DOCS/tech/patches.txt,v
retrieving revision 1.16
retrieving revision 1.17
diff -u -r1.16 -r1.17
--- patches.txt 22 Apr 2004 12:16:58 -0000 1.16
+++ patches.txt 4 May 2004 14:50:16 -0000 1.17
@@ -25,6 +25,11 @@
4. Read your patch. We'll *refuse* it if it changes indentation of the
code or if it does tab/space conversion or other cosmetical changes!
+ NOTE: If you alread wrote some code and did cosmetic changes, you can use
+ 'diff -uwbBE' to help you remove them. Don't forget to check the patch
+ to make sure diff didn't ignore some important change and remove any
+ remaining cosmetics!
+
5. Comment parts that really need it (tricky side-effects etc).
Commenting trivial code not required. Comments must be English!
More information about the MPlayer-cvslog
mailing list