Update of /cvsroot/fb-contrib/fb-contrib/htdocs
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv20310/htdocs
Modified Files:
index.html
Log Message:
intial checkin - NIR detector
Index: index.html
===================================================================
RCS file: /cvsroot/fb-contrib/fb-contrib/htdocs/index.html,v
retrieving revision 1.58
retrieving revision 1.59
diff -u -d -r1.58 -r1.59
--- index.html 2 Feb 2006 06:18:02 -0000 1.58
+++ index.html 4 Feb 2006 02:23:50 -0000 1.59
@@ -50,12 +50,6 @@
Note: fb-contrib v2.2.0 requires FindBugs™ v0.95 or later<br/>
<div id="cvs" style="display:block;">
<ul>
- <li><b>Sluggish Gui</b><br/>
- Looks for methods that implement awt or swing listeners and perform time
- consuming operations. Doing these operations in the gui thread will cause the
- interface to appear sluggish and non-responsive to the user. It is better to
- use a separate thread to do the time consuming work so that the user
- has a better experience.</li>
<li><b>Bloated Synchronized Block</b><br/>
Looks for methods that implement synchronized blocks that appear to contain some code at
the beginning that does not need to be synchronized. Moving these lines out of the synchronized
@@ -75,6 +69,16 @@
Looks for methods that use an array of length one to pass a variable to achieve call
by pointer ala C++. It is better to define a proper return class type that holds all
the relevant information retrieved from the called method.</li>
+ <li><b>Sluggish Gui</b><br/>
+ Looks for methods that implement awt or swing listeners and perform time
+ consuming operations. Doing these operations in the gui thread will cause the
+ interface to appear sluggish and non-responsive to the user. It is better to
+ use a separate thread to do the time consuming work so that the user
+ has a better experience.</li>
+ <li><b>Needless Instance Retrieval</b><br/>
+ Looks for methods that call a method to retrieve a reference to an object
+ only to then load a static field of that object's class. It is simpler and more
+ preformant to just directly load the field from the class itself.</li>
</ul>
</div>
|
