From 395188726a1546d4e0fa178780189dfd5d183c04 Mon Sep 17 00:00:00 2001
From: danij <danij@dengine.net>
Date: Tue, 6 May 2014 01:32:26 +0100
Subject: [PATCH] World|BspLeaf: Simplified BspLeaf (no need for pimpl here)

---
 doomsday/client/include/world/bspleaf.h |  2 +-
 doomsday/client/src/world/bspleaf.cpp   | 24 +++++++++---------------
 2 files changed, 10 insertions(+), 16 deletions(-)

diff --git a/doomsday/client/include/world/bspleaf.h b/doomsday/client/include/world/bspleaf.h
index 8220537b53..c905de6afd 100644
--- a/doomsday/client/include/world/bspleaf.h
+++ b/doomsday/client/include/world/bspleaf.h
@@ -99,7 +99,7 @@ class BspLeaf : public de::MapElement
     Sector const *sectorPtr() const;
 
 private:
-    DENG2_PRIVATE(d)
+    ConvexSubspace *_subspace;
 };
 
 #endif // DENG_WORLD_BSPLEAF_H
diff --git a/doomsday/client/src/world/bspleaf.cpp b/doomsday/client/src/world/bspleaf.cpp
index 0e73b99d88..36fd842479 100644
--- a/doomsday/client/src/world/bspleaf.cpp
+++ b/doomsday/client/src/world/bspleaf.cpp
@@ -27,27 +27,21 @@
 
 using namespace de;
 
-DENG2_PIMPL_NOREF(BspLeaf)
-{
-    ConvexSubspace *subspace;
-};
-
 BspLeaf::BspLeaf(Sector *sector)
-    : MapElement(DMU_BSPLEAF, sector), d(new Instance)
-{
-    d->subspace = 0;
-}
+    : MapElement(DMU_BSPLEAF, sector)
+    , _subspace(0)
+{}
 
 bool BspLeaf::hasSubspace() const
 {
-    return d->subspace != 0;
+    return _subspace != 0;
 }
 
 ConvexSubspace &BspLeaf::subspace() const
 {
     if(hasSubspace())
     {
-        return *d->subspace;
+        return *_subspace;
     }
     /// @throw MissingSubspaceError Attempted with no subspace attributed.
     throw MissingSubspaceError("BspLeaf::subspace", "No subspace is attributed");
@@ -55,18 +49,18 @@ ConvexSubspace &BspLeaf::subspace() const
 
 void BspLeaf::setSubspace(ConvexSubspace *newSubspace)
 {
-    if(d->subspace == newSubspace) return;
+    if(_subspace == newSubspace) return;
 
     if(hasSubspace())
     {
-        d->subspace->setBspLeaf(0);
+        _subspace->setBspLeaf(0);
     }
 
-    d->subspace = newSubspace;
+    _subspace = newSubspace;
 
     if(hasSubspace())
     {
-        d->subspace->setBspLeaf(this);
+        _subspace->setBspLeaf(this);
     }
 }