13-06-2020, Saturday-16:08:45
(13-06-2020, Saturday-16:02:44 )prissi schrieb: Es war schon dein Patch, nur wurde bisher fixed cost nie abgefragt, sondern die maintenace aus dem übergeordneten
obj_desc_transport_related_t, die im übrigen bereits ein sint32 war. Ich habe nur den Wert in desc vergessen auf sint32 zu ändern.
Und noch eine zweite new_month Routine nur für die fixen Kosten ist ganz schlecht für die Wartbarkeit. Das fällt einem später auf die Füße. Wenn es schlecht aufgeläste abhängigkeiten gibt (Linienmonat muss vor Convoimonat aufgerufen werden), dann muss dass halt so sein. Hacks sind immer eine schlechte Lösung auf lange Sicht.
Ich hab mich nicht getraut die Reihenfolge zu ändern, weil ich da die Folgen nicht überblicke.
Es fehlt noch was aus meinem Patch. Convois müssen nicht auf einer Strecke stehen.
Deshalb :
Zitat:waytype_t wtyp = ignore_wt;
if(schedule_t* s = get_schedule())
{
wtyp = s->get_waytype();
}
else
{
wtyp = fahr[0]->get_waytype();
}
get_owner()->book_running_costs( sum_fixed_costs, wtyp);
Sonst gibt es Absturz. Also get_schedule()->get_waytype() zeigt gelegentlich ins leere.