Revision: 55707
          http://sourceforge.net/p/brlcad/code/55707
Author:   brlcad
Date:     2013-06-10 22:49:18 +0000 (Mon, 10 Jun 2013)
Log Message:
-----------
introducing new magic numbers (literal values) without explanation is taboo, go 
with something (an order of magnitude) that should be defensible.  value is 
still significantly smaller than the previous quarter-mm value...

Modified Paths:
--------------
    brlcad/trunk/src/libged/gqa.c

Modified: brlcad/trunk/src/libged/gqa.c
===================================================================
--- brlcad/trunk/src/libged/gqa.c       2013-06-10 22:42:11 UTC (rev 55706)
+++ brlcad/trunk/src/libged/gqa.c       2013-06-10 22:49:18 UTC (rev 55707)
@@ -2358,9 +2358,19 @@
     azimuth_deg = 0.0;
     elevation_deg = 0.0;
     densityFileName = (char *)0;
+
+    /* FIXME: this is completely arbitrary, should probably be based
+     * on the model size.
+     */
     gridSpacing = 50.0;
-    /* set default grid spacing limit based on distance tolerance */
-    gridSpacingLimit = 4.0 * gedp->ged_wdbp->wdb_tol.dist;
+
+    /* default grid spacing limit is based on the current distance
+     * tolerance, one order of magnitude greater.
+     *
+     * FIXME: should probably be based on the model size.
+     */
+    gridSpacingLimit = 10.0 * gedp->ged_wdbp->wdb_tol.dist;
+
     makeOverlapAssemblies = 0;
     require_num_hits = 1;
     max_cpus = ncpu = bu_avail_cpus();

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


------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits

Reply via email to