Add actor::set_position.

This should be used instead of setting position directly.
actor::position could be made "protected" except for the arena's
unwind_var(you.position), which is too difficult for me to fix.

The reason behind this change is that actors should get their
own LOS, whose origin should be synchronized with the actor's
position.

This change also removes the non-const "coord_def& actor::pos()" (yuk).

1 files changed, 3 insertions, 2 deletions

diff --git a/crawl-ref/source/tags.cc b/crawl-ref/source/tags.cc
index f65421e5a1..0d140664e5 100644
--- a/crawl-ref/source/tags.cc
+++ b/crawl-ref/source/tags.cc
@@ -2186,8 +2186,9 @@ static void unmarshall_monster(reader &th, monsters &m)
     // Avoid sign extension when loading files (Elethiomel's hang)
     m.speed_increment = (unsigned char) unmarshallByte(th);
     m.behaviour       = static_cast<beh_type>(unmarshallByte(th));
-    m.position.x      = unmarshallByte(th);
-    m.position.y      = unmarshallByte(th);
+    int x             = unmarshallByte(th);
+    int y             = unmarshallByte(th);
+    m.set_position(coord_def(x,y));
     m.target.x        = unmarshallByte(th);
     m.target.y        = unmarshallByte(th);