Rename actor::visible to actor::observable.

This should reduce potential for confusion -- compare mon->visible() and
mon->invisible(). The uses of actor::visible for messaging are replaced by
actor::observable, others by you.can_see(...) or actor::visible_to(&you).

1 files changed, 11 insertions, 0 deletions

diff --git a/crawl-ref/source/actor.h b/crawl-ref/source/actor.h
index f8ff1ad599..e5cf5e6277 100644
--- a/crawl-ref/source/actor.h
+++ b/crawl-ref/source/actor.h
@@ -101,9 +101,20 @@ public:
     virtual bool can_go_berserk() const = 0;
     virtual bool can_see_invisible() const = 0;
     virtual bool invisible() const = 0;
+
+    // Would looker be able to see the actor when in LOS?
     virtual bool visible_to(const actor *looker) const = 0;
+
+    // Is the given cell within LOS of the actor?
     virtual bool see_cell(const coord_def &c) const = 0;
+
+    // Can the actor actually see the target?
     virtual bool can_see(const actor *target) const = 0;
+
+    // Visibility as required by messaging. In usual play:
+    //   Does the player know what's happening to the actor?
+    virtual bool observable();
+
     virtual bool is_icy() const = 0;
     virtual bool is_fiery() const = 0;
     virtual void go_berserk(bool intentional) = 0;