Replace `mpr(s, ch)` with `mprf(ch, s)`

Also simplify quite a few cases.

It turns out in >90% cases of non-literals the argument had .c_str(),
which meant it was pointlessly malloc()ed and converted from and to
std::string.  I believe a sprintf is faster, so even the argument of
miniscule speed-up doesn't apply.

1 files changed, 1 insertions, 1 deletions

diff --git a/crawl-ref/source/beam.h b/crawl-ref/source/beam.h
index daa624548c..3edb1edace 100644
--- a/crawl-ref/source/beam.h
+++ b/crawl-ref/source/beam.h
@@ -236,7 +236,7 @@ private:
     string zapper() const;
 
     set<string> message_cache;
-    void emit_message(msg_channel_type chan, const char* msg);
+    void emit_message(const char* msg);
     void step();
     bool hit_wall();