[Fb-contrib-commit] SF.net SVN: fb-contrib: [966] trunk/fb-contrib/htdocs/index.html

[<dbr...@us...>]

Revision: 966
          http://fb-contrib.svn.sourceforge.net/fb-contrib/?rev=966&view=rev
Author:   dbrosius
Date:     2007-11-09 19:26:09 -0800 (Fri, 09 Nov 2007)

Log Message:
-----------
document cfs

Modified Paths:
--------------
    trunk/fb-contrib/htdocs/index.html

Modified: trunk/fb-contrib/htdocs/index.html
===================================================================
--- trunk/fb-contrib/htdocs/index.html	2007-11-10 00:52:21 UTC (rev 965)
+++ trunk/fb-contrib/htdocs/index.html	2007-11-10 03:26:09 UTC (rev 966)
@@ -48,6 +48,19 @@
 		<a href="bugdescriptions.html">Bug Descriptions</a>
 
 		<hr/>
+		<img id="svn_image" src="flip1.gif" onClick="toggleBlock('svn', 'svn_image');" align="top"/>
+		Detectors added in svn<br/>
+		<div id="svn" style="display:none;">
+			<ul>
+				<li><b>[CFS] Confusing Function Semantics</b><br/>
+				Looks for methods that return a parameter after making what looks like
+				modifications to that parameter. This leads to confusing on the user of this
+				method as it isn't obvious that the 'original' object is modified. If the
+				point of this method is to modify the parameter, it is probably better just
+				to have the method be a void method, to avoid confusion.</li>
+			</ul>
+		</div>
+		<hr/>
 		<img id="v3_4_0_image" src="flip2.gif" onClick="toggleBlock('v3_4_0', 'v3_4_0_image');" align="top"/>
 		Detectors add in v3.4.0<br/>
 		<div id="v3_4_0" style="display:block;">


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.