Drop a bunch of parentheses from return statements.
authorAdam Borowski <kilobyte@angband.pl>
Tue, 7 Aug 2012 09:23:15 +0000 (11:23 +0200)
committerAdam Borowski <kilobyte@angband.pl>
Wed, 8 Aug 2012 13:55:12 +0000 (15:55 +0200)
This is incomplete, partially because of me getting bored, partially because
of doubts about the point of leaving simple addition/etc in parentheses.

56 files changed:
crawl-ref/source/abyss.cc
crawl-ref/source/act-iter.cc
crawl-ref/source/actor-los.cc
crawl-ref/source/actor.cc
crawl-ref/source/areas.cc
crawl-ref/source/artefact.cc
crawl-ref/source/attack.cc
crawl-ref/source/beam.cc
crawl-ref/source/behold.cc
crawl-ref/source/branch.cc
crawl-ref/source/cloud.cc
crawl-ref/source/clua.cc
crawl-ref/source/colour.cc
crawl-ref/source/command.cc
crawl-ref/source/decks.cc
crawl-ref/source/delay.cc
crawl-ref/source/describe.cc
crawl-ref/source/dgnevent.cc
crawl-ref/source/directn.cc
crawl-ref/source/dungeon.cc
crawl-ref/source/exclude.cc
crawl-ref/source/exercise.cc
crawl-ref/source/fearmonger.cc
crawl-ref/source/feature.cc
crawl-ref/source/fight.cc
crawl-ref/source/files.cc
crawl-ref/source/food.cc
crawl-ref/source/format.cc
crawl-ref/source/fprop.cc
crawl-ref/source/geom2d.cc
crawl-ref/source/godprayer.cc
crawl-ref/source/hiscores.cc
crawl-ref/source/initfile.cc
crawl-ref/source/invent.cc
crawl-ref/source/item_use.cc
crawl-ref/source/itemname.cc
crawl-ref/source/itemprop.cc
crawl-ref/source/items.cc
crawl-ref/source/jobs.cc
crawl-ref/source/kills.cc
crawl-ref/source/l_crawl.cc
crawl-ref/source/l_debug.cc
crawl-ref/source/l_dgnevt.cc
crawl-ref/source/l_item.cc
crawl-ref/source/l_mapgrd.cc
crawl-ref/source/l_mons.cc
crawl-ref/source/l_option.cc
crawl-ref/source/l_you.cc
crawl-ref/source/libgui.cc
crawl-ref/source/libunix.cc
crawl-ref/source/libutil.cc
crawl-ref/source/losglobal.cc
crawl-ref/source/macro.cc
crawl-ref/source/menu.h
crawl-ref/source/mutation.h
crawl-ref/source/random-weight.h

index 82c464e..84ad8dc 100644 (file)
@@ -1786,7 +1786,7 @@ static bool _is_level_corrupted()
     if (player_in_branch(BRANCH_ABYSS))
         return true;
 
-    return (!!env.markers.find(MAT_CORRUPTION_NEXUS));
+    return !!env.markers.find(MAT_CORRUPTION_NEXUS);
 }
 
 bool is_level_incorruptible()
index 251c9c7..3eb0d19 100644 (file)
@@ -70,9 +70,9 @@ bool actor_iterator::valid(const actor* a) const
     switch (restr)
     {
     case R_CIRC:
-        return (circle->contains(a->pos()));
+        return circle->contains(a->pos());
     case R_LOS:
-        return (los->see_cell(a->pos()));
+        return los->see_cell(a->pos());
     case R_ACT:
         return act->can_see(a);
     default:
index dd566f9..136e0e1 100644 (file)
@@ -18,7 +18,7 @@ bool actor::see_cell(const coord_def &p) const
     if (!in_bounds(pos()))
         return false; // actor is off the map
 
-    return (cell_see_cell(pos(), p, LOS_DEFAULT));
+    return cell_see_cell(pos(), p, LOS_DEFAULT);
 }
 
 bool player::see_cell(const coord_def &p) const
@@ -39,7 +39,7 @@ bool actor::can_see(const actor *target) const
 
 bool actor::see_cell_no_trans(const coord_def &p) const
 {
-    return (cell_see_cell(pos(), p, LOS_NO_TRANS));
+    return cell_see_cell(pos(), p, LOS_NO_TRANS);
 }
 
 bool player::trans_wall_blocking(const coord_def &p) const
@@ -81,7 +81,7 @@ const los_base* actor::get_los_no_trans()
 bool player::can_see(const actor* a) const
 {
     if (crawl_state.game_is_arena() || crawl_state.arena_suspended)
-        return (see_cell(a->pos()));
+        return see_cell(a->pos());
     else if (xray_vision)
         return see_cell(a->pos());
     else
index 7afe166..a71ef72 100644 (file)
@@ -422,12 +422,12 @@ void actor::start_constricting(actor &whom, int dur)
 
 int actor::num_constricting() const
 {
-    return (constricting ? constricting->size() : 0);
+    return constricting ? constricting->size() : 0;
 }
 
 bool actor::is_constricting() const
 {
-    return (constricting && !constricting->empty());
+    return constricting && !constricting->empty();
 }
 
 bool actor::is_constricted() const
index 0e3edec..6e97da2 100644 (file)
@@ -536,7 +536,7 @@ bool haloed(const coord_def& p)
 
 bool actor::haloed() const
 {
-    return (::haloed(pos()));
+    return ::haloed(pos());
 }
 
 int player::halo_radius2() const
@@ -680,7 +680,7 @@ bool umbraed(const coord_def& p)
 // Whether actor is in an umbra.
 bool actor::umbraed() const
 {
-    return (::umbraed(pos()));
+    return ::umbraed(pos());
 }
 
 // Stub for player umbra.
@@ -726,7 +726,7 @@ int monster::suppression_radius2() const
 
 bool actor::suppressed() const
 {
-    return (::suppressed(pos()));
+    return ::suppressed(pos());
 }
 
 int player::suppression_radius2() const
index 722c87a..918c203 100644 (file)
@@ -362,7 +362,7 @@ void autoid_unrand(item_def &item)
 unique_item_status_type get_unique_item_status(int art)
 {
     ASSERT(art > UNRAND_START && art < UNRAND_LAST);
-    return (you.unique_items[art - UNRAND_START]);
+    return you.unique_items[art - UNRAND_START];
 }
 
 static void _set_unique_item_status(int art, unique_item_status_type status)
@@ -681,7 +681,7 @@ static int _need_bonus_stat_props(const artefact_properties_t &proprt)
     if (num_acc_dam > 0)
         return random2(3);
 
-    return (1 + random2(2));
+    return 1 + random2(2);
 }
 
 static void _get_randart_properties(const item_def &item,
@@ -1295,7 +1295,7 @@ int artefact_wpn_property(const item_def &item, artefact_prop_type prop,
 
     _known = known[prop];
 
-    return (proprt[prop]);
+    return proprt[prop];
 }
 
 int artefact_wpn_property(const item_def &item, artefact_prop_type prop)
@@ -1314,7 +1314,7 @@ int artefact_known_wpn_property(const item_def &item,
     artefact_wpn_properties(item, proprt, known);
 
     if (known[prop])
-        return (proprt[prop]);
+        return proprt[prop];
     else
         return 0;
 }
@@ -1396,7 +1396,7 @@ static std::string _artefact_name_lookup(const item_def &item,
                                  const std::string &lookup)
 {
     const std::string name = getRandNameString(lookup);
-    return (!name.empty()? replace_name_parts(name, item) : name);
+    return name.empty() ? name : replace_name_parts(name, item);
 }
 
 static bool _artefact_name_lookup(std::string &result,
@@ -1630,7 +1630,7 @@ int get_unrandart_num(const char *name)
         art = replace_all(art, "'", "");
         lowercase(art);
         if (art == name || art.find(quoted) != std::string::npos)
-            return (UNRAND_START + i);
+            return UNRAND_START + i;
     }
     return SPWPN_NORMAL;
 }
index ed030a6..3e6924e 100644 (file)
@@ -62,7 +62,7 @@ bool attack::handle_phase_attempted()
 std::string attack::actor_name(const actor *a, description_level_type desc,
                                bool actor_visible, bool actor_invisible)
 {
-    return (actor_visible ? a->name(desc) : anon_name(desc, actor_invisible));
+    return actor_visible ? a->name(desc) : anon_name(desc, actor_invisible);
 }
 
 /* Returns an actor's pronoun
@@ -72,7 +72,7 @@ std::string attack::actor_name(const actor *a, description_level_type desc,
 std::string attack::actor_pronoun(const actor *a, pronoun_type pron,
                                   bool actor_visible)
 {
-    return (actor_visible ? a->pronoun(pron) : anon_pronoun(pron));
+    return actor_visible ? a->pronoun(pron) : anon_pronoun(pron);
 }
 
 /* Returns an anonymous actor's name
@@ -94,7 +94,7 @@ std::string attack::anon_name(description_level_type desc,
     case DESC_A:
     case DESC_PLAIN:
     default:
-        return (actor_invisible? "it" : "something");
+        return actor_invisible? "it" : "something";
     }
 }
 
index f2fb120..5f5b736 100644 (file)
@@ -408,7 +408,7 @@ static const zap_info* _seek_zap(zap_type z_type)
     if (zap_index[z_type] == -1)
         return NULL;
     else
-        return (&zap_data[zap_index[z_type]]);
+        return &zap_data[zap_index[z_type]];
 }
 
 int zap_power_cap(zap_type z_type)
@@ -494,8 +494,8 @@ bool bolt::can_affect_actor(const actor *act) const
 
 bool bolt::actor_wall_shielded(const actor *act) const
 {
-    return (act->is_player()? false :
-            mons_wall_shielded(act->as_monster()));
+    return act->is_player()? false :
+           mons_wall_shielded(act->as_monster());
 }
 
 // Affect actor in wall unless it can shield itself using the wall.
@@ -2359,7 +2359,7 @@ static int _potion_beam_flavour_to_colour(beam_type flavour)
         return GREEN;
 
     case BEAM_POTION_POISON:
-        return (coinflip() ? GREEN : LIGHTGREEN);
+        return coinflip() ? GREEN : LIGHTGREEN;
 
     case BEAM_POTION_MIASMA:
     case BEAM_POTION_BLACK_SMOKE:
@@ -2370,10 +2370,10 @@ static int _potion_beam_flavour_to_colour(beam_type flavour)
         return LIGHTGREY;
 
     case BEAM_POTION_FIRE:
-        return (coinflip() ? RED : LIGHTRED);
+        return coinflip() ? RED : LIGHTRED;
 
     case BEAM_POTION_COLD:
-        return (coinflip() ? BLUE : LIGHTBLUE);
+        return coinflip() ? BLUE : LIGHTBLUE;
 
     case BEAM_POTION_BLUE_SMOKE:
         return LIGHTBLUE;
@@ -5644,7 +5644,7 @@ bool bolt::nasty_to(const monster* mon) const
 
     // Friendly and good neutral monsters don't mind being teleported.
     if (flavour == BEAM_TELEPORT)
-        return (!mon->wont_attack());
+        return !mon->wont_attack();
 
     // degeneration / sleep / enslave soul
     if (flavour == BEAM_DEGENERATE
@@ -5660,7 +5660,7 @@ bool bolt::nasty_to(const monster* mon) const
 
     // pain / agony
     if (flavour == BEAM_PAIN)
-        return (!mon->res_negative_energy());
+        return !mon->res_negative_energy();
 
     // everything else is considered nasty by everyone
     return true;
@@ -5803,7 +5803,7 @@ actor* bolt::agent() const
     if (YOU_KILL(nominal_ktype))
         return &you;
     else if (!invalid_monster_index(nominal_source))
-        return (&menv[nominal_source]);
+        return &menv[nominal_source];
     else
         return NULL;
 }
index 5420d3b..633e3a7 100644 (file)
@@ -102,7 +102,7 @@ monster* player::get_beholder(const coord_def &target) const
 monster* player::get_any_beholder() const
 {
     if (!beholders.empty())
-        return (&menv[beholders[0]]);
+        return &menv[beholders[0]];
     else
         return NULL;
 }
index 6078976..0b515c8 100644 (file)
@@ -58,7 +58,7 @@ branch_type str_to_branch(const std::string &branch, branch_type err)
 {
     for (int i = 0; i < NUM_BRANCHES; ++i)
         if (branches[i].abbrevname && branches[i].abbrevname == branch)
-            return (static_cast<branch_type>(i));
+            return static_cast<branch_type>(i);
 
     return err;
 }
index 08ba2f6..0ab4acc 100644 (file)
@@ -1227,7 +1227,7 @@ bool in_what_cloud(cloud_type type)
 std::string cloud_name_at_index(int cloudno)
 {
     if (!env.cloud[cloudno].name.empty())
-        return (env.cloud[cloudno].name);
+        return env.cloud[cloudno].name;
     else
         return cloud_type_name(env.cloud[cloudno].type);
 }
@@ -1377,7 +1377,7 @@ int get_cloud_colour(int cloudno)
 {
     int which_colour = LIGHTGREY;
     if (env.cloud[cloudno].colour != -1)
-        return (env.cloud[cloudno].colour);
+        return env.cloud[cloudno].colour;
 
     switch (env.cloud[cloudno].type)
     {
index b8b3f75..7b5146c 100644 (file)
@@ -484,7 +484,7 @@ bool CLua::calltopfn(lua_State *ls, const char *params, va_list args,
     lua_call_throttle strangler(this);
     int err = lua_pcall(ls, argc, retc, 0);
     set_error(err, ls);
-    return (!err);
+    return !err;
 }
 
 maybe_bool CLua::callmbooleanfn(const char *fn, const char *params,
index c88efad..9234bd4 100644 (file)
@@ -693,7 +693,7 @@ int str_to_colour(const std::string &str, int default_colour,
             ret = ci;
     }
 
-    return ((ret == 16) ? default_colour : ret);
+    return (ret == 16) ? default_colour : ret;
 }
 
 #if defined(TARGET_OS_WINDOWS) || defined(USE_TILE_LOCAL)
index bf8fda6..a6a133c 100644 (file)
@@ -964,7 +964,7 @@ static bool _spell_filter(std::string key, std::string body)
         return true;
 
     if (get_spell_flags(spell) & (SPFLAG_MONSTER | SPFLAG_TESTING))
-        return (!you.wizard);
+        return !you.wizard;
 
     return false;
 }
index 3204274..72bf66c 100644 (file)
@@ -229,7 +229,7 @@ int cards_in_deck(const item_def &deck)
     const CrawlHashTable &props = deck.props;
     ASSERT(props.exists("cards"));
 
-    return (props["cards"].get_vector().size());
+    return props["cards"].get_vector().size();
 }
 
 static void _shuffle_deck(item_def &deck)
@@ -3117,7 +3117,7 @@ colour_t deck_rarity_to_colour(deck_rarity_type rarity)
     }
 
     case DECK_RARITY_RARE:
-        return (coinflip() ? MAGENTA : BROWN);
+        return coinflip() ? MAGENTA : BROWN;
 
     case DECK_RARITY_LEGENDARY:
         return LIGHTMAGENTA;
index 280311c..d71b786 100644 (file)
@@ -85,7 +85,7 @@ static const char *_activity_interrupt_name(activity_interrupt_type ai);
 static int _zin_recite_to_monsters(coord_def where, int prayertype, int, actor *)
 {
     ASSERT(prayertype >= 0 && prayertype < NUM_RECITE_TYPES);
-    return (zin_recite_to_single_monster(where, (recite_type)prayertype));
+    return zin_recite_to_single_monster(where, (recite_type)prayertype);
 }
 
 static std::string _get_zin_recite_speech(int trits[], size_t len, int prayertype, int step)
index 67a6e4d..7ffcc8b 100644 (file)
@@ -548,7 +548,7 @@ std::string trap_name(trap_type trap)
     COMPILE_CHECK(ARRAYSZ(trap_names) == NUM_TRAPS);
 
     if (trap >= TRAP_DART && trap < NUM_TRAPS)
-        return (trap_names[trap]);
+        return trap_names[trap];
     return "";
 }
 
@@ -4017,14 +4017,14 @@ static const char *divine_title[NUM_GODS][8] =
 
 static int _piety_level()
 {
-    return ((you.piety >  160) ? 7 :
-            (you.piety >= 120) ? 6 :
-            (you.piety >= 100) ? 5 :
-            (you.piety >=  75) ? 4 :
-            (you.piety >=  50) ? 3 :
-            (you.piety >=  30) ? 2 :
-            (you.piety >    5) ? 1
-                               : 0);
+    return (you.piety >  160) ? 7 :
+           (you.piety >= 120) ? 6 :
+           (you.piety >= 100) ? 5 :
+           (you.piety >=  75) ? 4 :
+           (you.piety >=  50) ? 3 :
+           (you.piety >=  30) ? 2 :
+           (you.piety >    5) ? 1
+                              : 0;
 }
 
 std::string god_title(god_type which_god, species_type which_species)
index b870b01..9f07997 100644 (file)
@@ -35,7 +35,7 @@ void dgn_event_dispatcher::move_listeners(
 
 bool dgn_event_dispatcher::has_listeners_at(const coord_def &pos) const
 {
-    return (grid_triggers[pos.x][pos.y].get());
+    return grid_triggers[pos.x][pos.y].get();
 }
 
 bool dgn_event_dispatcher::fire_vetoable_position_event(
index 7da85b0..6c7c155 100644 (file)
@@ -2739,9 +2739,9 @@ static bool _find_square(coord_def &mfp, int direction,
     }
 
     mfp = (direction > 0 ? coord_def(ctrx, ctry) : coord_def(minx, maxy));
-    return (_find_square(mfp, direction, find_targ, need_path,
-                         mode, range, hitfunc, false,
-                         _next_los(direction, los, wrap)));
+    return _find_square(mfp, direction, find_targ, need_path,
+                        mode, range, hitfunc, false,
+                        _next_los(direction, los, wrap));
 }
 
 // XXX Unbelievably hacky. And to think that my goal was to clean up the code.
@@ -3290,9 +3290,9 @@ std::string feature_description_at(const coord_def& where, bool covering,
     case DNGN_TRAP_MAGICAL:
     case DNGN_TRAP_NATURAL:
     case DNGN_TRAP_WEB:
-        return (feature_description(grid, get_trap_type(where),
-                                    covering_description, dtype,
-                                    add_stop, base_desc));
+        return feature_description(grid, get_trap_type(where),
+                                   covering_description, dtype,
+                                   add_stop, base_desc);
     case DNGN_ABANDONED_SHOP:
         return thing_do_grammar(dtype, add_stop, false, "an abandoned shop");
 
@@ -3301,9 +3301,9 @@ std::string feature_description_at(const coord_def& where, bool covering,
 
     case DNGN_ENTER_PORTAL_VAULT:
         // Should have been handled at the top of the function.
-        return (thing_do_grammar(
-                    dtype, add_stop, false,
-                    "UNAMED PORTAL VAULT ENTRY"));
+        return thing_do_grammar(
+                   dtype, add_stop, false,
+                   "UNAMED PORTAL VAULT ENTRY");
     default:
         return thing_do_grammar(dtype, add_stop, feat_is_trap(grid),
                    raw_feature_description(where) + covering_description);
index 37ee7c4..40acf9e 100644 (file)
@@ -2547,14 +2547,13 @@ static int _setup_temple_altars(CrawlHashTable &temple)
     for (unsigned int i = 0; i < god_list.size(); i++)
         _temple_altar_list.push_back((god_type) god_list[i].get_byte());
 
-    return ((int) god_list.size());
+    return (int)god_list.size();
 }
 
 struct map_component
 {
     int label;
 
-
     map_component()
     {
         min_equivalent = NULL;
@@ -3751,7 +3750,7 @@ const map_def *dgn_safe_place_map(const map_def *mdef,
         {
             const bool placed =
                 dgn_place_map(mdef, check_collision, make_no_exits, where);
-            return (placed? mdef : NULL);
+            return placed? mdef : NULL;
         }
         catch (map_load_exception &mload)
         {
index 8ad61c0..911ea95 100644 (file)
@@ -31,7 +31,7 @@ extern std::set<std::pair<std::string, level_id> > auto_unique_annotations;
 static bool _mon_needs_auto_exclude(const monster* mon, bool sleepy = false)
 {
     if (mons_is_stationary(mon))
-        return (!sleepy);
+        return !sleepy;
 
     // Auto exclusion only makes sense if the monster is still asleep or if it
     // is lurking (discovered mimics).
@@ -117,7 +117,7 @@ void remove_auto_exclude(const monster* mon, bool sleepy)
 
 static opacity_type _feat_opacity(dungeon_feature_type feat)
 {
-    return (feat_is_opaque(feat) ? OPC_OPAQUE : OPC_CLEAR);
+    return feat_is_opaque(feat) ? OPC_OPAQUE : OPC_CLEAR;
 }
 
 // A cell is considered clear unless the player knows it's
@@ -295,7 +295,7 @@ travel_exclude* exclude_set::get_exclude_root(const coord_def &p)
     exclude_set::iterator it = exclude_roots.find(p);
 
     if (it != exclude_roots.end())
-        return (&(it->second));
+        return &it->second;
 
     return NULL;
 }
index c73ebc5..2db7e4e 100644 (file)
@@ -101,15 +101,15 @@ static int _abil_degree(ability_type abil)
         return 1;
 
     case ABIL_NEMELEX_DRAW_ONE:
-        return (1 + random2(2));
+        return 1 + random2(2);
     case ABIL_NEMELEX_PEEK_TWO:
-        return (2 + random2(2));
+        return 2 + random2(2);
     case ABIL_NEMELEX_TRIPLE_DRAW:
-        return (3 + random2(3));
+        return 3 + random2(3);
     case ABIL_NEMELEX_DEAL_FOUR:
-        return (4 + random2(4));
+        return 4 + random2(4);
     case ABIL_NEMELEX_STACK_FIVE:
-        return (5 + random2(5));
+        return 5 + random2(5);
 
     case ABIL_YRED_RECALL_UNDEAD_SLAVES:
     case ABIL_MAKHLEB_MINOR_DESTRUCTION:
@@ -120,7 +120,7 @@ static int _abil_degree(ability_type abil)
     case ABIL_SIF_MUNA_CHANNEL_ENERGY:
     case ABIL_OKAWARU_HEROISM:
     case ABIL_JIYVA_CALL_JELLY:
-        return (1 + random2(3));
+        return 1 + random2(3);
 
     case ABIL_ZIN_RECITE:
         return 2;
@@ -128,7 +128,7 @@ static int _abil_degree(ability_type abil)
     case ABIL_TSO_DIVINE_SHIELD:
     case ABIL_KIKU_RECEIVE_CORPSES:
     case ABIL_BEOGH_SMITING:
-        return (2 + random2(2));
+        return 2 + random2(2);
     case ABIL_MAKHLEB_LESSER_SERVANT_OF_MAKHLEB:
     case ABIL_LUGONU_BEND_SPACE:
     case ABIL_FEDHAS_SUNLIGHT:
@@ -137,11 +137,11 @@ static int _abil_degree(ability_type abil)
     case ABIL_FEDHAS_SPAWN_SPORES:
     case ABIL_FEDHAS_EVOLUTION:
     case ABIL_CHEIBRIADOS_TIME_BEND:
-        return (2 + random2(3));
+        return 2 + random2(3);
     case ABIL_YRED_ANIMATE_REMAINS:
     case ABIL_YRED_ANIMATE_DEAD:
     case ABIL_YRED_DRAIN_LIFE:
-        return (2 + random2(4));
+        return 2 + random2(4);
 
     case ABIL_ZIN_IMPRISON:
     case ABIL_MAKHLEB_MAJOR_DESTRUCTION:
@@ -150,34 +150,34 @@ static int _abil_degree(ability_type abil)
     case ABIL_LUGONU_BANISH:
     case ABIL_JIYVA_SLIMIFY:
     case ABIL_CHEIBRIADOS_DISTORTION:
-        return (3 + random2(5));
+        return 3 + random2(5);
     case ABIL_TSO_CLEANSING_FLAME:
-        return (3 + random2(6));
+        return 3 + random2(6);
     case ABIL_OKAWARU_FINESSE:
-        return (3 + random2(7));
+        return 3 + random2(7);
 
     case ABIL_CHEIBRIADOS_SLOUCH:
-        return (4 + random2(4));
+        return 4 + random2(4);
     case ABIL_ELYVILON_PURIFICATION:
-        return (4 + random2(6));
+        return 4 + random2(6);
 
     case ABIL_LUGONU_CORRUPT:
     case ABIL_JIYVA_CURE_BAD_MUTATION:
     case ABIL_CHEIBRIADOS_TIME_STEP:
     case ABIL_KIKU_TORMENT:
-        return (5 + random2(5));
+        return 5 + random2(5);
     case ABIL_ZIN_SANCTUARY:
-        return (5 + random2(8));
+        return 5 + random2(8);
 
     case ABIL_MAKHLEB_GREATER_SERVANT_OF_MAKHLEB:
-        return (6 + random2(6));
+        return 6 + random2(6);
     case ABIL_ELYVILON_DIVINE_VIGOUR:
-        return (6 + random2(10));
+        return 6 + random2(10);
 
     case ABIL_TSO_SUMMON_DIVINE_WARRIOR:
     case ABIL_YRED_ENSLAVE_SOUL:
     case ABIL_LUGONU_ABYSS_EXIT:
-        return (8 + random2(10));
+        return 8 + random2(10);
 
     default:
         return 0;
index 8e856a6..628e011 100644 (file)
@@ -84,7 +84,7 @@ monster* player::get_fearmonger(const coord_def &target) const
 monster* player::get_any_fearmonger() const
 {
     if (!fearmongers.empty())
-        return (&menv[fearmongers[0]]);
+        return &menv[fearmongers[0]];
     else
         return NULL;
 }
index 15f625b..445c932 100644 (file)
@@ -16,7 +16,7 @@ const feature_def &get_feature_def(show_type object)
     // any instead, or the base feature if there are no items.
     if (object.cls == SH_MONSTER)
         object.cls = (object.item != SHOW_ITEM_NONE)? SH_ITEM : SH_FEATURE;
-    return (Features[object]);
+    return Features[object];
 }
 
 const feature_def &get_feature_def(dungeon_feature_type feat)
@@ -25,7 +25,7 @@ const feature_def &get_feature_def(dungeon_feature_type feat)
     show_type object;
     object.cls = SH_FEATURE;
     object.feat = feat;
-    return (Features[object]);
+    return Features[object];
 }
 
 static void _apply_feature_overrides()
index 615e40f..14f8b1a 100644 (file)
@@ -488,5 +488,5 @@ int player_weapon_str_weight()
 // define one of these.
 int player_weapon_dex_weight(void)
 {
-    return (10 - player_weapon_str_weight());
+    return 10 - player_weapon_str_weight();
 }
index 50611c3..17c2bbf 100644 (file)
@@ -384,8 +384,8 @@ void assert_read_safe_path(const std::string &path) throw (std::string)
 std::string canonicalise_file_separator(const std::string &path)
 {
     const std::string sep(1, FILE_SEPARATOR);
-    return (replace_all_of(replace_all_of(path, "/", sep),
-                           "\\", sep));
+    return replace_all_of(replace_all_of(path, "/", sep),
+                          "\\", sep);
 }
 
 static std::vector<std::string> _get_base_dirs()
index 8a872a4..a55ff20 100644 (file)
@@ -1702,8 +1702,8 @@ static int _apply_herbivore_nutrition_effects(int nutrition)
 
 static int _apply_gourmand_nutrition_effects(int nutrition, int gourmand)
 {
-    return (nutrition * (gourmand + GOURMAND_NUTRITION_BASE)
-                      / (GOURMAND_MAX + GOURMAND_NUTRITION_BASE));
+    return nutrition * (gourmand + GOURMAND_NUTRITION_BASE)
+                     / (GOURMAND_MAX + GOURMAND_NUTRITION_BASE);
 }
 
 static int _chunk_nutrition(int likes_chunks)
index dd424d4..26ef08d 100644 (file)
@@ -387,7 +387,7 @@ int formatted_string::find_last_colour() const
     {
         for (int i = ops.size() - 1; i >= 0; --i)
             if (ops[i].type == FSOP_COLOUR)
-                return (ops[i].x);
+                return ops[i].x;
     }
     return LIGHTGREY;
 }
index 2422de8..18b4597 100644 (file)
@@ -24,7 +24,7 @@ bool is_sanctuary(const coord_def& p)
 
 bool is_bloodcovered(const coord_def& p)
 {
-    return (testbits(env.pgrid(p), FPROP_BLOODY));
+    return testbits(env.pgrid(p), FPROP_BLOODY);
 }
 
 bool is_tide_immune(const coord_def &p)
index 838a8da..37ac959 100644 (file)
@@ -166,7 +166,7 @@ double degrees(const vector &v)
 vector degree_to_vector(double d)
 {
     double rad = d / 180.0 * M_PI;
-    return (vector(cos(rad), sin(rad)));
+    return vector(cos(rad), sin(rad));
 }
 
 }
index e12869f..bacf77b 100644 (file)
@@ -764,34 +764,34 @@ static bool _check_nemelex_sacrificing_item_type(const item_def& item)
     switch (item.base_type)
     {
     case OBJ_ARMOUR:
-        return (you.nemelex_sacrificing[NEM_GIFT_ESCAPE]);
+        return you.nemelex_sacrificing[NEM_GIFT_ESCAPE];
 
     case OBJ_WEAPONS:
     case OBJ_STAVES:
     case OBJ_RODS:
     case OBJ_MISSILES:
-        return (you.nemelex_sacrificing[NEM_GIFT_DESTRUCTION]);
+        return you.nemelex_sacrificing[NEM_GIFT_DESTRUCTION];
 
     case OBJ_CORPSES:
-        return (you.nemelex_sacrificing[NEM_GIFT_SUMMONING]);
+        return you.nemelex_sacrificing[NEM_GIFT_SUMMONING];
 
     case OBJ_POTIONS:
         if (is_blood_potion(item))
-            return (you.nemelex_sacrificing[NEM_GIFT_SUMMONING]);
-        return (you.nemelex_sacrificing[NEM_GIFT_WONDERS]);
+            return you.nemelex_sacrificing[NEM_GIFT_SUMMONING];
+        return you.nemelex_sacrificing[NEM_GIFT_WONDERS];
 
     case OBJ_FOOD:
         if (item.sub_type == FOOD_CHUNK)
-            return (you.nemelex_sacrificing[NEM_GIFT_SUMMONING]);
+            return you.nemelex_sacrificing[NEM_GIFT_SUMMONING];
     // else fall through
     case OBJ_WANDS:
     case OBJ_SCROLLS:
-        return (you.nemelex_sacrificing[NEM_GIFT_WONDERS]);
+        return you.nemelex_sacrificing[NEM_GIFT_WONDERS];
 
     case OBJ_JEWELLERY:
     case OBJ_BOOKS:
     case OBJ_MISCELLANY:
-        return (you.nemelex_sacrificing[NEM_GIFT_DUNGEONS]);
+        return you.nemelex_sacrificing[NEM_GIFT_DUNGEONS];
 
     default:
         return false;
index 3f4a0e3..144a2f5 100644 (file)
@@ -609,7 +609,7 @@ static time_t _parse_time(const std::string &st)
     struct tm  date;
 
     if (st.length() < 15)
-        return (static_cast<time_t>(0));
+        return static_cast<time_t>(0);
 
     date.tm_year  =   _val_char(st[0]) * 1000 + _val_char(st[1]) * 100
                     + _val_char(st[2]) *   10 + _val_char(st[3]) - 1900;
@@ -1091,7 +1091,7 @@ std::string scorefile_entry::make_oneline(const std::string &ml) const
             trim_string(s);
         }
     }
-    return (comma_separated_line(lines.begin(), lines.end(), " ", " "));
+    return comma_separated_line(lines.begin(), lines.end(), " ", " ");
 }
 
 std::string scorefile_entry::long_kill_message() const
index bf2f7c2..e46491f 100644 (file)
@@ -482,7 +482,7 @@ static tag_pref _str_to_tag_pref(const char *opt)
     for (int i = 0; i < TAGPREF_MAX; i++)
     {
         if (!strcasecmp(opt, tag_prefs[i]))
-            return ((tag_pref)i);
+            return (tag_pref)i;
     }
 
     return TAGPREF_ENEMY;
@@ -3244,7 +3244,7 @@ std::string game_options::resolve_include(
     // multiuser system. On multiuser systems assert_read_safe_path()
     // will throw an exception if it sees absolute paths.
     if (is_absolute_path(included_file))
-        return (file_exists(included_file)? included_file : "");
+        return file_exists(included_file)? included_file : "";
 
     if (!parent_file.empty())
     {
@@ -3344,7 +3344,7 @@ void game_options::report_error(const char* format, ...)
 
 static std::string check_string(const char *s)
 {
-    return (s? s : "");
+    return s? s : "";
 }
 
 void get_system_environment(void)
index fa75bc3..b7812e3 100644 (file)
@@ -180,7 +180,7 @@ void InvEntry::select(int qty)
 
 std::string InvEntry::get_filter_text() const
 {
-    return (filtering_item_prefix(*item) + " " + get_text());
+    return filtering_item_prefix(*item) + " " + get_text();
 }
 
 std::string InvEntry::get_text(bool need_cursor) const
@@ -458,8 +458,8 @@ static std::string _no_selectables_message(int item_selector)
         if (_has_melded_armour())
             return "Your armour is currently melded into you.";
         else if (_has_tran_unwearable_armour())
-            return ("You aren't carrying any armour you can wear in your "
-                   "current form.");
+            return "You aren't carrying any armour you can wear in your "
+                   "current form.";
         else
             return "You aren't carrying any armour.";
     }
@@ -468,8 +468,7 @@ static std::string _no_selectables_message(int item_selector)
     case OSEL_RECHARGE:
         return "You aren't carrying any rechargeable items.";
     case OSEL_ENCH_ARM:
-        return ("You aren't carrying any armour which can be enchanted "
-               "further.");
+        return "You aren't carrying any armour which can be enchanted further.";
     case OBJ_CORPSES:
     case OSEL_VAMP_EAT:
         return "You aren't carrying any corpses which you can drain.";
@@ -643,7 +642,7 @@ bool InvMenu::allow_easy_exit() const
 template <std::string (*proc)(const InvEntry *a)>
 static int compare_item_str(const InvEntry *a, const InvEntry *b)
 {
-    return (proc(a).compare(proc(b)));
+    return proc(a).compare(proc(b));
 }
 
 template <typename T, T (*proc)(const InvEntry *a)>
index ff94dfa..1564579 100644 (file)
@@ -993,7 +993,7 @@ static int _prompt_ring_to_remove(int new_ring)
 
     const int eqslot = (c == lslot || c == '<') ? EQ_LEFT_RING
                                                 : EQ_RIGHT_RING;
-    return (you.equip[eqslot]);
+    return you.equip[eqslot];
 }
 
 static int _prompt_ring_to_remove_octopode(int new_ring)
@@ -1051,7 +1051,7 @@ static int _prompt_ring_to_remove_octopode(int new_ring)
     else if (key_is_escape(c) || eqslot == EQ_NONE)
         return -2;
 
-    return (you.equip[eqslot]);
+    return you.equip[eqslot];
 }
 
 // Checks whether a to-be-worn or to-be-removed item affects
@@ -1187,7 +1187,7 @@ bool safe_to_remove(const item_def &item, bool quiet)
         {
             std::string fname = (feat == DNGN_LAVA ? "lava" : "deep water");
             std::string prompt = "Really remove this item over " + fname + "?";
-            return (yesno(prompt.c_str(), false, 'n'));
+            return yesno(prompt.c_str(), false, 'n');
         }
     }
 
@@ -2589,7 +2589,7 @@ static int _handle_enchant_armour(int item_slot, std::string *pre_msg)
         if (ac_change)
             you.redraw_armour_class = true;
 
-        return (result ? 1 : 0);
+        return result ? 1 : 0;
     }
     while (true);
 
index 9474183..86ec8fd 100644 (file)
@@ -391,38 +391,38 @@ const char* weapon_brand_name(const item_def& item, bool terse)
     switch (get_weapon_brand(item))
     {
     case SPWPN_NORMAL: return "";
-    case SPWPN_FLAMING: return ((terse) ? " (flame)" : " of flaming");
-    case SPWPN_FREEZING: return ((terse) ? " (freeze)" : " of freezing");
-    case SPWPN_HOLY_WRATH: return ((terse) ? " (holy)" : " of holy wrath");
-    case SPWPN_ELECTROCUTION: return ((terse) ? " (elec)":" of electrocution");
-    case SPWPN_ORC_SLAYING: return ((terse) ? " (slay orc)":" of orc slaying");
-    case SPWPN_DRAGON_SLAYING: return ((terse) ? " (slay drac)":" of dragon slaying");
-    case SPWPN_VENOM: return ((terse) ? " (venom)" : " of venom");
-    case SPWPN_PROTECTION: return ((terse) ? " (protect)" : " of protection");
-    case SPWPN_EVASION: return ((terse) ? " (evade)" : " of evasion");
-    case SPWPN_DRAINING: return ((terse) ? " (drain)" : " of draining");
-    case SPWPN_SPEED: return ((terse) ? " (speed)" : " of speed");
-    case SPWPN_PAIN: return ((terse) ? " (pain)" : " of pain");
-    case SPWPN_DISTORTION: return ((terse) ? " (distort)" : " of distortion");
-    case SPWPN_REACHING: return ((terse) ? " (reach)" : " of reaching");
-    case SPWPN_RETURNING: return ((terse) ? " (return)" : " of returning");
+    case SPWPN_FLAMING: return terse ? " (flame)" : " of flaming";
+    case SPWPN_FREEZING: return terse ? " (freeze)" : " of freezing";
+    case SPWPN_HOLY_WRATH: return terse ? " (holy)" : " of holy wrath";
+    case SPWPN_ELECTROCUTION: return terse ? " (elec)":" of electrocution";
+    case SPWPN_ORC_SLAYING: return terse ? " (slay orc)":" of orc slaying";
+    case SPWPN_DRAGON_SLAYING: return terse ? " (slay drac)":" of dragon slaying";
+    case SPWPN_VENOM: return terse ? " (venom)" : " of venom";
+    case SPWPN_PROTECTION: return terse ? " (protect)" : " of protection";
+    case SPWPN_EVASION: return terse ? " (evade)" : " of evasion";
+    case SPWPN_DRAINING: return terse ? " (drain)" : " of draining";
+    case SPWPN_SPEED: return terse ? " (speed)" : " of speed";
+    case SPWPN_PAIN: return terse ? " (pain)" : " of pain";
+    case SPWPN_DISTORTION: return terse ? " (distort)" : " of distortion";
+    case SPWPN_REACHING: return terse ? " (reach)" : " of reaching";
+    case SPWPN_RETURNING: return terse ? " (return)" : " of returning";
 
     case SPWPN_VAMPIRICISM:
-        return ((terse) ? " (vamp)" : ""); // non-terse already handled
+        return terse ? " (vamp)" : ""; // non-terse already handled
 
     case SPWPN_VORPAL:
         if (is_range_weapon(item))
-            return ((terse) ? " (velocity)" : " of velocity");
+            return terse ? " (velocity)" : " of velocity";
         else
         {
             switch (get_vorpal_type(item))
             {
-            case DVORP_CRUSHING: return ((terse) ? " (crush)" :" of crushing");
-            case DVORP_SLICING:  return ((terse) ? " (slice)" : " of slicing");
-            case DVORP_PIERCING: return ((terse) ? " (pierce)":" of piercing");
-            case DVORP_CHOPPING: return ((terse) ? " (chop)" : " of chopping");
-            case DVORP_SLASHING: return ((terse) ? " (slash)" :" of slashing");
-            case DVORP_STABBING: return ((terse) ? " (stab)" : " of stabbing");
+            case DVORP_CRUSHING: return terse ? " (crush)" :" of crushing";
+            case DVORP_SLICING:  return terse ? " (slice)" : " of slicing";
+            case DVORP_PIERCING: return terse ? " (pierce)":" of piercing";
+            case DVORP_CHOPPING: return terse ? " (chop)" : " of chopping";
+            case DVORP_SLASHING: return terse ? " (slash)" :" of slashing";
+            case DVORP_STABBING: return terse ? " (stab)" : " of stabbing";
             default:             return "";
             }
         }
@@ -430,13 +430,13 @@ const char* weapon_brand_name(const item_def& item, bool terse)
                                                       // handled elsewhere
 
     // ranged weapon brands
-    case SPWPN_FLAME: return ((terse) ? " (flame)" : " of flame");
-    case SPWPN_FROST: return ((terse) ? " (frost)" : " of frost");
-    case SPWPN_PENETRATION: return ((terse) ? " (penet)" : " of penetration");
-    case SPWPN_REAPING: return ((terse) ? " (reap)" : " of reaping");
+    case SPWPN_FLAME: return terse ? " (flame)" : " of flame";
+    case SPWPN_FROST: return terse ? " (frost)" : " of frost";
+    case SPWPN_PENETRATION: return terse ? " (penet)" : " of penetration";
+    case SPWPN_REAPING: return terse ? " (reap)" : " of reaping";
 
     // both ranged and non-ranged
-    case SPWPN_CHAOS: return ((terse) ? " (chaos)" : " of chaos");
+    case SPWPN_CHAOS: return terse ? " (chaos)" : " of chaos";
 
     // randart brands
     default: return "";
@@ -2723,7 +2723,7 @@ static bool _is_random_name_vowel(char let)
 static char _random_vowel(int seed)
 {
     static const char vowels[] = "aeiouaeiouaeiouy  ";
-    return (vowels[ seed % (sizeof(vowels) - 1) ]);
+    return vowels[ seed % (sizeof(vowels) - 1) ];
 }
 
 // Returns a random consonant with not quite equal probability.
@@ -2731,7 +2731,7 @@ static char _random_vowel(int seed)
 static char _random_cons(int seed)
 {
     static const char consonants[] = "bcdfghjklmnpqrstvwxzcdfghlmnrstlmnrst";
-    return (consonants[ seed % (sizeof(consonants) - 1) ]);
+    return consonants[ seed % (sizeof(consonants) - 1) ];
 }
 
 bool is_interesting_item(const item_def& item)
@@ -3570,5 +3570,5 @@ std::string get_corpse_name(const item_def &corpse, uint64_t *name_type)
     if (name_type != NULL)
         *name_type = corpse.props[CORPSE_NAME_TYPE_KEY].get_int64();
 
-    return (corpse.props[CORPSE_NAME_KEY].get_string());
+    return corpse.props[CORPSE_NAME_KEY].get_string();
 }
index d35338d..9c371d0 100644 (file)
@@ -1012,7 +1012,7 @@ special_missile_type get_ammo_brand(const item_def &item)
     if (item.base_type != OBJ_MISSILES || is_artefact(item))
         return SPMSL_NORMAL;
 
-    return (static_cast<special_missile_type>(item.special));
+    return static_cast<special_missile_type>(item.special);
 }
 
 special_armour_type get_armour_ego_type(const item_def &item)
@@ -1023,11 +1023,11 @@ special_armour_type get_armour_ego_type(const item_def &item)
 
     if (is_artefact(item))
     {
-        return (static_cast<special_armour_type>(
-                    artefact_wpn_property(item, ARTP_BRAND)));
+        return static_cast<special_armour_type>(
+                   artefact_wpn_property(item, ARTP_BRAND));
     }
 
-    return (static_cast<special_armour_type>(item.special));
+    return static_cast<special_armour_type>(item.special);
 }
 
 // Armour information and checking functions.
@@ -1139,12 +1139,12 @@ equipment_type get_armour_slot(const item_def &item)
 {
     ASSERT(item.base_type == OBJ_ARMOUR);
 
-    return (Armour_prop[ Armour_index[item.sub_type] ].slot);
+    return Armour_prop[ Armour_index[item.sub_type] ].slot;
 }
 
 equipment_type get_armour_slot(armour_type arm)
 {
-    return (Armour_prop[ Armour_index[arm] ].slot);
+    return Armour_prop[ Armour_index[arm] ].slot;
 }
 
 bool jewellery_is_amulet(const item_def &item)
@@ -1543,7 +1543,7 @@ hands_reqd_type hands_reqd(const item_def &item, size_type size)
     else if (ret < HANDS_ONE)
         ret = HANDS_ONE;
 
-    return (static_cast< hands_reqd_type >(ret));
+    return static_cast<hands_reqd_type>(ret);
 }
 
 bool is_whip_type(int wpn_type)
@@ -1742,19 +1742,19 @@ int weapon_str_weight(const item_def &wpn)
     ASSERT(is_weapon(wpn));
 
     if (wpn.base_type == OBJ_STAVES)
-        return (Weapon_prop[ Weapon_index[WPN_STAFF] ].str_weight);
+        return Weapon_prop[ Weapon_index[WPN_STAFF] ].str_weight;
 
     if (wpn.base_type == OBJ_RODS)
-        return (Weapon_prop[ Weapon_index[WPN_CLUB] ].str_weight);
+        return Weapon_prop[ Weapon_index[WPN_CLUB] ].str_weight;
 
-    return (Weapon_prop[ Weapon_index[wpn.sub_type] ].str_weight);
+    return Weapon_prop[ Weapon_index[wpn.sub_type] ].str_weight;
 }
 
 // Returns melee skill of item.
 skill_type weapon_skill(const item_def &item)
 {
     if (item.base_type == OBJ_WEAPONS && !is_range_weapon(item))
-        return (Weapon_prop[ Weapon_index[item.sub_type] ].skill);
+        return Weapon_prop[ Weapon_index[item.sub_type] ].skill;
     else if (item.base_type == OBJ_RODS)
         return SK_MACES_FLAILS; // Rods are short and stubby
     else if (item.base_type == OBJ_STAVES)
@@ -1779,7 +1779,7 @@ skill_type weapon_skill(object_class_type wclass, int wtype)
 skill_type range_skill(const item_def &item)
 {
     if (item.base_type == OBJ_WEAPONS && is_range_weapon(item))
-        return (Weapon_prop[ Weapon_index[item.sub_type] ].skill);
+        return Weapon_prop[ Weapon_index[item.sub_type] ].skill;
     else if (item.base_type == OBJ_MISSILES)
     {
         if (!has_launcher(item))
@@ -1988,7 +1988,7 @@ missile_type fires_ammo_type(const item_def &item)
     if (item.base_type != OBJ_WEAPONS)
         return MI_NONE;
 
-    return (Weapon_prop[Weapon_index[item.sub_type]].ammo);
+    return Weapon_prop[Weapon_index[item.sub_type]].ammo;
 }
 
 bool is_range_weapon(const item_def &item)
@@ -2024,7 +2024,7 @@ bool is_throwable(const actor *actor, const item_def &wpn, bool force)
     size_type bodysize = actor->body_size();
 
     if (wpn.base_type == OBJ_WEAPONS)
-        return (Weapon_prop[Weapon_index[wpn.sub_type]].throwable);
+        return Weapon_prop[Weapon_index[wpn.sub_type]].throwable;
     else if (wpn.base_type == OBJ_MISSILES)
     {
         if (!force)
@@ -2044,7 +2044,7 @@ bool is_throwable(const actor *actor, const item_def &wpn, bool force)
             }
         }
 
-        return (Missile_prop[Missile_index[wpn.sub_type]].throwable);
+        return Missile_prop[Missile_index[wpn.sub_type]].throwable;
     }
 
     return false;
@@ -2061,7 +2061,7 @@ launch_retval is_launched(const actor *actor, const item_def *launcher,
         return LRET_LAUNCHED;
     }
 
-    return (is_throwable(actor, missile) ? LRET_THROWN : LRET_FUMBLED);
+    return is_throwable(actor, missile) ? LRET_THROWN : LRET_FUMBLED;
 }
 
 //
@@ -2218,7 +2218,7 @@ int food_value(const item_def &item)
 int food_turns(const item_def &item)
 {
     ASSERT(item.defined() && item.base_type == OBJ_FOOD);
-    return (Food_prop[Food_index[item.sub_type]].turns);
+    return Food_prop[Food_index[item.sub_type]].turns;
 }
 
 bool can_cut_meat(const item_def &item)
@@ -2573,9 +2573,9 @@ int property(const item_def &item, int prop_type)
     {
     case OBJ_ARMOUR:
         if (prop_type == PARM_AC)
-            return (Armour_prop[ Armour_index[item.sub_type] ].ac);
+            return Armour_prop[ Armour_index[item.sub_type] ].ac;
         else if (prop_type == PARM_EVASION)
-            return (Armour_prop[ Armour_index[item.sub_type] ].ev);
+            return Armour_prop[ Armour_index[item.sub_type] ].ev;
         break;
 
     case OBJ_WEAPONS:
@@ -2594,18 +2594,18 @@ int property(const item_def &item, int prop_type)
             }
         }
         if (prop_type == PWPN_DAMAGE)
-            return (Weapon_prop[ Weapon_index[item.sub_type] ].dam);
+            return Weapon_prop[ Weapon_index[item.sub_type] ].dam;
         else if (prop_type == PWPN_HIT)
-            return (Weapon_prop[ Weapon_index[item.sub_type] ].hit);
+            return Weapon_prop[ Weapon_index[item.sub_type] ].hit;
         else if (prop_type == PWPN_SPEED)
-            return (Weapon_prop[ Weapon_index[item.sub_type] ].speed);
+            return Weapon_prop[ Weapon_index[item.sub_type] ].speed;
         else if (prop_type == PWPN_ACQ_WEIGHT)
-            return (Weapon_prop[ Weapon_index[item.sub_type] ].acquire_weight);
+            return Weapon_prop[ Weapon_index[item.sub_type] ].acquire_weight;
         break;
 
     case OBJ_MISSILES:
         if (prop_type == PWPN_DAMAGE)
-            return (Missile_prop[ Missile_index[item.sub_type] ].dam);
+            return Missile_prop[ Missile_index[item.sub_type] ].dam;
         break;
 
     case OBJ_STAVES:
@@ -2613,11 +2613,11 @@ int property(const item_def &item, int prop_type)
         weapon_sub = (item.base_type == OBJ_RODS) ? WPN_CLUB : WPN_STAFF;
 
         if (prop_type == PWPN_DAMAGE)
-            return (Weapon_prop[ Weapon_index[weapon_sub] ].dam);
+            return Weapon_prop[ Weapon_index[weapon_sub] ].dam;
         else if (prop_type == PWPN_HIT)
-            return (Weapon_prop[ Weapon_index[weapon_sub] ].hit);
+            return Weapon_prop[ Weapon_index[weapon_sub] ].hit;
         else if (prop_type == PWPN_SPEED)
-            return (Weapon_prop[ Weapon_index[weapon_sub] ].speed);
+            return Weapon_prop[ Weapon_index[weapon_sub] ].speed;
         break;
 
     default:
@@ -2924,7 +2924,7 @@ std::string item_base_name(object_class_type type, int sub_type)
     case OBJ_ARMOUR:
         return Armour_prop[Armour_index[sub_type]].name;
     case OBJ_JEWELLERY:
-        return (jewellery_is_amulet(sub_type) ? "amulet" : "ring");
+        return jewellery_is_amulet(sub_type) ? "amulet" : "ring";
     default:
         return "";
     }
@@ -2932,7 +2932,7 @@ std::string item_base_name(object_class_type type, int sub_type)
 
 std::string food_type_name(int sub_type)
 {
-    return (Food_prop[Food_index[sub_type]].name);
+    return Food_prop[Food_index[sub_type]].name;
 }
 
 const char* weapon_base_name(uint8_t subtype)
index 2682cc4..d267f74 100644 (file)
@@ -2979,7 +2979,7 @@ item_def *find_floor_item(object_class_type cls, int sub_type)
                     && si->base_type == cls && si->sub_type == sub_type
                     && !(si->flags & ISFLAG_MIMIC))
                 {
-                    return (& (*si));
+                    return &*si;
                 }
 
     return NULL;
@@ -3019,7 +3019,7 @@ int get_max_subtype(object_class_type base_type)
 
     ASSERT(base_type < NUM_OBJECT_CLASSES);
 
-    return (max_subtype[base_type]);
+    return max_subtype[base_type];
 }
 
 equipment_type item_equip_slot(const item_def& item)
@@ -3029,7 +3029,7 @@ equipment_type item_equip_slot(const item_def& item)
 
     for (int i = 0; i < NUM_EQUIP; i++)
         if (item.link == you.equip[i])
-            return (static_cast<equipment_type>(i));
+            return static_cast<equipment_type>(i);
 
     return EQ_NONE;
 }
@@ -3144,7 +3144,7 @@ monster* item_def::holding_monster() const
     if (invalid_monster_index(midx))
         return NULL;
 
-    return (&menv[midx]);
+    return &menv[midx];
 }
 
 void item_def::set_holding_monster(int midx)
@@ -3734,7 +3734,7 @@ bool get_item_by_name(item_def *item, char* specs,
 coord_def orb_position()
 {
     item_def* orb = find_floor_item(OBJ_ORBS, ORB_ZOT);
-    return (orb ? orb->pos: coord_def());
+    return orb ? orb->pos: coord_def();
 }
 
 void move_items(const coord_def r, const coord_def p)
index c93d612..01895f4 100644 (file)
@@ -33,7 +33,7 @@ const char *get_job_abbrev(int which_job)
     COMPILE_CHECK(ARRAYSZ(Job_Abbrev_List) == NUM_JOBS);
     ASSERT(which_job >= 0 && which_job < NUM_JOBS);
 
-    return (Job_Abbrev_List[which_job]);
+    return Job_Abbrev_List[which_job];
 }
 
 job_type get_job_by_abbrev(const char *abbrev)
@@ -59,7 +59,7 @@ const char *get_job_name(int which_job)
     COMPILE_CHECK(ARRAYSZ(Job_Name_List) == NUM_JOBS);
     ASSERT(which_job >= 0 && which_job < NUM_JOBS);
 
-    return (Job_Name_List[which_job]);
+    return Job_Name_List[which_job];
 }
 
 job_type get_job_by_name(const char *name)
index 3760011..ec863b4 100644 (file)
@@ -56,7 +56,7 @@ KillMaster::~KillMaster()
 const char *KillMaster::category_name(kill_category kc) const
 {
     if (kc >= KC_YOU && kc < KC_NCATEGORIES)
-        return (kill_category_names[kc]);
+        return kill_category_names[kc];
     return NULL;
 }
 
index 10558f7..73b4743 100644 (file)
@@ -992,14 +992,14 @@ static int _crawl_god_speaks(lua_State *ls)
     if (!god_name)
     {
         std::string err = "god_speaks requires a god!";
-        return (luaL_argerror(ls, 1, err.c_str()));
+        return luaL_argerror(ls, 1, err.c_str());
     }
 
     god_type god = str_to_god(god_name);
     if (god == GOD_NO_GOD)
     {
         std::string err = make_stringf("'%s' matches no god.", god_name);
-        return (luaL_argerror(ls, 1, err.c_str()));
+        return luaL_argerror(ls, 1, err.c_str());
     }
 
     const char *message = luaL_checkstring(ls, 2);
index 657d23d..8ed586c 100644 (file)
@@ -221,14 +221,14 @@ LUAFN(debug_god_wrath)
     if (!god_name)
     {
         std::string err = "god_wrath requires a god!";
-        return (luaL_argerror(ls, 1, err.c_str()));
+        return luaL_argerror(ls, 1, err.c_str());
     }
 
     god_type god = strcmp(god_name, "random") ? str_to_god(god_name) : GOD_RANDOM;
     if (god == GOD_NO_GOD)
     {
         std::string err = make_stringf("'%s' matches no god.", god_name);
-        return (luaL_argerror(ls, 1, err.c_str()));
+        return luaL_argerror(ls, 1, err.c_str());
     }
 
     bool no_bonus = lua_toboolean(ls, 2);
index 4677676..d877a9f 100644 (file)
@@ -123,7 +123,7 @@ static int dgn_dgn_event(lua_State *ls)
 static dgn_event_type dgn_param_to_event_type(lua_State *ls, int n)
 {
     if (lua_isnumber(ls, n))
-        return (static_cast<dgn_event_type>(luaL_checkint(ls, n)));
+        return static_cast<dgn_event_type>(luaL_checkint(ls, n));
     else if (lua_isstring(ls, n))
         return dgn_event_type_by_name(lua_tostring(ls, n));
     else
index eca394e..a46f264 100644 (file)
@@ -1087,7 +1087,7 @@ static int item_get(lua_State *ls)
 
     for (unsigned i = 0; i < ARRAYSZ(item_attrs); ++i)
         if (!strcmp(attr, item_attrs[i].attribute))
-            return (item_attrs[i].accessor(ls, iw, attr));
+            return item_attrs[i].accessor(ls, iw, attr);
 
     return 0;
 }
index 64f4b09..e730254 100644 (file)
@@ -44,7 +44,7 @@ static char* mapgrd_glyph(lua_State *ls, int &col, int &row)
         return NULL;
 
     coord_def mc(col, row);
-    return (&lines(mc));
+    return &lines(mc);
 }
 
 static int mapgrd_col_get(lua_State *ls)
index 974afc0..0864c3e 100644 (file)
@@ -533,7 +533,7 @@ static int monster_get(lua_State *ls)
 
     for (unsigned i = 0; i < ARRAYSZ(mons_attrs); ++i)
         if (!strcmp(attr, mons_attrs[i].attribute))
-            return (mons_attrs[i].accessor(ls, mons, attr));
+            return mons_attrs[i].accessor(ls, mons, attr);
 
     return 0;
 }
@@ -552,7 +552,7 @@ static const char *_monster_behaviour_names[] = {
 static const char* _behaviour_name(beh_type beh)
 {
     if (0 <= beh && beh < NUM_BEHAVIOURS)
-        return (_monster_behaviour_names[beh]);
+        return _monster_behaviour_names[beh];
     else
         return "invalid";
 }
index 4eee112..12e05e2 100644 (file)
@@ -95,7 +95,7 @@ static int option_get(lua_State *ls)
     const option_handler *oh = get_handler(opt);
     if (oh)
 #ifdef DEBUG_GLOBALS
-        return (oh->handler(ls, opt, (char*)real_Options+(intptr_t)oh->data, true));
+        return oh->handler(ls, opt, (char*)real_Options+(intptr_t)oh->data, true);
 #else
         return oh->handler(ls, opt, oh->data, true);
 #endif
index fa01b47..23dd746 100644 (file)
@@ -289,7 +289,7 @@ LUAFN(you_mutation)
     }
 
     std::string err = make_stringf("No such mutation: '%s'.", mutname.c_str());
-    return (luaL_argerror(ls, 1, err.c_str()));
+    return luaL_argerror(ls, 1, err.c_str());
 }
 
 LUAFN(you_is_level_on_stack)
@@ -558,7 +558,7 @@ LUAFN(you_in_branch)
                     "'%s' matches both branch '%s' and '%s'",
                     name, branches[br].abbrevname,
                     branches[i].abbrevname);
-                return (luaL_argerror(ls, 1, err.c_str()));
+                return luaL_argerror(ls, 1, err.c_str());
             }
             br = i;
         }
@@ -567,7 +567,7 @@ LUAFN(you_in_branch)
     if (br == NUM_BRANCHES)
     {
         std::string err = make_stringf("'%s' matches no branches.", name);
-        return (luaL_argerror(ls, 1, err.c_str()));
+        return luaL_argerror(ls, 1, err.c_str());
     }
 
     bool in_branch = (br == you.where_are_you);
index c983bf9..348003e 100644 (file)
@@ -170,7 +170,7 @@ void cgotoxy(int x, int y, GotoRegion region)
 coord_def cgetpos(GotoRegion region)
 {
     ASSERT(region == get_cursor_region());
-    return (coord_def(wherex(), wherey()));
+    return coord_def(wherex(), wherey());
 }
 
 GotoRegion get_cursor_region()
index 20fa1fa..367e0e1 100644 (file)
@@ -87,7 +87,7 @@ static unsigned int convert_to_curses_attr(int chattr)
 
 static inline short macro_colour(short col)
 {
-    return (Options.colour[ col ]);
+    return Options.colour[ col ];
 }
 
 // Translate DOS colors to curses.
index 08187e2..fb7ae90 100644 (file)
@@ -92,7 +92,7 @@ description_level_type description_type_by_name(const char *desc)
 
 static std::string _number_to_string(unsigned number, bool in_words)
 {
-    return (in_words? number_in_words(number) : make_stringf("%u", number));
+    return in_words? number_in_words(number) : make_stringf("%u", number);
 }
 
 std::string apply_description(description_level_type desc,
index aefb882..712bd82 100644 (file)
@@ -28,9 +28,9 @@ static losfield_t* _lookup_globallos(const coord_def& p, const coord_def& q)
         return NULL;
     // p < q iff p.x < q.x || p.x == q.x && p.y < q.y
     if (diff < coord_def(0, 0))
-        return (&globallos[q.x][q.y][-diff.x + o_half_x][-diff.y + o_half_y]);
+        return &globallos[q.x][q.y][-diff.x + o_half_x][-diff.y + o_half_y];
     else
-        return (&globallos[p.x][p.y][ diff.x + o_half_x][ diff.y + o_half_y]);
+        return &globallos[p.x][p.y][ diff.x + o_half_x][ diff.y + o_half_y];
 }
 
 static void _save_los(los_def* los, los_type l)
index 4c11e3d..6e62fe8 100644 (file)
@@ -215,7 +215,7 @@ static std::string get_macro_file()
     }
 
 #if defined(DGL_MACRO_ABSOLUTE_PATH)
-    return (dir.empty()? "macro.txt" : dir);
+    return dir.empty()? "macro.txt" : dir;
 #endif
 
     check_mkdir("Macro directory", &dir, true);
@@ -1272,13 +1272,13 @@ int command_to_key(command_type cmd)
     KeymapContext context = _context_for_command(cmd);
 
     if (context == KMC_NONE)
-        return ('\0');
+        return '\0';
 
     cmd_to_key_map           &cmd_map = _cmds_to_keys[context];
     cmd_to_key_map::iterator it       = cmd_map.find(cmd);
 
     if (it == cmd_map.end())
-        return ('\0');
+        return '\0';
 
     return it->second;
 }
index 0d7addb..c9173f1 100644 (file)
@@ -159,7 +159,7 @@ public:
 
     virtual int highlight_colour() const
     {
-        return (menu_colour(get_text(), "", tag));
+        return menu_colour(get_text(), "", tag);
     }
 
     virtual bool selected() const
index 79946e3..d5b6b9d 100644 (file)
@@ -79,8 +79,8 @@ inline bool give_bad_mutation(const std::string &reason,
                               bool failMsg = true,
                               bool force_mutation = false)
 {
-    return (mutate(RANDOM_BAD_MUTATION, reason, failMsg, force_mutation,
-                   false, false, false));
+    return mutate(RANDOM_BAD_MUTATION, reason, failMsg, force_mutation,
+                  false, false, false);
 }
 
 void display_mutations();
index a73eb50..c7ea14b 100644 (file)
@@ -19,7 +19,7 @@ T* random_choose_weighted(std::vector<std::pair<T, int> >& choices)
     {
         sum += choices[i].second;
         if (sum > r)
-            return (&choices[i].first);
+            return &choices[i].first;
     }
     return NULL;
 }