Line|Map: Respect logical constness when manipulating Line::Side's sh…

…adow count
skyjake · Dec 11, 2013 · fa8c0fe · fa8c0fe
1 parent f79d60f
commit fa8c0fe
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/doomsday/client/include/world/line.h b/doomsday/client/include/world/line.h
@@ -532,7 +532,7 @@ class Line : public de::MapElement
          *
          * @param newCount  New shadow vis count.
          */
-        void setShadowVisCount(int newCount) const;
+        void setShadowVisCount(int newCount);
 
 #ifdef __CLIENT__
 

diff --git a/doomsday/client/src/render/rend_fakeradio.cpp b/doomsday/client/src/render/rend_fakeradio.cpp
@@ -1328,7 +1328,8 @@ void Rend_RadioBspLeafEdges(BspLeaf const &bspLeaf)
 
     // We need to check all the shadow lines linked to this BspLeaf for
     // the purpose of fakeradio shadowing.
-    foreach(LineSide const *side, bspLeaf.shadowLines())
+    BspLeaf::ShadowLines const &shadowLines = bspLeaf.shadowLines();
+    foreach(LineSide *side, shadowLines)
     {
         // Already rendered during the current frame? We only want to
         // render each shadow once per frame.

diff --git a/doomsday/client/src/world/line.cpp b/doomsday/client/src/world/line.cpp
@@ -666,7 +666,7 @@ int Line::Side::shadowVisCount() const
     return d->shadowVisCount;
 }
 
-void Line::Side::setShadowVisCount(int newCount) const
+void Line::Side::setShadowVisCount(int newCount)
 {
     d->shadowVisCount = newCount;
 }