[MPlayer-DOCS] [PATCH] comments in patches.txt

Alexander Strasser eclipse7 at gmx.net
Sun Jun 27 14:14:45 CEST 2004


Extended the comment section of patches.txt a little.
The origin of these changes are the security problems
discovered recently. This rule should avoid most of
them in the future.

Alex (beastd)
-------------- next part --------------
Index: patches.txt
===================================================================
RCS file: /cvsroot/mplayer/main/DOCS/tech/patches.txt,v
retrieving revision 1.18
diff -u -r1.18 patches.txt
--- patches.txt	17 Jun 2004 08:43:18 -0000	1.18
+++ patches.txt	27 Jun 2004 11:56:00 -0000
@@ -31,7 +31,10 @@
     any remaining cosmetics!
 
  5. Comment parts that really need it (tricky side-effects etc).
-    Commenting trivial code not required. Comments must be English!
+    Always document string operations! Comment on what you are doing
+    and why it is safe. This makes it easy to review and change your
+    code if needed. Commenting trivial code not required.
+    Comments must be English!
 
  6. If you implement new features, add or change command line switches or
     modify the behavior of existing features, please do not forget to also


More information about the MPlayer-DOCS mailing list