Make inline functions static.
authorAdam Borowski <kilobyte@angband.pl>
Thu, 9 Aug 2012 12:05:34 +0000 (14:05 +0200)
committerAdam Borowski <kilobyte@angband.pl>
Thu, 9 Aug 2012 12:35:28 +0000 (14:35 +0200)
This avoids unnecessary non-inlined copies.

Also, we had a few large functions that had no reason for inlining, let's
have them be regular ones.

I also made "static inline" always use the same order, for easier grepping.

27 files changed:
crawl-ref/source/AppHdr.h
crawl-ref/source/artefact.cc
crawl-ref/source/cio.h
crawl-ref/source/cluautil.h
crawl-ref/source/coord.h
crawl-ref/source/debug.h
crawl-ref/source/delay.cc
crawl-ref/source/dgn-overview.cc
crawl-ref/source/dungeon.h
crawl-ref/source/effects.cc
crawl-ref/source/externs.h
crawl-ref/source/format.cc
crawl-ref/source/libconsole.h
crawl-ref/source/libunix.cc
crawl-ref/source/libutil.h
crawl-ref/source/map_knowledge.h
crawl-ref/source/mpr.h
crawl-ref/source/msvc.h
crawl-ref/source/mutation.h
crawl-ref/source/process_desc.h
crawl-ref/source/skills.h
crawl-ref/source/tags.h
crawl-ref/source/tileview.h
crawl-ref/source/tileweb.cc
crawl-ref/source/travel.cc
crawl-ref/source/viewgeom.h
crawl-ref/source/xom.h

index 19c30a9..9749b8a 100644 (file)
 #ifdef __cplusplus
 
 template < class T >
-inline void UNUSED(const volatile T &)
+static inline void UNUSED(const volatile T &)
 {
 }
 
index 918c203..a4851e7 100644 (file)
@@ -578,7 +578,7 @@ void artefact_desc_properties(const item_def &item,
     }
 }
 
-inline static void _randart_propset(artefact_properties_t &p,
+static inline void _randart_propset(artefact_properties_t &p,
                                      artefact_prop_type pt,
                                      int value,
                                      bool neg)
index 357404b..6d102cd 100644 (file)
@@ -33,7 +33,7 @@ private:
 };
 
 void cursorxy(int x, int y);
-inline void cursorxy(const coord_def& p) { cursorxy(p.x, p.y); }
+static inline void cursorxy(const coord_def& p) { cursorxy(p.x, p.y); }
 
 // Read one key, flag it as a mouse event if appropriate, but apply no
 // other conversions. Defined in lib$OS.cc, not in cio.cc.
index 6818dec..55b729b 100644 (file)
@@ -67,7 +67,7 @@ int clua_stringtable(lua_State *ls, const std::vector<std::string> &s);
  */
 
 template <class T>
-inline static T *clua_get_lightuserdata(lua_State *ls, int ndx)
+static inline T *clua_get_lightuserdata(lua_State *ls, int ndx)
 {
     return (lua_islightuserdata(ls, ndx))?
             static_cast<T *>(lua_touserdata(ls, ndx))
@@ -75,7 +75,7 @@ inline static T *clua_get_lightuserdata(lua_State *ls, int ndx)
 }
 
 template <class T>
-inline static T *clua_get_userdata(lua_State *ls, const char *mt, int ndx = 1)
+static inline T *clua_get_userdata(lua_State *ls, const char *mt, int ndx = 1)
 {
     return static_cast<T*>(luaL_checkudata(ls, ndx, mt));
 }
@@ -99,7 +99,7 @@ template <class T> T *clua_new_userdata(
 }
 
 template <typename T>
-inline void dlua_push_userdata(lua_State *ls, T udata, const char *meta)
+static inline void dlua_push_userdata(lua_State *ls, T udata, const char *meta)
 {
     T *de = clua_new_userdata<T>(ls, meta);
     *de = udata;
index d83b5df..62f7773 100644 (file)
@@ -42,7 +42,7 @@ static inline bool map_bounds(const coord_def &p)
 bool map_bounds_with_margin(coord_def p, int margin);
 
 // Determines if the coordinate is within bounds of an LOS array.
-inline bool show_bounds(const coord_def &p)
+static inline bool show_bounds(const coord_def &p)
 {
     return (p.x >= 0 && p.x < ENV_SHOW_DIAMETER
             && p.y >= 0 && p.y < ENV_SHOW_DIAMETER);
index e8182ca..5ffe046 100644 (file)
@@ -66,7 +66,7 @@ NORETURN void AssertFailed(const char *expr, const char *file, int line);
 #define ASSERT(p)       ((void) 0)
 #define VERIFY(p)       do {if (p) ;} while (false)
 
-inline void __DUMMY_TRACE__(...)
+static inline void __DUMMY_TRACE__(...)
 {
 }
 
index d71b786..3c7f685 100644 (file)
@@ -1621,7 +1621,7 @@ static bool _should_stop_activity(const delay_queue_item &item,
             || Options.activity_interrupts[item.type][ai]);
 }
 
-inline static bool _monster_warning(activity_interrupt_type ai,
+static inline bool _monster_warning(activity_interrupt_type ai,
                                     const activity_interrupt_data &at,
                                     delay_type atype,
                                     std::vector<std::string>* msgs_buf = NULL)
index afd5a1b..bec9b71 100644 (file)
@@ -167,12 +167,12 @@ static std::string shoptype_to_string(shop_type s)
     }
 }
 
-inline static std::string place_desc(const level_pos &pos)
+static inline std::string place_desc(const level_pos &pos)
 {
     return "[" + pos.id.describe(false, true) + "] ";
 }
 
-inline static std::string altar_description(god_type god)
+static inline std::string altar_description(god_type god)
 {
     return feature_description(altar_for_god(god));
 }
@@ -588,7 +588,7 @@ static std::string _get_notes()
 }
 
 template <typename Z, typename Key>
-inline static bool _find_erase(Z &map, const Key &k)
+static inline bool _find_erase(Z &map, const Key &k)
 {
     if (map.find(k) != map.end())
     {
index 99341f6..ff0efa8 100644 (file)
@@ -297,7 +297,7 @@ vault_placement *dgn_vault_at(coord_def gp);
 void dgn_seen_vault_at(coord_def gp);
 
 int count_neighbours(int x, int y, dungeon_feature_type feat);
-inline int count_neighbours(const coord_def& p, dungeon_feature_type feat)
+static inline int count_neighbours(const coord_def& p, dungeon_feature_type feat)
 {
     return count_neighbours(p.x, p.y, feat);
 }
index 9c361aa..456e450 100644 (file)
@@ -1103,7 +1103,7 @@ void yell(bool force)
     noisy(noise_level, you.pos());
 }
 
-inline static dungeon_feature_type _vitrified_feature(dungeon_feature_type feat)
+static inline dungeon_feature_type _vitrified_feature(dungeon_feature_type feat)
 {
     switch (feat)
     {
index 179e99d..5addd1a 100644 (file)
@@ -95,7 +95,7 @@ template <typename Z> inline Z sgn(Z x)
     return (x < 0 ? -1 : (x > 0 ? 1 : 0));
 }
 
-inline int dist_range(int x) { return x*x + 1; };
+static inline int dist_range(int x) { return x*x + 1; };
 
 struct coord_def
 {
index 26ef08d..f3bf039 100644 (file)
@@ -290,7 +290,7 @@ int formatted_string::width() const
     return len;
 }
 
-inline void cap(int &i, int max)
+static inline void cap(int &i, int max)
 {
     if (i < 0 && -i <= max)
         i += max;
index b440330..5ad692d 100644 (file)
@@ -35,7 +35,7 @@ void enable_smart_cursor(bool cursor);
 bool is_smart_cursor_enabled();
 void set_mouse_enabled(bool enabled);
 
-inline void put_colour_ch(int colour, unsigned ch)
+static inline void put_colour_ch(int colour, unsigned ch)
 {
     textcolor(colour);
     putwch(ch);
index 367e0e1..38fe1df 100644 (file)
@@ -587,7 +587,7 @@ void enable_smart_cursor(bool dummy)
 {
 }
 
-inline unsigned get_brand(int col)
+static inline unsigned get_brand(int col)
 {
     return (col & COLFLAG_FRIENDLY_MONSTER) ? Options.friend_brand :
            (col & COLFLAG_NEUTRAL_MONSTER)  ? Options.neutral_brand :
index abeac9e..81fe9b0 100644 (file)
@@ -129,7 +129,7 @@ std::string &trim_string(std::string &str);
 std::string &trim_string_right(std::string &str);
 std::string trimmed_string(std::string s);
 
-inline bool starts_with(const std::string &s, const std::string &prefix)
+static inline bool starts_with(const std::string &s, const std::string &prefix)
 {
     return (s.rfind(prefix, 0) != std::string::npos);
 }
index a101f23..82ee2e5 100644 (file)
@@ -298,19 +298,19 @@ private:
 };
 
 void set_terrain_mapped(int x, int y);
-inline void set_terrain_mapped(const coord_def& c)
+static inline void set_terrain_mapped(const coord_def& c)
 {
     set_terrain_mapped(c.x,c.y);
 }
 
 void set_terrain_seen(int x, int y);
-inline void set_terrain_seen(const coord_def& c)
+static inline void set_terrain_seen(const coord_def& c)
 {
     set_terrain_seen(c.x, c.y);
 }
 
 void set_terrain_changed(int x, int y);
-inline void set_terrain_changed(const coord_def &c)
+static inline void set_terrain_changed(const coord_def &c)
 {
     set_terrain_changed(c.x, c.y);
 }
index b64d966..055526f 100644 (file)
@@ -85,13 +85,13 @@ msg_colour_type msg_colour(int colour);
 void mpr(std::string text, msg_channel_type channel=MSGCH_PLAIN, int param=0,
          bool nojoin=false, bool cap=true);
 
-inline void mprnojoin(std::string text, msg_channel_type channel=MSGCH_PLAIN,
+static inline void mprnojoin(std::string text, msg_channel_type channel=MSGCH_PLAIN,
                       int param=0)
 {
     mpr(text, channel, param, true);
 }
 
-inline void mpr_nocap(std::string text, msg_channel_type channel=MSGCH_PLAIN,
+static inline void mpr_nocap(std::string text, msg_channel_type channel=MSGCH_PLAIN,
                       int param=0, bool nojoin=false)
 {
     mpr(text, channel, param, nojoin, false);
index 4ebe802..b07e524 100644 (file)
@@ -49,7 +49,7 @@
 #pragma warning(disable : 4996)
 
 
-inline double round(double x)
+static inline double round(double x)
 {
     if (x >= 0.0)
         return floor(x + 0.5);
@@ -57,27 +57,27 @@ inline double round(double x)
         return ceil(x - 0.5);
 }
 
-inline double sqrt(int x)
+static inline double sqrt(int x)
 {
     return sqrt((double)x);
 }
 
-inline double atan2(int x, int y)
+static inline double atan2(int x, int y)
 {
     return atan2((double)x, (double)y);
 }
 
-inline double pow(int x, double y)
+static inline double pow(int x, double y)
 {
     return pow((double)x, y);
 }
 
-inline double pow(int x, int y)
+static inline double pow(int x, int y)
 {
     return pow((double)x, y);
 }
 
-inline double log(int x)
+static inline double log(int x)
 {
     return log((double)x);
 }
index d5b6b9d..e0a1827 100644 (file)
@@ -75,7 +75,7 @@ bool mutate(mutation_type which_mutation, const std::string &reason,
             bool stat_gain_potion = false, bool demonspawn = false,
             bool no_rot = false);
 
-inline bool give_bad_mutation(const std::string &reason,
+static inline bool give_bad_mutation(const std::string &reason,
                               bool failMsg = true,
                               bool force_mutation = false)
 {
index 3f4afbf..a1cfa71 100644 (file)
@@ -17,7 +17,7 @@ template<class T> void process_quote(T &proc, const describe_info &inf);
  * *********************************************************************** */
 // My kingdom for a closure.
 template<class T>
-inline void process_description(T &proc, const describe_info &inf)
+void process_description(T &proc, const describe_info &inf)
 {
     const unsigned int line_width = proc.width();
     const          int height     = proc.height();
@@ -99,7 +99,7 @@ inline void process_description(T &proc, const describe_info &inf)
 }
 
 template<class T>
-inline void process_quote(T &proc, const describe_info &inf)
+void process_quote(T &proc, const describe_info &inf)
 {
     const unsigned int line_width = proc.width();
     const          int height     = proc.height();
index 7090899..6b4e937 100644 (file)
@@ -34,7 +34,7 @@ void exercise(skill_type exsk, int deg);
 void train_skills(bool simu = false);
 void train_skill(skill_type skill, int exp);
 bool skill_trained(int i);
-inline bool skill_trained(skill_type sk) { return skill_trained((int) sk); }
+static inline bool skill_trained(skill_type sk) { return skill_trained((int) sk); }
 void redraw_skill(skill_type exsk, skill_type old_best_skill = SK_NONE);
 void set_skill_level(skill_type skill, double amount);
 #endif
index de092cd..567fbce 100644 (file)
@@ -140,7 +140,7 @@ void marshall_level_id(writer& th, const level_id& id);
 void marshallEnumVal (writer &, const enum_info *, int);
 
 template<typename enm>
-inline void marshallEnum(writer& wr, enm value)
+static inline void marshallEnum(writer& wr, enm value)
 {
     marshallEnumVal(wr, &enum_details<enm>::desc, static_cast<int>(value));
 }
@@ -211,7 +211,7 @@ level_id    unmarshall_level_id(reader& th);
 int         unmarshallEnumVal (reader &, const enum_info *);
 
 template<typename enm>
-inline enm unmarshallEnum(writer& wr)
+static inline enm unmarshallEnum(writer& wr)
 {
     return static_cast<enm>(unmarshallEnumVal(wr, &enum_details<enm>::desc));
 }
index fe5f75f..b7f0a10 100644 (file)
@@ -21,7 +21,7 @@ struct packed_cell;
 // Initialize the flavour and the tile env when changing or creating levels.
 void tile_new_level(bool first_time, bool init_unseen);
 
-inline void tile_new_level(bool first_time)
+static inline void tile_new_level(bool first_time)
 {
     return tile_new_level(first_time, first_time);
 }
index 3d766f0..a370be3 100644 (file)
@@ -612,7 +612,7 @@ static bool _needs_flavour(const packed_cell &cell)
     return false;
 }
 
-inline unsigned _get_brand(int col)
+static inline unsigned _get_brand(int col)
 {
     return (col & COLFLAG_FRIENDLY_MONSTER) ? Options.friend_brand :
            (col & COLFLAG_NEUTRAL_MONSTER)  ? Options.neutral_brand :
@@ -625,7 +625,7 @@ inline unsigned _get_brand(int col)
                                             : CHATTR_NORMAL;
 }
 
-inline void _write_tileidx(tileidx_t t)
+static inline void _write_tileidx(tileidx_t t)
 {
     // JS can only handle signed ints
     const int lo = t & 0xFFFFFFFF;
index 6057c43..8d21cd3 100644 (file)
@@ -285,7 +285,7 @@ uint8_t is_waypoint(const coord_def &p)
     return curr_waypoints[p.x][p.y];
 }
 
-inline bool is_stash(const LevelStashes *ls, const coord_def& p)
+static inline bool is_stash(const LevelStashes *ls, const coord_def& p)
 {
     if (!ls)
         return false;
@@ -555,7 +555,7 @@ static bool _prompt_stop_explore(int es_why)
 #define ES_sacrificiable (Options.explore_stop & ES_GREEDY_SACRIFICIABLE)
 
 // Adds interesting stuff on the point p to explore_discoveries.
-inline static void _check_interesting_square(const coord_def pos,
+static inline void _check_interesting_square(const coord_def pos,
                                              explore_discoveries &ed)
 {
     if ((ES_item || ES_greedy || ES_glow || ES_art || ES_rune)
index 8bf32f0..862246a 100644 (file)
@@ -160,42 +160,42 @@ public:
 
 extern crawl_view_geometry crawl_view;
 
-inline coord_def view2grid(const coord_def &pos)
+static inline coord_def view2grid(const coord_def &pos)
 {
     return crawl_view.view2grid(pos);
 }
 
-inline coord_def grid2view(const coord_def &pos)
+static inline coord_def grid2view(const coord_def &pos)
 {
     return crawl_view.grid2view(pos);
 }
 
-inline coord_def view2show(const coord_def &pos)
+static inline coord_def view2show(const coord_def &pos)
 {
     return crawl_view.view2show(pos);
 }
 
-inline coord_def show2view(const coord_def &pos)
+static inline coord_def show2view(const coord_def &pos)
 {
     return crawl_view.show2view(pos);
 }
 
-inline coord_def grid2show(const coord_def &pos)
+static inline coord_def grid2show(const coord_def &pos)
 {
     return crawl_view.grid2show(pos);
 }
 
-inline coord_def show2grid(const coord_def &pos)
+static inline coord_def show2grid(const coord_def &pos)
 {
     return crawl_view.show2grid(pos);
 }
 
-inline bool in_los_bounds_v(const coord_def& pos)
+static inline bool in_los_bounds_v(const coord_def& pos)
 {
     return crawl_view.in_los_bounds_v(pos);
 }
 
-inline bool in_los_bounds_g(const coord_def& pos)
+static inline bool in_los_bounds_g(const coord_def& pos)
 {
     return crawl_view.in_los_bounds_g(pos);
 }
index dcd85cb..e47ee6f 100644 (file)
@@ -81,7 +81,7 @@ bool xom_is_nice(int tension = -1);
 int xom_acts(bool niceness, int sever, int tension = -1, bool debug = false);
 const std::string describe_xom_favour();
 
-inline int xom_acts(int sever, int tension = -1)
+static inline int xom_acts(int sever, int tension = -1)
 {
     return xom_acts(xom_is_nice(tension), sever, tension);
 }