Author: aniedbala
Date: Tue Nov 25 12:13:14 2008
New Revision: 2847

Modified:
trunk/src/ca/sqlpower/architect/layout/FruchtermanReingoldForceLayout.java

Log:
Fixed an improper use of Math.random() being cast immediately to an int (the result will always be 0). This should make the layout algorithm more correct, and fixes a findbugs warning.

Modified: trunk/src/ca/sqlpower/architect/layout/FruchtermanReingoldForceLayout.java
==============================================================================
--- trunk/src/ca/sqlpower/architect/layout/FruchtermanReingoldForceLayout.java (original) +++ trunk/src/ca/sqlpower/architect/layout/FruchtermanReingoldForceLayout.java Tue Nov 25 12:13:14 2008
@@ -185,7 +185,7 @@
                                        break;
                                }
                                if(uDisp.equals(vDisp)) {
-                                       
vDisp.translate((int)Math.random()+1,(int)Math.random()+1 );
+ vDisp.translate((int) (Math.random() + 0.5), (int) (Math.random() + 0.5));
                                }
                                
                                vDisp.translate(

Reply via email to