diff --git a/src/order_gui.cpp b/src/order_gui.cpp index 6da0298ec5..b3a5b08d1a 100644 --- a/src/order_gui.cpp +++ b/src/order_gui.cpp @@ -115,7 +115,6 @@ static void DrawOrdersWindow(Window *w) int sel; int y, i; bool shared_orders; - byte color; v = GetVehicle(w->window_number); @@ -192,6 +191,11 @@ static void DrawOrdersWindow(Window *w) SetDParam(1, 6); switch (order->type) { + case OT_DUMMY: + SetDParam(1, STR_INVALID_ORDER); + SetDParam(2, order->dest); + break; + case OT_GOTO_STATION: SetDParam(1, StationOrderStrings[order->flags]); SetDParam(2, order->dest); @@ -234,15 +238,10 @@ static void DrawOrdersWindow(Window *w) default: break; } - color = (i == WP(w,order_d).sel) ? 0xC : 0x10; + const byte colour = (i == WP(w,order_d).sel) ? 0xC : 0x10; SetDParam(0, i + 1); - if (order->type != OT_DUMMY) { - DrawString(2, y, str, color); - } else { - SetDParam(1, STR_INVALID_ORDER); - SetDParam(2, order->dest); - DrawString(2, y, str, color); - } + DrawString(2, y, str, colour); + y += 10; } @@ -252,8 +251,8 @@ static void DrawOrdersWindow(Window *w) if (i - w->vscroll.pos < w->vscroll.cap) { str = shared_orders ? STR_END_OF_SHARED_ORDERS : STR_882A_END_OF_ORDERS; - color = (i == WP(w,order_d).sel) ? 0xC : 0x10; - DrawString(2, y, str, color); + const byte colour = (i == WP(w,order_d).sel) ? 0xC : 0x10; + DrawString(2, y, str, colour); } } diff --git a/src/timetable_gui.cpp b/src/timetable_gui.cpp index a978e559c5..0c831692fd 100644 --- a/src/timetable_gui.cpp +++ b/src/timetable_gui.cpp @@ -96,6 +96,10 @@ static void DrawTimetableWindow(Window *w) SetDParam(2, STR_EMPTY); switch (order->type) { + case OT_DUMMY: + SetDParam(0, STR_INVALID_ORDER); + break; + case OT_GOTO_STATION: SetDParam(0, (order->flags & OF_NON_STOP) ? STR_880C_GO_NON_STOP_TO : STR_8806_GO_TO); SetDParam(1, order->dest); @@ -137,14 +141,8 @@ static void DrawTimetableWindow(Window *w) default: break; } - byte colour = (i == selected) ? 0xC : 0x10; - - if (order->type != OT_DUMMY) { - DrawString(2, y, STR_TIMETABLE_GO_TO, colour); - } else { - SetDParam(0, STR_INVALID_ORDER); - DrawString(2, y, STR_TIMETABLE_GO_TO, colour); - } + const byte colour = (i == selected) ? 0xC : 0x10; + DrawString(2, y, STR_TIMETABLE_GO_TO, colour); order_id++; @@ -164,7 +162,7 @@ static void DrawTimetableWindow(Window *w) string = STR_TIMETABLE_TRAVEL_FOR; } - byte colour = (i == selected) ? 0xC : 0x10; + const byte colour = (i == selected) ? 0xC : 0x10; DrawString(12, y, string, colour); if (final_order) break;