From e1bf90c612314a0e72ee737bca3640a74bebdab5 Mon Sep 17 00:00:00 2001 From: Marko Lindqvist Date: Sat, 8 Jul 2023 22:01:16 +0300 Subject: [PATCH 21/21] Unify "Traderoute" to "Trade Route" The only things not changed: - Ruleset format (i.e. requirement and effect names) - File names See osdn #45913 Signed-off-by: Marko Lindqvist --- ai/default/daicity.c | 42 +++++++-------- ai/default/daidomestic.c | 6 +-- ai/default/daieffects.c | 2 +- client/citydlg_common.c | 2 +- client/control.c | 4 +- client/gui-qt/menu.cpp | 12 ++--- client/gui-qt/menu.h | 4 +- client/gui-qt/shortcuts.cpp | 2 +- client/helpdata.c | 8 +-- client/packhand.c | 7 +-- common/aicore/caravan.c | 2 +- common/aicore/cm.c | 6 +-- common/city.c | 2 +- common/effects.h | 3 +- common/metaknowledge.c | 20 +++---- common/metaknowledge.h | 2 +- common/networking/packets.def | 8 +-- common/reqtext.c | 76 +++++++++++++------------- common/requirements.c | 86 +++++++++++++++--------------- common/requirements.h | 3 +- common/traderoutes.c | 20 +++---- common/traderoutes.h | 14 ++--- data/alien/README.alien | 2 +- data/alien/actions.ruleset | 2 +- data/alien/game.ruleset | 6 +-- data/civ1/actions.ruleset | 2 +- data/civ1/game.ruleset | 12 ++--- data/civ2/actions.ruleset | 2 +- data/civ2/game.ruleset | 8 +-- data/civ2civ3/actions.ruleset | 2 +- data/civ2civ3/game.ruleset | 6 +-- data/classic/actions.ruleset | 2 +- data/classic/game.ruleset | 6 +-- data/goldkeep/README.goldkeep | 2 +- data/goldkeep/actions.ruleset | 2 +- data/goldkeep/game.ruleset | 6 +-- data/granularity/game.ruleset | 6 +-- data/multiplayer/game.ruleset | 6 +-- data/ruledit/comments-3.3.txt | 6 +-- data/sandbox/actions.ruleset | 2 +- data/sandbox/game.ruleset | 6 +-- data/stub/game.ruleset | 6 +-- data/webperimental/actions.ruleset | 2 +- data/webperimental/game.ruleset | 6 +-- doc/README.effects | 6 +-- server/citytools.c | 56 +++++++++---------- server/citytools.h | 12 ++--- server/cityturn.c | 6 +-- server/diplomats.c | 12 ++--- server/rssanity.c | 2 +- server/ruleset.c | 10 ++-- server/settings.c | 2 +- server/unithand.c | 4 +- tools/ruleutil/rulesave.c | 4 +- 54 files changed, 274 insertions(+), 271 deletions(-) diff --git a/ai/default/daicity.c b/ai/default/daicity.c index cda8f98f29..0a8ad0a73a 100644 --- a/ai/default/daicity.c +++ b/ai/default/daicity.c @@ -85,28 +85,28 @@ /* Iterate over cities within a certain range around a given city * (city_here) that exist within a given city list. */ -#define city_range_iterate(city_here, list, range, city) \ -{ \ - city_list_iterate(list, city) { \ - if (range == REQ_RANGE_PLAYER \ - || range == REQ_RANGE_TEAM \ +#define city_range_iterate(city_here, list, range, city) \ +{ \ + city_list_iterate(list, city) { \ + if (range == REQ_RANGE_PLAYER \ + || range == REQ_RANGE_TEAM \ || range == REQ_RANGE_ALLIANCE \ - || (range == REQ_RANGE_TRADEROUTE \ + || (range == REQ_RANGE_TRADE_ROUTE \ && (city == city_here \ || have_cities_trade_route(city, city_here))) \ - || ((range == REQ_RANGE_CITY || range == REQ_RANGE_LOCAL) \ - && city == city_here) \ - || (range == REQ_RANGE_CONTINENT \ - && tile_continent(city->tile) == \ - tile_continent(city_here->tile))) { - -#define city_range_iterate_end \ - } \ - } city_list_iterate_end; \ + || ((range == REQ_RANGE_CITY || range == REQ_RANGE_LOCAL) \ + && city == city_here) \ + || (range == REQ_RANGE_CONTINENT \ + && tile_continent(city->tile) == \ + tile_continent(city_here->tile))) { + +#define city_range_iterate_end \ + } \ + } city_list_iterate_end; \ } -#define CITY_EMERGENCY(pcity) \ - (pcity->surplus[O_SHIELD] < 0 || city_unhappy(pcity) \ +#define CITY_EMERGENCY(pcity) \ + (pcity->surplus[O_SHIELD] < 0 || city_unhappy(pcity) \ || pcity->food_stock + pcity->surplus[O_FOOD] < 0) static void dai_city_sell_noncritical(struct city *pcity, bool redundant_only); @@ -278,7 +278,7 @@ static void dai_city_choose_build(struct ai_type *ait, struct player *pplayer, if (city_data->choice.want == 0) { /* Fallbacks do happen with techlevel 0, which is now default. -- Per */ CITY_LOG(LOG_WANT, pcity, "Falling back - didn't want to build soldiers," - " workers, caravans, settlers, or buildings!"); + " workers, caravans, settlers, or buildings!"); city_data->choice.want = 1; if (best_role_unit(pcity, action_id_get_role(ACTION_TRADE_ROUTE))) { city_data->choice.value.utype @@ -322,8 +322,8 @@ static void dai_city_choose_build(struct ai_type *ait, struct player *pplayer, #endif /* ADV_CHOICE_TRACK */ CITY_LOG(LOG_DEBUG, pcity, "wants %s with desire " ADV_WANT_PRINTF ".", - dai_choice_rule_name(&city_data->choice), - city_data->choice.want); + dai_choice_rule_name(&city_data->choice), + city_data->choice.want); switch (city_data->choice.type) { case CT_CIVILIAN: @@ -1659,7 +1659,7 @@ static void adjust_improvement_wants_by_effects(struct ai_type *ait, } /* All the trade partners and the city being considered. */ - cities[REQ_RANGE_TRADEROUTE] = city_num_trade_routes(pcity)+1; + cities[REQ_RANGE_TRADE_ROUTE] = city_num_trade_routes(pcity) + 1; cities[REQ_RANGE_CITY] = cities[REQ_RANGE_LOCAL] = 1; diff --git a/ai/default/daidomestic.c b/ai/default/daidomestic.c index 8be340cbe3..2f401ae916 100644 --- a/ai/default/daidomestic.c +++ b/ai/default/daidomestic.c @@ -253,7 +253,7 @@ static void dai_choose_trade_route(struct ai_type *ait, struct city *pcity, /* Look for proper destination city for trade. */ if (trade_route_type_trade_pct(TRT_NATIONAL) > 0 || trade_route_type_trade_pct(TRT_NATIONAL_IC) > 0) { - /* National traderoutes have value */ + /* National trade routes have value */ city_list_iterate(pplayer->cities, acity) { if (can_cities_trade(pcity, acity)) { if (tile_continent(acity->tile) != continent) { @@ -278,7 +278,7 @@ static void dai_choose_trade_route(struct ai_type *ait, struct city *pcity, } /* FIXME: This check should consider more about relative - * income from different traderoute types. This works just + * income from different trade route types. This works just * with more typical ruleset setups. */ if (prefer_different_cont && !dest_city_nat_different_cont) { if (trade_route_type_trade_pct(TRT_IN_IC) > @@ -328,7 +328,7 @@ static void dai_choose_trade_route(struct ai_type *ait, struct city *pcity, } trade_routes = city_num_trade_routes(pcity); - /* Count also caravans enroute to establish traderoutes */ + /* Count also caravans enroute to establish trade routes */ caravan_units = 0; unit_list_iterate(pcity->units_supported, punit) { if (unit_can_do_action(punit, ACTION_TRADE_ROUTE)) { diff --git a/ai/default/daieffects.c b/ai/default/daieffects.c index 9b7ac21b44..09336d5ff1 100644 --- a/ai/default/daieffects.c +++ b/ai/default/daieffects.c @@ -684,7 +684,7 @@ adv_want dai_effect_value(struct player *pplayer, } } break; - case EFT_TRADEROUTE_PCT: + case EFT_TRADE_ROUTE_PCT: { int trade = 0; diff --git a/client/citydlg_common.c b/client/citydlg_common.c index e1b68a272c..f17f07a9cf 100644 --- a/client/citydlg_common.c +++ b/client/citydlg_common.c @@ -779,7 +779,7 @@ void get_city_dialog_output_text(const struct city *pcity, const char *name = trade_city ? city_name_get(trade_city) : _("(unknown)"); int value = proute->value - * (100 + get_city_bonus(pcity, EFT_TRADEROUTE_PCT)) / 100; + * (100 + get_city_bonus(pcity, EFT_TRADE_ROUTE_PCT)) / 100; switch (proute->dir) { case RDIR_BIDIRECTIONAL: diff --git a/client/control.c b/client/control.c index 326cad26c3..38c6c7052f 100644 --- a/client/control.c +++ b/client/control.c @@ -2183,7 +2183,7 @@ void request_unit_unload(struct unit *pcargo) } /**********************************************************************//** - Send request to do caravan action - establishing traderoute or + Send request to do caravan action - establishing trade route or helping in wonder building - to server. **************************************************************************/ void request_unit_caravan_action(struct unit *punit, action_id action) @@ -3463,7 +3463,7 @@ void key_unit_teleport(void) } /**********************************************************************//** - Handle user 'establish traderoute' input + Handle user 'establish trade route' input **************************************************************************/ void key_unit_trade_route(void) { diff --git a/client/gui-qt/menu.cpp b/client/gui-qt/menu.cpp index 824d7f5dbe..1d04aa3c22 100644 --- a/client/gui-qt/menu.cpp +++ b/client/gui-qt/menu.cpp @@ -1122,12 +1122,12 @@ void mr_menu::setup_menus() act->setCheckable(true); act->setChecked(gui_options.draw_city_buycost); connect(act, &QAction::triggered, this, &mr_menu::slot_city_buycost); - act = main_menu->addAction(_("City Traderoutes")); + act = main_menu->addAction(_("City Trade Routes")); act->setCheckable(true); act->setChecked(gui_options.draw_city_trade_routes); act->setShortcut(QKeySequence(shortcut_to_string( fc_shortcuts::sc()->get_shortcut(SC_TRADE_ROUTES)))); - connect(act, &QAction::triggered, this, &mr_menu::slot_city_traderoutes); + connect(act, &QAction::triggered, this, &mr_menu::slot_city_trade_routes); act = main_menu->addAction(_("Unit Stack Size")); act->setCheckable(true); act->setChecked(gui_options.draw_unit_stack_size); @@ -1369,7 +1369,7 @@ void mr_menu::setup_menus() act = main_menu->addAction(QString(action_id_name_translation(ACTION_TRADE_ROUTE)) .replace("&", "&&")); act->setShortcut(QKeySequence(tr("r"))); - menu_list.insert(ORDER_TRADEROUTE, act); + menu_list.insert(ORDER_TRADE_ROUTE, act); connect(act, &QAction::triggered, this, &mr_menu::slot_build_road); multiplayer_menu = this->addMenu(_("Multiplayer")); @@ -2475,7 +2475,7 @@ void mr_menu::menus_sensitive() } break; - case ORDER_TRADEROUTE: + case ORDER_TRADE_ROUTE: i.value()->setText( QString(action_id_name_translation(ACTION_TRADE_ROUTE)) .replace("&", "&&")); @@ -3312,9 +3312,9 @@ void mr_menu::slot_city_production() } /**********************************************************************//** - Action "SHOW CITY TRADEROUTES" + Action "SHOW CITY TRADE ROUTES" **************************************************************************/ -void mr_menu::slot_city_traderoutes() +void mr_menu::slot_city_trade_routes() { key_city_trade_routes_toggle(); } diff --git a/client/gui-qt/menu.h b/client/gui-qt/menu.h index 716b82ffab..e0d073471d 100644 --- a/client/gui-qt/menu.h +++ b/client/gui-qt/menu.h @@ -69,7 +69,7 @@ enum munit { AIRLIFT, BUILD_WONDER, AUTOTRADEROUTE, - ORDER_TRADEROUTE, + ORDER_TRADE_ROUTE, ORDER_DIPLOMAT_DLG, UPGRADE, NOT_4_OBS, @@ -367,7 +367,7 @@ private slots: void slot_city_growth(); void slot_city_production(); void slot_city_buycost(); - void slot_city_traderoutes(); + void slot_city_trade_routes(); void slot_stack_size(); void slot_city_names(); void zoom_in(); diff --git a/client/gui-qt/shortcuts.cpp b/client/gui-qt/shortcuts.cpp index 94b0b5c959..453ea43c42 100644 --- a/client/gui-qt/shortcuts.cpp +++ b/client/gui-qt/shortcuts.cpp @@ -94,7 +94,7 @@ fc_shortcut default_shortcuts[] = { {SC_SHOW_UNITS, Qt::Key_Space, Qt::AllButtons, Qt::ControlModifier, _("Units selection (for tile under mouse position)") }, {SC_TRADE_ROUTES, Qt::Key_D, Qt::AllButtons, Qt::ControlModifier, - _("City Traderoutes") }, + _("City Trade Routes") }, {SC_CITY_PROD, Qt::Key_P, Qt::AllButtons, Qt::ControlModifier, _("City Production Levels") }, {SC_CITY_NAMES, Qt::Key_N, Qt::AllButtons, Qt::ControlModifier, diff --git a/client/helpdata.c b/client/helpdata.c index 1fb3b381e4..11ff6ae1eb 100644 --- a/client/helpdata.c +++ b/client/helpdata.c @@ -1428,7 +1428,7 @@ char *helptext_building(char *buf, size_t bufsz, struct player *pplayer, "action \'%s\'.\n"), BULLET, action_id_name_translation(act)); break; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: /* At least one action enabler needed the building in its target * requirements. */ cat_snprintf(buf, bufsz, @@ -1572,7 +1572,7 @@ char *helptext_building(char *buf, size_t bufsz, struct player *pplayer, BULLET, action_id_name_translation(act)); break; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: cat_snprintf(buf, bufsz, /* TRANS: Incite City */ _("%s Makes it impossible to do the action \'%s\' to " @@ -4108,10 +4108,10 @@ void helptext_goods(char *buf, size_t bufsz, struct player *pplayer, if (pgood->onetime_pct == 0) { cat_snprintf(buf, bufsz, - _("There's no bonuses paid when traderoute is established.\n\n")); + _("There's no bonuses paid when trade route gets established.\n\n")); } else if (pgood->onetime_pct != 100) { cat_snprintf(buf, bufsz, - _("When traderoute is established, %d%% of the normal bonus is paid.\n"), + _("When trade route gets established, %d%% of the normal bonus is paid.\n"), pgood->onetime_pct); } cat_snprintf(buf, bufsz, _("Sending city enjoys %d%% income from the route.\n"), diff --git a/client/packhand.c b/client/packhand.c index 234c3ba30c..db6ce7bfa2 100644 --- a/client/packhand.c +++ b/client/packhand.c @@ -731,7 +731,7 @@ void handle_city_info(const struct packet_city_info *packet) name_changed = (0 != strncmp(packet->name, pcity->name, MAX_LEN_CITYNAME)); - while (trade_route_list_size(pcity->routes) > packet->traderoute_count) { + while (trade_route_list_size(pcity->routes) > packet->trade_route_count) { struct trade_route *proute = trade_route_list_get(pcity->routes, -1); trade_route_list_remove(pcity->routes, proute); @@ -1109,9 +1109,10 @@ static void city_packet_common(struct city *pcity, struct tile *pcenter, } /************************************************************************//** - A traderoute-info packet contains information about one end of a traderoute + A trade route-info packet contains information about one end + of a trade route ****************************************************************************/ -void handle_traderoute_info(const struct packet_traderoute_info *packet) +void handle_trade_route_info(const struct packet_trade_route_info *packet) { struct city *pcity = game_city_by_number(packet->city); struct trade_route *proute; diff --git a/common/aicore/caravan.c b/common/aicore/caravan.c index 9d63def718..5cb6f42bfc 100644 --- a/common/aicore/caravan.c +++ b/common/aicore/caravan.c @@ -349,7 +349,7 @@ static double trade_benefit(const struct player *caravan_owner, return 0; } if (max_trade_routes(src) <= 0 || max_trade_routes(dest) <= 0) { - /* Can't create new traderoutes even by replacing old ones if + /* Can't create new trade routes even by replacing old ones if * there's no slots at all. */ return 0; } diff --git a/common/aicore/cm.c b/common/aicore/cm.c index 4f3d361d96..071e185782 100644 --- a/common/aicore/cm.c +++ b/common/aicore/cm.c @@ -1623,9 +1623,9 @@ static void compute_max_stats_heuristic(const struct cm_state *state, } - /* we found the basic production, however, bonus, taxes, - free production, tithes, traderoutes are missing - we add free production, and have the city.c code do the rest */ + /* We found the basic production, however, bonus, taxes, + * free production, tithes, trade routes are missing. + * We add free production, and have the city.c code do the rest */ struct city *pcity = state->pcity; struct tile *pcenter = city_tile(pcity); diff --git a/common/city.c b/common/city.c index e122a992e4..d94f690ebb 100644 --- a/common/city.c +++ b/common/city.c @@ -2953,7 +2953,7 @@ inline void set_city_production(struct city *pcity) trade_base_between_cities(pcity, game_city_by_number(proute->partner)); proute->value = trade_from_route(pcity, proute, value); pcity->prod[O_TRADE] += proute->value - * (100 + get_city_bonus(pcity, EFT_TRADEROUTE_PCT)) / 100; + * (100 + get_city_bonus(pcity, EFT_TRADE_ROUTE_PCT)) / 100; } else { proute->value = 0; } diff --git a/common/effects.h b/common/effects.h index 58c3f0d8cf..686140ce4f 100644 --- a/common/effects.h +++ b/common/effects.h @@ -144,7 +144,8 @@ struct multiplier; /* Interacts with UTYF_BADWALLATTACKER */ #define SPECENUM_VALUE48 EFT_DEFEND_BONUS #define SPECENUM_VALUE48NAME "Defend_Bonus" -#define SPECENUM_VALUE49 EFT_TRADEROUTE_PCT +#define SPECENUM_VALUE49 EFT_TRADE_ROUTE_PCT +/* FIXME: -> "Trade_Route_Pct" */ #define SPECENUM_VALUE49NAME "Traderoute_Pct" #define SPECENUM_VALUE50 EFT_GAIN_AI_LOVE #define SPECENUM_VALUE50NAME "Gain_AI_Love" diff --git a/common/metaknowledge.c b/common/metaknowledge.c index 4560a4fb60..428c1652e7 100644 --- a/common/metaknowledge.c +++ b/common/metaknowledge.c @@ -96,8 +96,8 @@ static bool is_tile_seen_city(const struct player *pow_player, Returns TRUE iff all the tiles of a city and all the tiles of its trade partners are seen by pow_player. **************************************************************************/ -static bool is_tile_seen_traderoute(const struct player *pow_player, - const struct city *target_city) +static bool is_tile_seen_trade_route(const struct player *pow_player, + const struct city *target_city) { /* Don't know who the trade routes will go to. */ if (!can_player_see_city_internals(pow_player, target_city)) { @@ -187,7 +187,7 @@ static bool is_req_knowable(const struct player *pov_player, case REQ_RANGE_ADJACENT: case REQ_RANGE_CONTINENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: case REQ_RANGE_ALLIANCE: @@ -243,7 +243,7 @@ static bool is_req_knowable(const struct player *pov_player, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -329,7 +329,7 @@ static bool is_req_knowable(const struct player *pov_player, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: /* Invalid range */ @@ -422,7 +422,7 @@ static bool is_req_knowable(const struct player *pov_player, /* Unknown */ return FALSE; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -459,7 +459,7 @@ static bool is_req_knowable(const struct player *pov_player, /* Only wonders (great or small) can be required in those ranges. * Wonders are always visible. */ return TRUE; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: /* Could be known for trade routes to cities owned by pov_player as * long as the requirement is present. Not present requirements would * require knowledge that no trade routes to another foreign city @@ -619,7 +619,7 @@ static bool is_req_knowable(const struct player *pov_player, return TRUE; case REQ_RANGE_CONTINENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: case REQ_RANGE_ALLIANCE: @@ -677,11 +677,11 @@ static bool is_req_knowable(const struct player *pov_player, * tile. Is returning TRUE in those cases worth the added complexity * and the extra work for the computer? */ return is_tile_seen_city(pov_player, context->city); - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: /* TODO: The answer is known when the universal is located on a seen * tile. Is returning TRUE in those cases worth the added complexity * and the extra work for the computer? */ - return is_tile_seen_traderoute(pov_player, context->city); + return is_tile_seen_trade_route(pov_player, context->city); case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_ALLIANCE: diff --git a/common/metaknowledge.h b/common/metaknowledge.h index 14ebeb2b21..97e7e6e103 100644 --- a/common/metaknowledge.h +++ b/common/metaknowledge.h @@ -1,4 +1,4 @@ -/********************************************************************** +/*********************************************************************** Freeciv - Copyright (C) 1996-2013 - Freeciv Development Team This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by diff --git a/common/networking/packets.def b/common/networking/packets.def index 5b9eaf4317..5d05b50fea 100644 --- a/common/networking/packets.def +++ b/common/networking/packets.def @@ -243,7 +243,7 @@ type FTM = uint8(enum free_tech_method) type GLS = uint8(enum gameloss_style) type UTYF = uint8(enum unit_type_flag_id) type CBONUS_TYPE = uint8(enum combat_bonus_type) -type TRI = uint8(enum traderoute_illegal_cancelling) +type TRI = uint8(enum trade_route_illegal_cancelling) type RDIR = uint8(enum route_direction) type MOVE_MODE = uint8(enum road_move_mode) type ACTION_AUTO_CAUSE = uint8(enum action_auto_perf_cause) @@ -258,7 +258,7 @@ type HAPPYBORDERSTYPE = uint8(enum happyborders_type) type TECH_COST_STYLE = uint8(enum tech_cost_style) type TECH_LEAKAGE_STYLE = uint8(enum tech_leakage_style) type GOLD_UPKEEP_STYLE = uint8(enum gold_upkeep_style) -type TR_BONUS_TYPE = uint8(enum traderoute_bonus_type) +type TR_BONUS_TYPE = uint8(enum trade_route_bonus_type) type GOODS_SELECTION = uint8(enum goods_selection_method) type EUS = uint8(enum extra_unit_seen_type) type VLAYER = uint8(enum vision_layer) @@ -747,7 +747,7 @@ PACKET_CITY_INFO = 31; sc, lsend, is-game-info, force, cancel(PACKET_CITY_SHORT_ SINT16 food_stock; UINT16 shield_stock; - UINT8 traderoute_count; + UINT8 trade_route_count; UINT16 pollution; UINT16 illness_trade; @@ -815,7 +815,7 @@ PACKET_CITY_SHORT_INFO = 32; sc, lsend, is-game-info, cancel(PACKET_CITY_INFO), ESTRING name[MAX_LEN_CITYNAME]; end -PACKET_TRADEROUTE_INFO = 249; sc, lsend +PACKET_TRADE_ROUTE_INFO = 249; sc, lsend CITY city; UINT8 index; CITY partner; diff --git a/common/reqtext.c b/common/reqtext.c index 77d90f72c4..8e7fa8d35a 100644 --- a/common/reqtext.c +++ b/common/reqtext.c @@ -149,7 +149,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: /* Not supported. */ @@ -228,7 +228,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: /* Not supported. */ @@ -304,7 +304,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: /* Not supported. */ @@ -715,9 +715,9 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, } return TRUE; } - /* surviving or non-wonder continent-ranged requirements not supported */ + /* Surviving or non-wonder continent-ranged requirements not supported */ break; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { if (can_improvement_go_obsolete(preq->source.value.building)) { @@ -892,7 +892,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, extra_name_translation(preq->source.value.extra)); } return TRUE; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { cat_snprintf(buf, bufsz, @@ -933,7 +933,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_TILE: case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -999,7 +999,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, terrain_name_translation(preq->source.value.terrain)); } return TRUE; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { cat_snprintf(buf, bufsz, @@ -1105,7 +1105,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: /* Not supported. */ @@ -1176,7 +1176,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: /* Not supported. */ @@ -1206,7 +1206,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case VUT_NATIONALITY: switch (preq->range) { - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { cat_snprintf(buf, bufsz, @@ -1266,7 +1266,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, nation_adjective_translation(preq->source.value.origowner)); } return TRUE; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_WORLD: case REQ_RANGE_ALLIANCE: case REQ_RANGE_TEAM: @@ -1364,7 +1364,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: /* Not supported. */ @@ -1440,7 +1440,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_WORLD: case REQ_RANGE_COUNT: @@ -1469,7 +1469,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -1555,7 +1555,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_WORLD: case REQ_RANGE_COUNT: @@ -1584,7 +1584,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -1614,7 +1614,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -1655,7 +1655,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -1685,7 +1685,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -1730,7 +1730,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -1833,7 +1833,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -1865,7 +1865,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -1901,7 +1901,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -1993,7 +1993,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case VUT_MINSIZE: switch (preq->range) { - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { cat_snprintf(buf, bufsz, @@ -2064,7 +2064,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, preq->source.value.minculture); } return TRUE; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { cat_snprintf(buf, bufsz, @@ -2190,7 +2190,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, preq->source.value.minforeignpct); } return TRUE; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { cat_snprintf(buf, bufsz, @@ -2279,7 +2279,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, } return TRUE; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -2382,7 +2382,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, (preq->source.value.terrainclass)); } return TRUE; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { cat_snprintf(buf, bufsz, @@ -2479,7 +2479,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, terrain_flag_id_translated_name(preq->source.value.terrainflag)); } return TRUE; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { cat_snprintf(buf, bufsz, @@ -2590,7 +2590,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, road_flag_id_translated_name(preq->source.value.roadflag)); } return TRUE; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { cat_snprintf(buf, bufsz, @@ -2700,7 +2700,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, extra_flag_id_translated_name(preq->source.value.extraflag)); } return TRUE; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { cat_snprintf(buf, bufsz, @@ -2861,7 +2861,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_TEAM: case REQ_RANGE_ALLIANCE: @@ -2890,7 +2890,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -2992,7 +2992,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, } return TRUE; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -3045,7 +3045,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, city_property); } return TRUE; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: fc_strlcat(buf, prefix, bufsz); if (preq->present) { /* TRANS: city property ("owned by original", etc) */ @@ -3130,7 +3130,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, } return TRUE; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -3199,7 +3199,7 @@ bool req_text_insert(char *buf, size_t bufsz, struct player *pplayer, } return TRUE; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: diff --git a/common/requirements.c b/common/requirements.c index 695b65d668..8661caeb2e 100644 --- a/common/requirements.c +++ b/common/requirements.c @@ -1105,12 +1105,12 @@ struct requirement req_from_str(const char *type, const char *range, && req.range != REQ_RANGE_CADJACENT && req.range != REQ_RANGE_ADJACENT && req.range != REQ_RANGE_CITY - && req.range != REQ_RANGE_TRADEROUTE); + && req.range != REQ_RANGE_TRADE_ROUTE); break; case VUT_EXTRA: case VUT_ROADFLAG: case VUT_EXTRAFLAG: - invalid = (req.range > REQ_RANGE_TRADEROUTE); + invalid = (req.range > REQ_RANGE_TRADE_ROUTE); break; case VUT_ADVANCE: case VUT_TECHFLAG: @@ -1128,7 +1128,7 @@ struct requirement req_from_str(const char *type, const char *range, case VUT_NATIONALITY: case VUT_CITYSTATUS: invalid = (req.range != REQ_RANGE_CITY - && req.range != REQ_RANGE_TRADEROUTE); + && req.range != REQ_RANGE_TRADE_ROUTE); break; case VUT_GOOD: case VUT_ORIGINAL_OWNER: @@ -1136,7 +1136,7 @@ struct requirement req_from_str(const char *type, const char *range, break; case VUT_MINCULTURE: invalid = (req.range != REQ_RANGE_CITY - && req.range != REQ_RANGE_TRADEROUTE + && req.range != REQ_RANGE_TRADE_ROUTE && req.range != REQ_RANGE_PLAYER && req.range != REQ_RANGE_TEAM && req.range != REQ_RANGE_ALLIANCE @@ -1225,7 +1225,7 @@ struct requirement req_from_str(const char *type, const char *range, invalid = (req.range != REQ_RANGE_WORLD); break; case VUT_AGE: - /* FIXME: could support TRADEROUTE, TEAM, etc */ + /* FIXME: could support TRADE_ROUTE, TEAM, etc */ invalid = (req.range != REQ_RANGE_LOCAL && req.range != REQ_RANGE_CITY && req.range != REQ_RANGE_PLAYER); @@ -1580,8 +1580,8 @@ static bool present_implies_not_present(const struct requirement *req1, if (present->range != absent->range) { /* Larger ranges are not always strict supersets of smaller ranges. - * Example: Traderoute > CAdjacent but something may be in CAdjacent - * but not in Traderoute. */ + * Example: Trade Route > CAdjacent but something may be in CAdjacent + * but not in Trade Route. */ return FALSE; } @@ -1823,7 +1823,7 @@ static inline bool are_tiles_in_range(const struct tile *tile1, case REQ_RANGE_CADJACENT: return map_distance(tile1, tile2) <= 1; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_LOCAL: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: @@ -1854,7 +1854,7 @@ static inline bool players_in_same_range(const struct player *pplayer1, case REQ_RANGE_PLAYER: return pplayer1 == pplayer2; case REQ_RANGE_CONTINENT: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CITY: case REQ_RANGE_ADJACENT: case REQ_RANGE_CADJACENT: @@ -2035,7 +2035,7 @@ is_building_req_active(const struct req_context *context, return BOOL_TO_TRISTATE(player_has_ever_built(context->player, building)); case REQ_RANGE_CONTINENT: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CITY: case REQ_RANGE_LOCAL: case REQ_RANGE_TILE: @@ -2085,7 +2085,7 @@ is_building_req_active(const struct req_context *context, } else { return TRI_MAYBE; } - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: if (context->city) { if (num_city_buildings(context->city, building) > 0) { return TRI_YES; @@ -2222,7 +2222,7 @@ is_buildingflag_req_active(const struct req_context *context, req->source.value.impr_flag); case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_ALLIANCE: @@ -2290,7 +2290,7 @@ is_tech_req_active(const struct req_context *context, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: break; @@ -2353,7 +2353,7 @@ is_techflag_req_active(const struct req_context *context, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: break; @@ -2388,7 +2388,7 @@ is_minculture_req_active(const struct req_context *context, return TRI_MAYBE; } return BOOL_TO_TRISTATE(city_culture(context->city) >= minculture); - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: if (!context->city) { return TRI_MAYBE; } @@ -2463,7 +2463,7 @@ is_minforeignpct_req_active(const struct req_context *context, foreign_pct = citizens_nation_foreign(context->city) * 100 / city_size_get(context->city); return BOOL_TO_TRISTATE(foreign_pct >= min_foreign_pct); - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: if (!context->city) { return TRI_MAYBE; } @@ -2559,7 +2559,7 @@ is_maxunitsontile_req_active(const struct req_context *context, } adjc_iterate_end; return TRI_NO; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -2631,7 +2631,7 @@ is_extra_req_active(const struct req_context *context, return TRI_NO; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: if (!context->city) { return TRI_MAYBE; } @@ -2703,7 +2703,7 @@ is_good_req_active(const struct req_context *context, case REQ_RANGE_TILE: case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -2824,7 +2824,7 @@ is_terrain_req_active(const struct req_context *context, } city_tile_iterate_end; } return TRI_NO; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: if (!context->city) { return TRI_MAYBE; } @@ -2925,7 +2925,7 @@ is_terrainclass_req_active(const struct req_context *context, } city_tile_iterate_end; return TRI_NO; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: if (!context->city) { return TRI_MAYBE; } @@ -3032,7 +3032,7 @@ is_terrainflag_req_active(const struct req_context *context, } city_tile_iterate_end; return TRI_NO; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: if (!context->city) { return TRI_MAYBE; } @@ -3140,7 +3140,7 @@ is_roadflag_req_active(const struct req_context *context, } city_tile_iterate_end; return TRI_NO; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: if (!context->city) { return TRI_MAYBE; } @@ -3235,7 +3235,7 @@ is_extraflag_req_active(const struct req_context *context, } city_tile_iterate_end; return TRI_NO; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: if (!context->city) { return TRI_MAYBE; } @@ -3305,7 +3305,7 @@ is_terrainalter_req_active(const struct req_context *context, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: /* XXX Could in principle support ADJACENT. */ case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -3471,7 +3471,7 @@ is_nation_req_active(const struct req_context *context, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: break; @@ -3527,7 +3527,7 @@ is_nationgroup_req_active(const struct req_context *context, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: break; @@ -3570,7 +3570,7 @@ is_nationality_req_active(const struct req_context *context, } citizens_iterate_end; return TRI_NO; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: if (context->city == NULL) { return TRI_MAYBE; } @@ -3641,7 +3641,7 @@ is_originalowner_req_active(const struct req_context *context, } return TRI_NO; - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: case REQ_RANGE_ALLIANCE: @@ -3697,7 +3697,7 @@ static enum fc_tristate is_diplrel_in_range(const struct player *target_player, case REQ_RANGE_CADJACENT: case REQ_RANGE_ADJACENT: case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_COUNT: break; @@ -4246,7 +4246,7 @@ is_citytile_req_active(const struct req_context *context, return TRI_NO; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -4286,7 +4286,7 @@ is_citytile_req_active(const struct req_context *context, return TRI_NO; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -4326,7 +4326,7 @@ is_citytile_req_active(const struct req_context *context, return TRI_NO; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -4366,7 +4366,7 @@ is_citytile_req_active(const struct req_context *context, return TRI_NO; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -4420,7 +4420,7 @@ is_citytile_req_active(const struct req_context *context, return TRI_NO; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -4555,7 +4555,7 @@ is_citytile_req_active(const struct req_context *context, return TRI_NO; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: @@ -4611,7 +4611,7 @@ is_citystatus_req_active(const struct req_context *context, return TRI_MAYBE; } return BOOL_TO_TRISTATE(city_owner(context->city) == context->city->original); - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: { enum fc_tristate ret; @@ -4652,7 +4652,7 @@ is_citystatus_req_active(const struct req_context *context, switch (req->range) { case REQ_RANGE_CITY: return BOOL_TO_TRISTATE(context->city->had_famine); - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: { enum fc_tristate ret; @@ -4693,7 +4693,7 @@ is_citystatus_req_active(const struct req_context *context, switch (req->range) { case REQ_RANGE_CITY: return BOOL_TO_TRISTATE(context->city->anarchy > 0); - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: { enum fc_tristate ret; @@ -4734,7 +4734,7 @@ is_citystatus_req_active(const struct req_context *context, switch (req->range) { case REQ_RANGE_CITY: return BOOL_TO_TRISTATE(context->city->rapture > 0); - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: { enum fc_tristate ret; @@ -4775,7 +4775,7 @@ is_citystatus_req_active(const struct req_context *context, switch (req->range) { case REQ_RANGE_CITY: return BOOL_TO_TRISTATE(context->city->acquire_t != CACQ_FOUNDED); - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: { enum fc_tristate ret; @@ -4840,7 +4840,7 @@ is_minsize_req_active(const struct req_context *context, if (context->city == NULL) { return TRI_MAYBE; } else { - if (req->range == REQ_RANGE_TRADEROUTE) { + if (req->range == REQ_RANGE_TRADE_ROUTE) { enum fc_tristate ret; if (city_size_get(context->city) >= req->source.value.minsize) { @@ -5009,7 +5009,7 @@ is_latitude_req_active(const struct req_context *context, return TRI_NO; case REQ_RANGE_CITY: - case REQ_RANGE_TRADEROUTE: + case REQ_RANGE_TRADE_ROUTE: case REQ_RANGE_CONTINENT: case REQ_RANGE_PLAYER: case REQ_RANGE_TEAM: diff --git a/common/requirements.h b/common/requirements.h index 1161b1e014..3e44832cc3 100644 --- a/common/requirements.h +++ b/common/requirements.h @@ -44,7 +44,8 @@ struct astring; #define SPECENUM_VALUE3NAME "Adjacent" #define SPECENUM_VALUE4 REQ_RANGE_CITY #define SPECENUM_VALUE4NAME "City" -#define SPECENUM_VALUE5 REQ_RANGE_TRADEROUTE +#define SPECENUM_VALUE5 REQ_RANGE_TRADE_ROUTE +/* FIXME: -> "Trade Route" */ #define SPECENUM_VALUE5NAME "Traderoute" #define SPECENUM_VALUE6 REQ_RANGE_CONTINENT #define SPECENUM_VALUE6NAME "Continent" diff --git a/common/traderoutes.c b/common/traderoutes.c index fc2e8d6ee0..c711d84ef3 100644 --- a/common/traderoutes.c +++ b/common/traderoutes.c @@ -34,7 +34,7 @@ const char *trade_route_type_names[] = { "Enemy", "EnemyIC", "Team", "TeamIC" }; -const char *traderoute_cancelling_type_names[] = { +const char *trade_route_cancelling_type_names[] = { "Active", "Inactive", "Cancel" }; @@ -53,7 +53,7 @@ unsigned max_trade_routes(const struct city *pcity) } /*********************************************************************//** - What is type of the traderoute between two cities. + What is type of the trade route between two cities. *************************************************************************/ enum trade_route_type cities_trade_route_type(const struct city *pcity1, const struct city *pcity2) @@ -169,24 +169,24 @@ enum trade_route_type trade_route_type_by_name(const char *name) } /*********************************************************************//** - Return human readable name of traderoute cancelling type + Return human readable name of trade route cancelling type *************************************************************************/ -const char *traderoute_cancelling_type_name(enum traderoute_illegal_cancelling type) +const char *trade_route_cancelling_type_name(enum trade_route_illegal_cancelling type) { fc_assert_ret_val(type >= TRI_ACTIVE && type < TRI_LAST, NULL); - return traderoute_cancelling_type_names[type]; + return trade_route_cancelling_type_names[type]; } /*********************************************************************//** - Get traderoute cancelling type by name. + Get trade route cancelling type by name. *************************************************************************/ -enum traderoute_illegal_cancelling traderoute_cancelling_type_by_name(const char *name) +enum trade_route_illegal_cancelling trade_route_cancelling_type_by_name(const char *name) { - enum traderoute_illegal_cancelling type; + enum trade_route_illegal_cancelling type; for (type = TRI_ACTIVE; type < TRI_LAST; type++) { - if (!fc_strcasecmp(traderoute_cancelling_type_names[type], name)) { + if (!fc_strcasecmp(trade_route_cancelling_type_names[type], name)) { return type; } } @@ -717,7 +717,7 @@ bool city_receives_goods(const struct city *pcity, } /*********************************************************************//** - Return goods type for the new traderoute between given cities. + Return goods type for the new trade route between given cities. *************************************************************************/ struct goods_type *goods_from_city_to_unit(struct city *src, struct unit *punit) { diff --git a/common/traderoutes.h b/common/traderoutes.h index d4aed1a4e7..3a1369fc4c 100644 --- a/common/traderoutes.h +++ b/common/traderoutes.h @@ -23,9 +23,9 @@ extern "C" { struct city; struct city_list; -/* What to do with previously established traderoutes that are now illegal. +/* What to do with previously established trade routes that are now illegal. * Used in the network protocol. */ -enum traderoute_illegal_cancelling +enum trade_route_illegal_cancelling { TRI_ACTIVE = 0, /* Keep them active */ TRI_INACTIVE = 1, /* They are inactive */ @@ -49,7 +49,7 @@ enum trade_route_type { }; /* Values used in the network protocol. */ -#define SPECENUM_NAME traderoute_bonus_type +#define SPECENUM_NAME trade_route_bonus_type #define SPECENUM_VALUE0 TBONUS_NONE #define SPECENUM_VALUE0NAME "None" #define SPECENUM_VALUE1 TBONUS_GOLD @@ -74,8 +74,8 @@ enum trade_route_type { struct trade_route_settings { int trade_pct; - enum traderoute_illegal_cancelling cancelling; - enum traderoute_bonus_type bonus_type; + enum trade_route_illegal_cancelling cancelling; + enum trade_route_bonus_type bonus_type; }; struct goods_type; @@ -105,8 +105,8 @@ void trade_route_types_init(void); const char *trade_route_type_name(enum trade_route_type type); enum trade_route_type trade_route_type_by_name(const char *name); -const char *traderoute_cancelling_type_name(enum traderoute_illegal_cancelling type); -enum traderoute_illegal_cancelling traderoute_cancelling_type_by_name(const char *name); +const char *trade_route_cancelling_type_name(enum trade_route_illegal_cancelling type); +enum trade_route_illegal_cancelling trade_route_cancelling_type_by_name(const char *name); struct trade_route_settings * trade_route_settings_by_type(enum trade_route_type type); diff --git a/data/alien/README.alien b/data/alien/README.alien index 1016480a7d..cefba0ebdb 100644 --- a/data/alien/README.alien +++ b/data/alien/README.alien @@ -38,7 +38,7 @@ This list is incomplete - Techs cannot be stolen from Lunatics or Madmen -- Intercontinental traderoutes give same trade than routes +- Intercontinental trade routes give same trade than routes within one continent. International routes give three times as much trade as national ones diff --git a/data/alien/actions.ruleset b/data/alien/actions.ruleset index 7302136eb8..5103fe9bf5 100644 --- a/data/alien/actions.ruleset +++ b/data/alien/actions.ruleset @@ -440,7 +440,7 @@ target_reqs = "MaxUnitsOnTile", "1", "Tile", TRUE } -[actionenabler_traderoute] +[actionenabler_trade_route] action = "Establish Trade Route" actor_reqs = { "type", "name", "range", "present" diff --git a/data/alien/game.ruleset b/data/alien/game.ruleset index 26c550c93c..211cd4a95b 100644 --- a/data/alien/game.ruleset +++ b/data/alien/game.ruleset @@ -486,14 +486,14 @@ effects = "ReducePopulation" ; For each of the trade route types: ; "pct" - Trade income %. If this is 0, trade route cannot be ; established at all -; "cancelling" - What to do to previously established traderoutes when they +; "cancelling" - What to do to previously established trade routes when they ; turn illegal ; "Active" - Keep them active (although they will only ; provide nonzero income if illegal due to ; trademindist rather than pct==0) ; "Inactive" - Keep them inactive ; "Cancel" - Cancel them altogether -; "bonus" - One-time bonuses granted when traderoute established +; "bonus" - One-time bonuses granted when trade route established ; "None" - No one-time bonus ; "Gold" - Bonus to gold ; "Science" - Bonus to research @@ -543,7 +543,7 @@ goods_selection = "Arrival" ; This value is applied to both ends of bidirectional routes. ; to_pct = Income for the receiving end of the trade route. Default is 100% ; This value is not used at all in case of bidirectional routes. -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100% +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100% ; flags ; - "Bidirectional" = Trade route carrying the goods does not have "from" and "to" ; ends, but both ends are considered the same. diff --git a/data/civ1/actions.ruleset b/data/civ1/actions.ruleset index bbbf391d6a..a231645dd6 100644 --- a/data/civ1/actions.ruleset +++ b/data/civ1/actions.ruleset @@ -436,7 +436,7 @@ target_reqs = "MaxUnitsOnTile", "1", "Tile", TRUE } -[actionenabler_traderoute] +[actionenabler_trade_route] action = "Establish Trade Route" actor_reqs = { "type", "name", "range", "present" diff --git a/data/civ1/game.ruleset b/data/civ1/game.ruleset index fbb60487b2..bdf89fab8e 100644 --- a/data/civ1/game.ruleset +++ b/data/civ1/game.ruleset @@ -513,7 +513,7 @@ effects = "ReducePopulation" ; For each of the trade route types: ; "pct" - Trade income %. If this is 0, trade route cannot be ; established at all -; "cancelling" - What to do to previously established traderoutes when they +; "cancelling" - What to do to previously established trade routes when they ; turn illegal ; "Active" - Keep them active (although they will only ; provide nonzero income if illegal due to @@ -521,11 +521,11 @@ effects = "ReducePopulation" ; "Inactive" - Keep them inactive ; "Cancel" - Cancel them altogether ; -; If you have established a traderoute with enemy city nearby, conquer it and -; thus get traderoute between your cities with less than trademindist distance, -; civ1 removes the traderoute completely. +; If you have established a trade route with enemy city nearby, conquer it and +; thus get trade route between your cities with less than trademindist distance, +; civ1 removes the trade route completely. ; -; "bonus" - One-time bonuses granted when traderoute established +; "bonus" - One-time bonuses granted when trade route established ; "None" - No one-time bonus ; "Gold" - Bonus to gold ; "Science" - Bonus to research @@ -575,7 +575,7 @@ goods_selection = "Arrival" ; This value is applied to both ends of bidirectional routes. ; to_pct = Income for the receiving end of the trade route. Default is 100% ; This value is not used at all in case of bidirectional routes. -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100% +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100% ; flags ; - "Bidirectional" = Trade route carrying the goods does not have "from" and "to" ; ends, but both ends are considered the same. diff --git a/data/civ2/actions.ruleset b/data/civ2/actions.ruleset index 8804009d9d..966c67154a 100644 --- a/data/civ2/actions.ruleset +++ b/data/civ2/actions.ruleset @@ -612,7 +612,7 @@ target_reqs = "MaxUnitsOnTile", "1", "Tile", TRUE } -[actionenabler_traderoute] +[actionenabler_trade_route] action = "Establish Trade Route" actor_reqs = { "type", "name", "range", "present" diff --git a/data/civ2/game.ruleset b/data/civ2/game.ruleset index 10e07648f6..cef60d38a5 100644 --- a/data/civ2/game.ruleset +++ b/data/civ2/game.ruleset @@ -442,7 +442,7 @@ negative_label = _("BC") ; For each of the trade route types: ; "pct" - Trade income %. If this is 0, trade route cannot be ; established at all -; "cancelling" - What to do to previously established traderoutes when they +; "cancelling" - What to do to previously established trade routes when they ; turn illegal ; "Active" - Keep them active (although they will only ; provide nonzero income if illegal due to @@ -451,10 +451,10 @@ negative_label = _("BC") ; "Cancel" - Cancel them altogether ; ; TODO: What`s correct "cancelling" behavior for civ2? If you have established -; traderoute with enemy city nearby and conquer it -> what happens to traderoute +; trade route with enemy city nearby and conquer it -> what happens to trade route ; between your cities with less than trademindist distance? ; -; "bonus" - One-time bonuses granted when traderoute established +; "bonus" - One-time bonuses granted when trade route established ; "None" - No one-time bonus ; "Gold" - Bonus to gold ; "Science" - Bonus to research @@ -506,7 +506,7 @@ goods_selection = "Arrival" ; This value is applied to both ends of bidirectional routes. ; to_pct = Income for the receiving end of the trade route. Default is 100% ; This value is not used at all in case of bidirectional routes. -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100% +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100% ; flags ; - "Bidirectional" = Trade route carrying the goods does not have "from" and "to" ; ends, but both ends are considered the same. diff --git a/data/civ2civ3/actions.ruleset b/data/civ2civ3/actions.ruleset index 393d3d7546..a884b7e76a 100644 --- a/data/civ2civ3/actions.ruleset +++ b/data/civ2civ3/actions.ruleset @@ -712,7 +712,7 @@ target_reqs = "CityTile", "Center", "Tile", FALSE } -[actionenabler_traderoute] +[actionenabler_trade_route] action = "Establish Trade Route" actor_reqs = { "type", "name", "range", "present" diff --git a/data/civ2civ3/game.ruleset b/data/civ2civ3/game.ruleset index 2be989e60e..3bc0936d1b 100644 --- a/data/civ2civ3/game.ruleset +++ b/data/civ2civ3/game.ruleset @@ -583,14 +583,14 @@ cons_msg = _("Your nation has achieved 10000 culture points!") ; For each of the trade route types: ; "pct" - Trade income %. If this is 0, trade route cannot be ; established at all -; "cancelling" - What to do to previously established traderoutes when they +; "cancelling" - What to do to previously established trade routes when they ; turn illegal ; "Active" - Keep them active (although they will only ; provide nonzero income if illegal due to ; trademindist rather than pct==0) ; "Inactive" - Keep them inactive ; "Cancel" - Cancel them altogether -; "bonus" - One-time bonuses granted when traderoute established +; "bonus" - One-time bonuses granted when trade route established ; "None" - No one-time bonus ; "Gold" - Bonus to gold ; "Science" - Bonus to research @@ -640,7 +640,7 @@ goods_selection = "Leaving" ; This value is applied to both ends of bidirectional routes. ; to_pct = Income for the receiving end of the trade route. Default is 100% ; This value is not used at all in case of bidirectional routes. -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100% +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100% ; flags ; - "Bidirectional" = Trade route carrying the goods does not have "from" and "to" ; ends, but both ends are considered the same. diff --git a/data/classic/actions.ruleset b/data/classic/actions.ruleset index 0b28b28a78..12ff867115 100644 --- a/data/classic/actions.ruleset +++ b/data/classic/actions.ruleset @@ -680,7 +680,7 @@ target_reqs = "MaxUnitsOnTile", "1", "Tile", TRUE } -[actionenabler_traderoute] +[actionenabler_trade_route] action = "Establish Trade Route" actor_reqs = { "type", "name", "range", "present" diff --git a/data/classic/game.ruleset b/data/classic/game.ruleset index 7235a0ac0a..2f973c9e79 100644 --- a/data/classic/game.ruleset +++ b/data/classic/game.ruleset @@ -496,14 +496,14 @@ effects = "ReducePopulation", "Fallout" ; For each of the trade route types: ; "pct" - Trade income %. If this is 0, trade route cannot be ; established at all -; "cancelling" - What to do to previously established traderoutes when they +; "cancelling" - What to do to previously established trade routes when they ; turn illegal ; "Active" - Keep them active (although they will only ; provide nonzero income if illegal due to ; trademindist rather than pct==0) ; "Inactive" - Keep them inactive ; "Cancel" - Cancel them altogether -; "bonus" - One-time bonuses granted when traderoute established +; "bonus" - One-time bonuses granted when trade route established ; "None" - No one-time bonus ; "Gold" - Bonus to gold ; "Science" - Bonus to research @@ -553,7 +553,7 @@ goods_selection = "Arrival" ; This value is applied to both ends of bidirectional routes. ; to_pct = Income for the receiving end of the trade route. Default is 100% ; This value is not used at all in case of bidirectional routes. -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100% +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100% ; flags ; - "Bidirectional" = Trade route carrying the goods does not have "from" and "to" ; ends, but both ends are considered the same. diff --git a/data/goldkeep/README.goldkeep b/data/goldkeep/README.goldkeep index dc05464c4c..a5f3e3e9b5 100644 --- a/data/goldkeep/README.goldkeep +++ b/data/goldkeep/README.goldkeep @@ -44,7 +44,7 @@ this file. and forest, unless roads are built; nor can they ride maglevs (they ride the railroad instead). Freight does not obsolete Caravan so player does not lose ability - to build units to establish traderoutes to remote cities. + to build units to establish trade routes to remote cities. - Triremes can move on rivers (but later ships cannot). diff --git a/data/goldkeep/actions.ruleset b/data/goldkeep/actions.ruleset index fbcb2e993d..b5eb694f00 100644 --- a/data/goldkeep/actions.ruleset +++ b/data/goldkeep/actions.ruleset @@ -694,7 +694,7 @@ target_reqs = "MinHitPoints", "2", "Local", TRUE } -[actionenabler_traderoute] +[actionenabler_trade_route] action = "Establish Trade Route" actor_reqs = { "type", "name", "range", "present" diff --git a/data/goldkeep/game.ruleset b/data/goldkeep/game.ruleset index e028478e2d..9744220d29 100644 --- a/data/goldkeep/game.ruleset +++ b/data/goldkeep/game.ruleset @@ -514,14 +514,14 @@ cons_msg = _("You sighted your first foreign shore!") ; For each of the trade route types: ; "pct" - Trade income %. If this is 0, trade route cannot be ; established at all -; "cancelling" - What to do to previously established traderoutes when they +; "cancelling" - What to do to previously established trade routes when they ; turn illegal ; "Active" - Keep them active (although they will only ; provide nonzero income if illegal due to ; trademindist rather than pct==0) ; "Inactive" - Keep them inactive ; "Cancel" - Cancel them altogether -; "bonus" - One-time bonuses granted when traderoute established +; "bonus" - One-time bonuses granted when trade route established ; "None" - No one-time bonus ; "Gold" - Bonus to gold ; "Science" - Bonus to research @@ -571,7 +571,7 @@ goods_selection = "Leaving" ; This value is applied to both ends of bidirectional routes. ; to_pct = Income for the receiving end of the trade route. Default is 100% ; This value is not used at all in case of bidirectional routes. -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100% +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100% ; flags ; - "Bidirectional" = Trade route carrying the goods does not have "from" and "to" ; ends, but both ends are considered the same. diff --git a/data/granularity/game.ruleset b/data/granularity/game.ruleset index 159c123007..fb500fe8d6 100644 --- a/data/granularity/game.ruleset +++ b/data/granularity/game.ruleset @@ -469,14 +469,14 @@ negative_label = _("BCE") ; For each of the trade route types: ; "pct" - Trade income %. If this is 0, trade route cannot be ; established at all -; "cancelling" - What to do to previously established traderoutes when they +; "cancelling" - What to do to previously established trade routes when they ; turn illegal ; "Active" - Keep them active (although they will only ; provide nonzero income if illegal due to ; trademindist rather than pct==0) ; "Inactive" - Keep them inactive ; "Cancel" - Cancel them altogether -; "bonus" - One-time bonuses granted when traderoute established +; "bonus" - One-time bonuses granted when trade route established ; "None" - No one-time bonus ; "Gold" - Bonus to gold ; "Science" - Bonus to research @@ -526,7 +526,7 @@ goods_selection = "Leaving" ; This value is applied to both ends of bidirectional routes. ; to_pct = Income for the receiving end of the trade route. Default is 100% ; This value is not used at all in case of bidirectional routes. -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100% +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100% ; flags ; - "Bidirectional" = Trade route carrying the goods does not have "from" and "to" ; ends, but both ends are considered the same. diff --git a/data/multiplayer/game.ruleset b/data/multiplayer/game.ruleset index d0a4513102..724f0f56b2 100644 --- a/data/multiplayer/game.ruleset +++ b/data/multiplayer/game.ruleset @@ -468,14 +468,14 @@ negative_label = _("BCE") ; For each of the trade route types: ; "pct" - Trade income %. If this is 0, trade route cannot be ; established at all -; "cancelling" - What to do to previously established traderoutes when they +; "cancelling" - What to do to previously established trade routes when they ; turn illegal ; "Active" - Keep them active (although they will only ; provide nonzero income if illegal due to ; trademindist rather than pct==0) ; "Inactive" - Keep them inactive ; "Cancel" - Cancel them altogether -; "bonus" - One-time bonuses granted when traderoute established +; "bonus" - One-time bonuses granted when trade route established ; "None" - No one-time bonus ; "Gold" - Bonus to gold ; "Science" - Bonus to research @@ -525,7 +525,7 @@ goods_selection = "Arrival" ; This value is applied to both ends of bidirectional routes. ; to_pct = Income for the receiving end of the trade route. Default is 100% ; This value is not used at all in case of bidirectional routes. -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100% +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100% ; flags ; - "Bidirectional" = Trade route carrying the goods does not have "from" and "to" ; ends, but both ends are considered the same. diff --git a/data/ruledit/comments-3.3.txt b/data/ruledit/comments-3.3.txt index d168559faf..c4fa5c80e8 100644 --- a/data/ruledit/comments-3.3.txt +++ b/data/ruledit/comments-3.3.txt @@ -1081,14 +1081,14 @@ trade_settings = "\ ; For each of the trade route types:\n\ ; \"pct\" - Trade income %. If this is 0, trade route cannot be\n\ ; established at all\n\ -; \"cancelling\" - What to do to previously established traderoutes when they\n\ +; \"cancelling\" - What to do to previously established trade routes when they\n\ ; turn illegal\n\ ; \"Active\" - Keep them active (although they will only\n\ ; provide nonzero income if illegal due to\n\ ; trademindist rather than pct==0)\n\ ; \"Inactive\" - Keep them inactive\n\ ; \"Cancel\" - Cancel them altogether\n\ -; \"bonus\" - One-time bonuses granted when traderoute established\n\ +; \"bonus\" - One-time bonuses granted when trade route established\n\ ; \"None\" - No one-time bonus\n\ ; \"Gold\" - Bonus to gold\n\ ; \"Science\" - Bonus to research\n\ @@ -1115,7 +1115,7 @@ goods = "\ ; This value is applied to both ends of bidirectional routes.\n\ ; to_pct = Income for the receiving end of the trade route. Default is 100\%\n\ ; This value is not used at all in case of bidirectional routes.\n\ -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100\%\n\ +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100\%\n\ ; flags\n\ ; - \"Bidirectional\" = Trade route carrying the goods does not have \"from\" and \"to\"\n\ ; ends, but both ends are considered the same.\n\ diff --git a/data/sandbox/actions.ruleset b/data/sandbox/actions.ruleset index 2bb246e2f9..91f821e38c 100644 --- a/data/sandbox/actions.ruleset +++ b/data/sandbox/actions.ruleset @@ -1053,7 +1053,7 @@ target_reqs = "CityTile", "Center", "Tile", FALSE } -[actionenabler_traderoute] +[actionenabler_trade_route] action = "Establish Trade Route" actor_reqs = { "type", "name", "range", "present" diff --git a/data/sandbox/game.ruleset b/data/sandbox/game.ruleset index f9064a0d5d..3da7710a52 100644 --- a/data/sandbox/game.ruleset +++ b/data/sandbox/game.ruleset @@ -581,14 +581,14 @@ cons_msg = _("Your nation has achieved 10000 culture points!") ; For each of the trade route types: ; "pct" - Trade income %. If this is 0, trade route cannot be ; established at all -; "cancelling" - What to do to previously established traderoutes when they +; "cancelling" - What to do to previously established trade routes when they ; turn illegal ; "Active" - Keep them active (although they will only ; provide nonzero income if illegal due to ; trademindist rather than pct==0) ; "Inactive" - Keep them inactive ; "Cancel" - Cancel them altogether -; "bonus" - One-time bonuses granted when traderoute established +; "bonus" - One-time bonuses granted when trade route established ; "None" - No one-time bonus ; "Gold" - Bonus to gold ; "Science" - Bonus to research @@ -638,7 +638,7 @@ goods_selection = "Leaving" ; This value is applied to both ends of bidirectional routes. ; to_pct = Income for the receiving end of the trade route. Default is 100% ; This value is not used at all in case of bidirectional routes. -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100% +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100% ; flags ; - "Bidirectional" = Trade route carrying the goods does not have "from" and "to" ; ends, but both ends are considered the same. diff --git a/data/stub/game.ruleset b/data/stub/game.ruleset index fa40f5e057..480b2b4d84 100644 --- a/data/stub/game.ruleset +++ b/data/stub/game.ruleset @@ -461,14 +461,14 @@ negative_label = _("BCE") ; For each of the trade route types: ; "pct" - Trade income %. If this is 0, trade route cannot be ; established at all -; "cancelling" - What to do to previously established traderoutes when they +; "cancelling" - What to do to previously established trade routes when they ; turn illegal ; "Active" - Keep them active (although they will only ; provide nonzero income if illegal due to ; trademindist rather than pct==0) ; "Inactive" - Keep them inactive ; "Cancel" - Cancel them altogether -; "bonus" - One-time bonuses granted when traderoute established +; "bonus" - One-time bonuses granted when trade route established ; "None" - No one-time bonus ; "Gold" - Bonus to gold ; "Science" - Bonus to research @@ -518,7 +518,7 @@ goods_selection = "Leaving" ; This value is applied to both ends of bidirectional routes. ; to_pct = Income for the receiving end of the trade route. Default is 100% ; This value is not used at all in case of bidirectional routes. -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100% +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100% ; flags ; - "Bidirectional" = Trade route carrying the goods does not have "from" and "to" ; ends, but both ends are considered the same. diff --git a/data/webperimental/actions.ruleset b/data/webperimental/actions.ruleset index 6283043393..7a898029c0 100644 --- a/data/webperimental/actions.ruleset +++ b/data/webperimental/actions.ruleset @@ -828,7 +828,7 @@ target_reqs = "CityTile", "Center", "Tile", FALSE } -[actionenabler_traderoute] +[actionenabler_trade_route] action = "Establish Trade Route" actor_reqs = { "type", "name", "range", "present" diff --git a/data/webperimental/game.ruleset b/data/webperimental/game.ruleset index 7b765566a6..2581cb2aad 100644 --- a/data/webperimental/game.ruleset +++ b/data/webperimental/game.ruleset @@ -500,14 +500,14 @@ effects = "ReducePopulation", "Fallout" ; For each of the trade route types: ; "pct" - Trade income %. If this is 0, trade route cannot be ; established at all -; "cancelling" - What to do to previously established traderoutes when they +; "cancelling" - What to do to previously established trade routes when they ; turn illegal ; "Active" - Keep them active (although they will only ; provide nonzero income if illegal due to ; trademindist rather than pct==0) ; "Inactive" - Keep them inactive ; "Cancel" - Cancel them altogether -; "bonus" - One-time bonuses granted when traderoute established +; "bonus" - One-time bonuses granted when trade route established ; "None" - No one-time bonus ; "Gold" - Bonus to gold ; "Science" - Bonus to research @@ -557,7 +557,7 @@ goods_selection = "Leaving" ; This value is applied to both ends of bidirectional routes. ; to_pct = Income for the receiving end of the trade route. Default is 100% ; This value is not used at all in case of bidirectional routes. -; onetime_pct = Onetime bonuses when traderoute is established. Default is 100% +; onetime_pct = Onetime bonuses when trade route gets established. Default is 100% ; flags ; - "Bidirectional" = Trade route carrying the goods does not have "from" and "to" ; ends, but both ends are considered the same. diff --git a/doc/README.effects b/doc/README.effects index 0d5a0b2537..5792e53925 100644 --- a/doc/README.effects +++ b/doc/README.effects @@ -684,9 +684,9 @@ Trade_Revenue_Bonus The amount value is taken from the caravan's home city. Traderoute_Pct - Percentage bonus for trade from traderoutes. This bonus applies after - the value of the traderoute is already calculated. It affects one end - of the traderoute only. + Percentage bonus for trade from trade routes. This bonus applies after + the value of the trade route is already calculated. It affects one end + of the trade route only. Turn_Fragments Year fragments advance by AMOUNT each turn. diff --git a/server/citytools.c b/server/citytools.c index 5e625fb110..bcb68ace52 100644 --- a/server/citytools.c +++ b/server/citytools.c @@ -981,7 +981,7 @@ static void reestablish_city_trade_routes(struct city *pcity) free(proute); free(back); - /* Now announce the traderoute removal */ + /* Now announce the trade route removal */ announce_trade_route_removal(pcity, partner, FALSE); } @@ -2223,7 +2223,7 @@ void broadcast_city_info(struct city *pcity) struct packet_web_city_info_addition web_packet; struct packet_city_short_info sc_pack; struct player *powner = city_owner(pcity); - struct traderoute_packet_list *routes; + struct trade_route_packet_list *routes; struct packet_web_city_info_addition *webp_ptr; /* Send to everyone who can see the city. */ @@ -2243,7 +2243,7 @@ void broadcast_city_info(struct city *pcity) webp_ptr = NULL; } - routes = traderoute_packet_list_new(); + routes = trade_route_packet_list_new(); package_city(pcity, &packet, &nat_packet, &rally_packet, webp_ptr, routes, FALSE); @@ -2257,9 +2257,9 @@ void broadcast_city_info(struct city *pcity) lsend_packet_city_rally_point(pplayer->connections, &rally_packet, FALSE); web_lsend_packet(city_info_addition, pplayer->connections, webp_ptr, FALSE); - traderoute_packet_list_iterate(routes, route_packet) { - lsend_packet_traderoute_info(pplayer->connections, route_packet); - } traderoute_packet_list_iterate_end; + trade_route_packet_list_iterate(routes, route_packet) { + lsend_packet_trade_route_info(pplayer->connections, route_packet); + } trade_route_packet_list_iterate_end; } else if (player_can_see_city_externals(pplayer, pcity)) { reality_check_city(pplayer, pcity->tile); update_dumb_city(pplayer, pcity); @@ -2280,10 +2280,10 @@ void broadcast_city_info(struct city *pcity) } } conn_list_iterate_end; - traderoute_packet_list_iterate(routes, route_packet) { + trade_route_packet_list_iterate(routes, route_packet) { FC_FREE(route_packet); - } traderoute_packet_list_iterate_end; - traderoute_packet_list_destroy(routes); + } trade_route_packet_list_iterate_end; + trade_route_packet_list_destroy(routes); } /************************************************************************//** @@ -2394,7 +2394,7 @@ void send_city_info_at_tile(struct player *pviewer, struct conn_list *dest, struct packet_web_city_info_addition web_packet; struct packet_city_short_info sc_pack; struct player *powner = NULL; - struct traderoute_packet_list *routes = NULL; + struct trade_route_packet_list *routes = NULL; struct packet_web_city_info_addition *webp_ptr; if (!pcity) { @@ -2422,7 +2422,7 @@ void send_city_info_at_tile(struct player *pviewer, struct conn_list *dest, /* Wait that city has been rearranged, if it's currently * not in sane state. */ - routes = traderoute_packet_list_new(); + routes = trade_route_packet_list_new(); /* Send all info to the owner */ update_dumb_city(powner, pcity); @@ -2433,18 +2433,18 @@ void send_city_info_at_tile(struct player *pviewer, struct conn_list *dest, lsend_packet_city_nationalities(dest, &nat_packet, FALSE); lsend_packet_city_rally_point(dest, &rally_packet, FALSE); web_lsend_packet(city_info_addition, dest, webp_ptr, FALSE); - traderoute_packet_list_iterate(routes, route_packet) { - lsend_packet_traderoute_info(dest, route_packet); - } traderoute_packet_list_iterate_end; + trade_route_packet_list_iterate(routes, route_packet) { + lsend_packet_trade_route_info(dest, route_packet); + } trade_route_packet_list_iterate_end; if (dest == powner->connections) { /* HACK: send also a copy to global observers. */ packet.original = city_original_owner(pcity, NULL); conn_list_iterate(game.est_connections, pconn) { if (conn_is_global_observer(pconn)) { send_packet_city_info(pconn, &packet, FALSE); - traderoute_packet_list_iterate(routes, route_packet) { - send_packet_traderoute_info(pconn, route_packet); - } traderoute_packet_list_iterate_end; + trade_route_packet_list_iterate(routes, route_packet) { + send_packet_trade_route_info(pconn, route_packet); + } trade_route_packet_list_iterate_end; } } conn_list_iterate_end; } @@ -2453,7 +2453,7 @@ void send_city_info_at_tile(struct player *pviewer, struct conn_list *dest, /* send info to non-owner */ if (!pviewer) { /* observer */ if (pcity) { - routes = traderoute_packet_list_new(); + routes = trade_route_packet_list_new(); /* Should be dumb_city info? */ package_city(pcity, &packet, &nat_packet, &rally_packet, @@ -2462,9 +2462,9 @@ void send_city_info_at_tile(struct player *pviewer, struct conn_list *dest, lsend_packet_city_nationalities(dest, &nat_packet, FALSE); lsend_packet_city_rally_point(dest, &rally_packet, FALSE); web_lsend_packet(city_info_addition, dest, webp_ptr, FALSE); - traderoute_packet_list_iterate(routes, route_packet) { - lsend_packet_traderoute_info(dest, route_packet); - } traderoute_packet_list_iterate_end; + trade_route_packet_list_iterate(routes, route_packet) { + lsend_packet_trade_route_info(dest, route_packet); + } trade_route_packet_list_iterate_end; } } else { if (map_is_known_and_seen(ptile, pviewer, V_MAIN)) { @@ -2484,10 +2484,10 @@ void send_city_info_at_tile(struct player *pviewer, struct conn_list *dest, } if (routes != NULL) { - traderoute_packet_list_iterate(routes, route_packet) { + trade_route_packet_list_iterate(routes, route_packet) { FC_FREE(route_packet); - } traderoute_packet_list_iterate_end; - traderoute_packet_list_destroy(routes); + } trade_route_packet_list_iterate_end; + trade_route_packet_list_destroy(routes); } } @@ -2500,7 +2500,7 @@ void package_city(struct city *pcity, struct packet_city_info *packet, struct packet_city_nationalities *nat_packet, struct packet_city_rally_point *rally_packet, struct packet_web_city_info_addition *web_packet, - struct traderoute_packet_list *routes, + struct trade_route_packet_list *routes, bool dipl_invest) { int i; @@ -2600,7 +2600,7 @@ void package_city(struct city *pcity, struct packet_city_info *packet, i = 0; trade_routes_iterate(pcity, proute) { - struct packet_traderoute_info *tri_packet = fc_malloc(sizeof(struct packet_traderoute_info)); + struct packet_trade_route_info *tri_packet = fc_malloc(sizeof(struct packet_trade_route_info)); tri_packet->city = pcity->id; tri_packet->index = i; @@ -2609,12 +2609,12 @@ void package_city(struct city *pcity, struct packet_city_info *packet, tri_packet->direction = proute->dir; tri_packet->goods = goods_number(proute->goods); - traderoute_packet_list_append(routes, tri_packet); + trade_route_packet_list_append(routes, tri_packet); i++; } trade_routes_iterate_end; - packet->traderoute_count = i; + packet->trade_route_count = i; output_type_iterate(o) { packet->surplus[o] = pcity->surplus[o]; diff --git a/server/citytools.h b/server/citytools.h index 939f273a5b..45c4e7a517 100644 --- a/server/citytools.h +++ b/server/citytools.h @@ -20,12 +20,12 @@ #define LOG_BUILD_TARGET LOG_DEBUG -#define SPECLIST_TAG traderoute_packet -#define SPECLIST_TYPE struct packet_traderoute_info +#define SPECLIST_TAG trade_route_packet +#define SPECLIST_TYPE struct packet_trade_route_info #include "speclist.h" -#define traderoute_packet_list_iterate(ptrlist, ptr) \ - TYPED_LIST_ITERATE(struct packet_traderoute_info, ptrlist, ptr) -#define traderoute_packet_list_iterate_end LIST_ITERATE_END +#define trade_route_packet_list_iterate(ptrlist, ptr) \ + TYPED_LIST_ITERATE(struct packet_trade_route_info, ptrlist, ptr) +#define trade_route_packet_list_iterate_end LIST_ITERATE_END int build_points_left(struct city *pcity); @@ -55,7 +55,7 @@ void package_city(struct city *pcity, struct packet_city_info *packet, struct packet_city_nationalities *nat_packet, struct packet_city_rally_point *rally_packet, struct packet_web_city_info_addition *web_packet, - struct traderoute_packet_list *routes, + struct trade_route_packet_list *routes, bool dipl_invest); void reality_check_city(struct player *pplayer, struct tile *ptile); diff --git a/server/cityturn.c b/server/cityturn.c index 36d72b6c51..62895d84ee 100644 --- a/server/cityturn.c +++ b/server/cityturn.c @@ -610,7 +610,7 @@ void update_city_activities(struct player *pplayer) citizens_convert(pcity); - /* Cancel traderoutes that cannot exist any more */ + /* Cancel trade routes that cannot exist any more */ trade_routes_iterate_safe(pcity, proute) { struct city *tcity = game_city_by_number(proute->partner); @@ -1427,7 +1427,7 @@ static bool worklist_item_postpone_req_vec(struct universal *target, } break; case VUT_NATION: - if (preq->range < REQ_RANGE_TRADEROUTE + if (preq->range < REQ_RANGE_TRADE_ROUTE || preq->range == REQ_RANGE_PLAYER) { /* At higher ranges, new players with their nations may arrive */ purge = TRUE; @@ -1457,7 +1457,7 @@ static bool worklist_item_postpone_req_vec(struct universal *target, } break; case VUT_NATIONGROUP: - if (preq->range < REQ_RANGE_TRADEROUTE + if (preq->range < REQ_RANGE_TRADE_ROUTE || preq->range == REQ_RANGE_PLAYER) { /* At higher ranges, new players with their nations may arrive */ purge = TRUE; diff --git a/server/diplomats.c b/server/diplomats.c index a54c86a48d..85d69ce128 100644 --- a/server/diplomats.c +++ b/server/diplomats.c @@ -331,7 +331,7 @@ bool diplomat_investigate(struct player *pplayer, struct unit *pdiplomat, struct packet_city_nationalities nat_packet; struct packet_city_rally_point rally_packet; struct packet_web_city_info_addition web_packet; - struct traderoute_packet_list *routes; + struct trade_route_packet_list *routes; const struct unit_type *act_utype; struct packet_web_city_info_addition *webp_ptr; @@ -386,7 +386,7 @@ bool diplomat_investigate(struct player *pplayer, struct unit *pdiplomat, webp_ptr = NULL; } - routes = traderoute_packet_list_new(); + routes = trade_route_packet_list_new(); package_city(pcity, &city_packet, &nat_packet, &rally_packet, webp_ptr, routes, TRUE); /* We need to force to send the packet to ensure the client will receive @@ -396,11 +396,11 @@ bool diplomat_investigate(struct player *pplayer, struct unit *pdiplomat, lsend_packet_city_nationalities(pplayer->connections, &nat_packet, TRUE); lsend_packet_city_rally_point(pplayer->connections, &rally_packet, TRUE); web_lsend_packet(city_info_addition, pplayer->connections, webp_ptr, TRUE); - traderoute_packet_list_iterate(routes, route_packet) { - lsend_packet_traderoute_info(pplayer->connections, route_packet); + trade_route_packet_list_iterate(routes, route_packet) { + lsend_packet_trade_route_info(pplayer->connections, route_packet); FC_FREE(route_packet); - } traderoute_packet_list_iterate_end; - traderoute_packet_list_destroy(routes); + } trade_route_packet_list_iterate_end; + trade_route_packet_list_destroy(routes); /* This may cause a diplomatic incident */ action_consequence_success(paction, pplayer, act_utype, cplayer, diff --git a/server/rssanity.c b/server/rssanity.c index 6d38347b76..6885e124d4 100644 --- a/server/rssanity.c +++ b/server/rssanity.c @@ -184,7 +184,7 @@ static bool sanity_check_req_individual(rs_conversion_logger logger, "%s (only great wonders supported)", list_for, improvement_name_translation(pimprove)); return FALSE; - } else if (preq->range > REQ_RANGE_TRADEROUTE && !is_wonder(pimprove)) { + } else if (preq->range > REQ_RANGE_TRADE_ROUTE && !is_wonder(pimprove)) { ruleset_error(logger, LOG_ERROR, "%s: %s-ranged requirement not supported for " "%s (only wonders supported)", list_for, diff --git a/server/ruleset.c b/server/ruleset.c index ae6fba55f2..60715fc731 100644 --- a/server/ruleset.c +++ b/server/ruleset.c @@ -7346,21 +7346,21 @@ static bool load_ruleset_game(struct section_file *file, bool act, "trade.settings%d.pct", i); cancelling = secfile_lookup_str_default(file, "Active", "trade.settings%d.cancelling", i); - set->cancelling = traderoute_cancelling_type_by_name(cancelling); + set->cancelling = trade_route_cancelling_type_by_name(cancelling); if (set->cancelling == TRI_LAST) { ruleset_error(NULL, LOG_ERROR, - "\"%s\" unknown traderoute cancelling type \"%s\".", + "\"%s\" unknown trade route cancelling type \"%s\".", filename, cancelling); ok = FALSE; } bonus = secfile_lookup_str_default(file, "None", "trade.settings%d.bonus", i); - set->bonus_type = traderoute_bonus_type_by_name(bonus, fc_strcasecmp); + set->bonus_type = trade_route_bonus_type_by_name(bonus, fc_strcasecmp); - if (!traderoute_bonus_type_is_valid(set->bonus_type)) { + if (!trade_route_bonus_type_is_valid(set->bonus_type)) { ruleset_error(NULL, LOG_ERROR, - "\"%s\" unknown traderoute bonus type \"%s\".", + "\"%s\" unknown trade route bonus type \"%s\".", filename, bonus); ok = FALSE; } diff --git a/server/settings.c b/server/settings.c index c03639e9fe..88634442dd 100644 --- a/server/settings.c +++ b/server/settings.c @@ -2349,7 +2349,7 @@ static struct setting settings[] = { GEN_ENUM("trade_revenue_style", game.info.trade_revenue_style, SSET_RULES, SSET_ECONOMICS, SSET_RARE, ALLOW_NONE, ALLOW_BASIC, N_("Trade revenue style"), - N_("The formula for the trade a city receives from a traderoute. " + N_("The formula for the trade a city receives from a trade route. " "CLASSIC revenues are given by the sum of the two city sizes " "plus the distance between them, with multipliers for overseas " "and international routes. " diff --git a/server/unithand.c b/server/unithand.c index 332a706601..34eee8f96c 100644 --- a/server/unithand.c +++ b/server/unithand.c @@ -5810,7 +5810,7 @@ static bool unit_do_help_build(struct player *pplayer, } /**********************************************************************//** - Handle request to establish traderoute. If pcity_dest is NULL, assumes + Handle request to establish trade route. If pcity_dest is NULL, assumes that unit is inside target city. Returns TRUE iff action could be done, FALSE if it couldn't. Even if @@ -5832,7 +5832,7 @@ static bool do_unit_establish_trade(struct player *pplayer, struct city *pcity_homecity; struct trade_route_list *routes_out_of_dest; struct trade_route_list *routes_out_of_home; - enum traderoute_bonus_type bonus_type; + enum trade_route_bonus_type bonus_type; struct goods_type *goods; const char *goods_str; const struct unit_type *act_utype; diff --git a/tools/ruleutil/rulesave.c b/tools/ruleutil/rulesave.c index 4050f861b1..a98607f369 100644 --- a/tools/ruleutil/rulesave.c +++ b/tools/ruleutil/rulesave.c @@ -1719,7 +1719,7 @@ static bool save_game_ruleset(const char *filename, const char *name) set_count = 0; for (trt = 0; trt < TRT_LAST; trt++) { struct trade_route_settings *set = trade_route_settings_by_type(trt); - const char *cancelling = traderoute_cancelling_type_name(set->cancelling); + const char *cancelling = trade_route_cancelling_type_name(set->cancelling); if (set->trade_pct != 100 || strcmp(cancelling, "Active")) { char path[256]; @@ -1733,7 +1733,7 @@ static bool save_game_ruleset(const char *filename, const char *name) "%s.pct", path); secfile_insert_str(sfile, cancelling, "%s.cancelling", path); - secfile_insert_str(sfile, traderoute_bonus_type_name(set->bonus_type), + secfile_insert_str(sfile, trade_route_bonus_type_name(set->bonus_type), "%s.bonus", path); } } -- 2.40.1