Revision: 57079
http://sourceforge.net/p/brlcad/code/57079
Author: brlcad
Date: 2013-08-22 20:03:04 +0000 (Thu, 22 Aug 2013)
Log Message:
-----------
integer division for percentages is generally a bad idea. do it as doubles.
Modified Paths:
--------------
brlcad/trunk/src/conv/nmg/g-nmg.c
Modified: brlcad/trunk/src/conv/nmg/g-nmg.c
===================================================================
--- brlcad/trunk/src/conv/nmg/g-nmg.c 2013-08-22 19:47:21 UTC (rev 57078)
+++ brlcad/trunk/src/conv/nmg/g-nmg.c 2013-08-22 20:03:04 UTC (rev 57079)
@@ -146,9 +146,9 @@
if (RT_G_DEBUG&DEBUG_TREEWALK || verbose) {
sofar = db_path_to_string(pathp);
- bu_log("\ndo_region_end(%d %d%%) %s\n",
+ bu_log("\ndo_region_end(%d %.2lf%%) %s\n",
regions_tried,
- regions_tried>0 ? (regions_converted * 100) / regions_tried : 0,
+ regions_tried>0 ? ((double)regions_converted * 100.0) /
(double)regions_tried : 0.0,
sofar);
bu_free(sofar, "path string");
}
@@ -534,9 +534,9 @@
db_close(dbip);
if (regions_tried > 0)
- percent = (regions_converted * 100) / regions_tried;
+ percent = (double)regions_converted * 100.0) / (double)regions_tried;
else
- percent = 100;
+ percent = 100.0;
printf("Tried %d regions, %d converted successfully. %g%%\n",
regions_tried, regions_converted, percent);
This was sent by the SourceForge.net collaborative development platform, the
world's largest Open Source development site.
------------------------------------------------------------------------------
Introducing Performance Central, a new site from SourceForge and
AppDynamics. Performance Central is your source for news, insights,
analysis and resources for efficient Application Performance Management.
Visit us today!
http://pubads.g.doubleclick.net/gampad/clk?id=48897511&iu=/4140/ostg.clktrk
_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits
