summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorpeter1138 <peter1138@openttd.org>2007-06-25 07:33:40 +0000
committerpeter1138 <peter1138@openttd.org>2007-06-25 07:33:40 +0000
commit74e34ee7291fe7aa668cad7afde229632aaf85e6 (patch)
tree7fdecdb9770ef230cc429a68a81b7d20d2770f76
parentd1805ed0546e85236bd44004e0a1cc8ad9cfa1b7 (diff)
downloadopenttd-74e34ee7291fe7aa668cad7afde229632aaf85e6.tar.xz
(svn r10316) -Codechange: (consistently) use index to refer to group names. Also the group string_id is irrelevant unless it is a custom name, so don't 'waste' a savegame string id.
-rw-r--r--src/group_cmd.cpp2
-rw-r--r--src/group_gui.cpp8
-rw-r--r--src/lang/english.txt3
-rw-r--r--src/strings.cpp2
4 files changed, 7 insertions, 8 deletions
diff --git a/src/group_cmd.cpp b/src/group_cmd.cpp
index c5de860dd..7505d63b5 100644
--- a/src/group_cmd.cpp
+++ b/src/group_cmd.cpp
@@ -103,7 +103,7 @@ CommandCost CmdCreateGroup(TileIndex tile, uint32 flags, uint32 p1, uint32 p2)
if (flags & DC_EXEC) {
g->owner = _current_player;
- g->string_id = STR_SV_GROUP_NAME;
+ g->string_id = STR_EMPTY;
g->replace_protection = false;
g->vehicle_type = vt;
diff --git a/src/group_gui.cpp b/src/group_gui.cpp
index 4d19ba925..2d1913b43 100644
--- a/src/group_gui.cpp
+++ b/src/group_gui.cpp
@@ -78,13 +78,13 @@ static int CDECL GroupNameSorter(const void *a, const void *b)
if (ga != last_group[0]) {
last_group[0] = ga;
SetDParam(0, ga->index);
- GetString(last_name[0], ga->string_id, lastof(last_name[0]));
+ GetString(last_name[0], STR_GROUP_NAME, lastof(last_name[0]));
}
if (gb != last_group[1]) {
last_group[1] = gb;
SetDParam(0, gb->index);
- GetString(last_name[1], gb->string_id, lastof(last_name[1]));
+ GetString(last_name[1], STR_GROUP_NAME, lastof(last_name[1]));
}
r = strcmp(last_name[0], last_name[1]); // sort by name
@@ -420,7 +420,7 @@ static void GroupWndProc(Window *w, WindowEvent *e)
/* draw the selected group in white, else we draw it in black */
SetDParam(0, g->index);
- DrawString(10, y1, STR_SV_GROUP_NAME, (gv->group_sel == g->index) ? 12 : 16);
+ DrawString(10, y1, STR_GROUP_NAME, (gv->group_sel == g->index) ? 12 : 16);
/* draw the number of vehicles of the group */
SetDParam(0, g->num_vehicle);
@@ -567,7 +567,7 @@ static void GroupWndProc(Window *w, WindowEvent *e)
const Group *g = GetGroup(gv->group_sel);
SetDParam(0, g->index);
- ShowQueryString(g->string_id, STR_GROUP_RENAME_CAPTION, 31, 150, w, CS_ALPHANUMERAL);
+ ShowQueryString(STR_GROUP_NAME, STR_GROUP_RENAME_CAPTION, 31, 150, w, CS_ALPHANUMERAL);
} break;
diff --git a/src/lang/english.txt b/src/lang/english.txt
index 372b2e518..a27fe393e 100644
--- a/src/lang/english.txt
+++ b/src/lang/english.txt
@@ -2083,8 +2083,6 @@ STR_SV_STNAME_LOWER :Lower {STRING1}
STR_SV_STNAME_HELIPORT :{STRING1} Heliport
STR_SV_STNAME_FOREST :{STRING1} Forest
-STR_SV_GROUP_NAME :{GROUP}
-
############ end of savegame specific region!
##id 0x6800
@@ -3340,6 +3338,7 @@ STR_PROFIT_BAD_THIS_YEAR_GOOD_LAST_YEAR :{TINYFONT}{BLAC
STR_PROFIT_GOOD_THIS_YEAR_BAD_LAST_YEAR :{TINYFONT}{BLACK}Profit this year: {GREEN}{CURRENCY} {BLACK}(last year: {RED}{CURRENCY}{BLACK})
STR_PROFIT_BAD_THIS_YEAR_BAD_LAST_YEAR :{TINYFONT}{BLACK}Profit this year: {RED}{CURRENCY} {BLACK}(last year: {RED}{CURRENCY}{BLACK})
+STR_GROUP_NAME :{GROUP}
STR_VEHICLE_NAME :{VEHICLE}
########
diff --git a/src/strings.cpp b/src/strings.cpp
index 2b1c5bb44..85ed61b5c 100644
--- a/src/strings.cpp
+++ b/src/strings.cpp
@@ -857,7 +857,7 @@ static char* FormatString(char* buff, const char* str, const int64* argv, uint c
assert(IsValidGroup(g));
args[0] = g->index;
- buff = GetStringWithArgs(buff, (g->string_id == STR_SV_GROUP_NAME) ? (StringID)STR_GROUP_NAME_FORMAT : g->string_id, args, last);
+ buff = GetStringWithArgs(buff, IsCustomName(g->string_id) ? g->string_id : (StringID)STR_GROUP_NAME_FORMAT, args, last);
break;
}