Bsp Builder|Refactor: Code clarity and minor optimization refactorings

skyjake · Aug 31, 2012 · 3e78fc2 · 3e78fc2
1 parent 7d9ae76
commit 3e78fc2
Show file tree

Hide file tree

Showing 4 changed files with 244 additions and 259 deletions.
diff --git a/doomsday/engine/portable/include/map/bsp/superblockmap.h b/doomsday/engine/portable/include/map/bsp/superblockmap.h
@@ -71,15 +71,17 @@ class SuperBlockmap
      * this then @a bounds will be initialized to the "cleared" state
      * (i.e., min[x,y] > max[x,y]).
      *
-     * @param bounds  Determined bounds are written here.
+     * @return bounds  Determined bounds are written here.
      */
-    void findHEdgeBounds(AABoxd& bounds);
+    AABoxd findHEdgeBounds();
 
     /**
      * Empty this SuperBlockmap clearing all HEdges and sub-blocks.
      */
     void clear();
 
+    operator SuperBlock&() { return root(); }
+
 private:
     struct Instance;
     Instance* d;

diff --git a/doomsday/engine/portable/src/edit_map.c b/doomsday/engine/portable/src/edit_map.c
@@ -1623,7 +1623,8 @@ static boolean buildBsp(GameMap* gamemap)
 
     if(!map) return false;
 
-    VERBOSE( Con_Message("Building BSP using tunable split factor of %d...\n", bspFactor) )
+    LegacyCore_PrintfLogFragmentAtLevel(DE2_LOG_INFO,
+        "Building BSP using tunable split factor of %d...\n", bspFactor);
 
     // It begins...
     startTime = Sys_GetRealTime();
@@ -1640,7 +1641,8 @@ static boolean buildBsp(GameMap* gamemap)
     BspBuilder_Delete(bspBuilder);
 
     // How much time did we spend?
-    VERBOSE2( Con_Message("  Done in %.2f seconds.\n", (Sys_GetRealTime() - startTime) / 1000.0f) );
+    LegacyCore_PrintfLogFragmentAtLevel(DE2_LOG_INFO,
+        "BSP built in %.2f seconds.\n", (Sys_GetRealTime() - startTime) / 1000.0f);
     return builtOK;
 }