diff options
author | glx <glx@openttd.org> | 2007-11-21 13:46:29 +0000 |
---|---|---|
committer | glx <glx@openttd.org> | 2007-11-21 13:46:29 +0000 |
commit | 4bb0c1b4b97647d7df706af9c3c3b26a5ab0ea40 (patch) | |
tree | 2fdd6e58fc7a1b3e94438d8e8df6fd8669116f6b | |
parent | f92f3aec06203081ca8fadcbc5fb652e92bf8e30 (diff) | |
download | openttd-4bb0c1b4b97647d7df706af9c3c3b26a5ab0ea40.tar.xz |
(svn r11486) -Fix [FS#1457]: industry count was wrong in smallmap
-rw-r--r-- | src/smallmap_gui.cpp | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/src/smallmap_gui.cpp b/src/smallmap_gui.cpp index dd9641d1b..fc80e46a6 100644 --- a/src/smallmap_gui.cpp +++ b/src/smallmap_gui.cpp @@ -60,17 +60,18 @@ static int _smallmap_industry_count; static uint _industries_per_column; /** Macro for ordinary entry of LegendAndColor */ -#define MK(a,b) {a, b, true, false, false} +#define MK(a,b) {a, b, INVALID_INDUSTRYTYPE, true, false, false} /** Macro for end of list marker in arrays of LegendAndColor */ -#define MKEND() {0, STR_NULL, true, true, false} +#define MKEND() {0, STR_NULL, INVALID_INDUSTRYTYPE, true, true, false} /** Macro for break marker in arrays of LegendAndColor. * It will have valid data, though */ -#define MS(a,b) {a, b, true, false, true} +#define MS(a,b) {a, b, INVALID_INDUSTRYTYPE, true, false, true} /** Structure for holding relevant data for legends in small map */ struct LegendAndColour { uint16 colour; ///< color of the item on the map StringID legend; ///< string corresponding to the colored item + IndustryType type; ///< type of industry bool show_on_map; ///< for filtering industries, if true is shown on map in color bool end; ///< this is the end of the list bool col_break; ///< perform a break and go one collumn further @@ -163,6 +164,7 @@ void BuildIndustriesLegend() if (indsp->enabled) { _legend_from_industries[j].legend = indsp->name; _legend_from_industries[j].colour = indsp->map_colour; + _legend_from_industries[j].type = i; _legend_from_industries[j].show_on_map = true; _legend_from_industries[j].col_break = false; _legend_from_industries[j].end = false; @@ -819,15 +821,15 @@ static void SmallMapWindowProc(Window *w, WindowEvent *e) y_org = w->height - 44 - 11 - diff; y = y_org; - uint i = 0; for (;;) { if (_smallmap_type == SMT_INDUSTRY) { /* Industry name must be formated, since it's not in tiny font in the specs. * So, draw with a parameter and use the STR_SMALLMAP_INDUSTRY string, which is tiny font.*/ SetDParam(0, tbl->legend); - SetDParam(1, _industry_counts[_industry_to_list_pos[i]]); - if (!_legend_from_industries[i].show_on_map) { + assert(tbl->type < NUM_INDUSTRYTYPES); + SetDParam(1, _industry_counts[tbl->type]); + if (!tbl->show_on_map) { /* Simply draw the string, not the black border of the legend color. * This will enforce the idea of the disabled item */ DrawString(x + 11, y, STR_SMALLMAP_INDUSTRY, TC_GREY); @@ -836,14 +838,13 @@ static void SmallMapWindowProc(Window *w, WindowEvent *e) GfxFillRect(x, y + 1, x + 8, y + 5, 0); // outer border of the legend color } } else { - /* Anything hat is not an industry is using normal process */ + /* Anything that is not an industry is using normal process */ GfxFillRect(x, y + 1, x + 8, y + 5, 0); DrawString(x + 11, y, tbl->legend, TC_FROMSTRING); } GfxFillRect(x + 1, y + 2, x + 7, y + 4, tbl->colour); // legend color tbl += 1; - i++; y += 6; if (tbl->end) { // end of the list |