+- TT-MS Headquarters (https://www.tt-ms.de/forum)
+-- Forum: Problemecke (/forumdisplay.php?fid=70)
+--- Forum: Probleme mit OpenTTD (/forumdisplay.php?fid=27)
+--- Thema: VarAction2 C4 (year built) (/showthread.php?tid=5143)
VarAction2 C4 (year built) - mb - 13.02.2011 13:53
Mehr oder weniger zufällig habe ich gesehen dass OTTD diese Variable, und schon seit r4611 (!), anscheinend völlig anders interpretiert als TTDPatch?
TTDPatch: Year built (counted from 1920) [d.h. das Jahr in dem das Fz gebaut wurde]
OTTD: ( c ) This is the current date. [d.h. das aktuelle Datum]
S. http://wiki.ttdpatch.net/tiki-index.php?page=VarAction2Vehicles
Hat das mal jemand ausprobiert? Da diese Variable schon seit Jahren im DB Set beo praktisch jedem Wagen bzw jeder Lok benutzt wird, stellt sich die Frage ob das Umfärben der Wagen eigentlich jemals richtig (d.h. wie vorgesehen) funktioniert hat?
Gruß
Michael
RE: VarAction2 C4 (year built) - Eddi - 13.02.2011 15:03
Es gab irgendeinen seltsamen Umbemalungs-Bug, wo die BR 110 von Blau in Beige/Blau (oder Beige/Rot? weiß nicht mehr genau) umbemalt wurde. Das wurde immer als GRF-Bug abgeschrieben.
Ansonsten kenne ich das Verhalten in TTDPatch nicht, kann das also nicht vergleichen.
RE: VarAction2 C4 (year built) - mb - 13.02.2011 15:13
Eddi schrieb:[...] Ansonsten kenne ich das Verhalten in TTDPatch nicht, kann das also nicht vergleichen.
Es geht im Grunde nicht um das (spezielle) Verhalten von TTDPatch, sondern es geht mir darum wie OTTD diese Variable "C4" interpretiert. Laut Spezifikation ist das das "Jahr in dem das Fz gebaut" wurde. Aber irgendwann hat jemand (in reichlich missverständlicher Weise) dazugeschrieben "OTTD: ( c ) This is the current date." Was immer auch damit gemeint war.
Wenn also diese Variable "C4" in OTTD tatsächlich als "current date" interpretiert wird, dann kann eigentlich keine der DB Set Umfärbungen seit r4611 jemals funktioniert haben? Und mich würde auch wundern aus welchem Grunde eine derart wichtige Variable in ihrem Verhalten so grundlegend geändert wurde? Denn "langes Datum" (var49) stand damals ja noch gar nicht zur Debatte, WIMRE.
Gruß
Michael
RE: VarAction2 C4 (year built) - Eddi - 13.02.2011 15:22
also in src/newgrf_engine.cpp steht:
Code:
case 0x44: return Clamp(v->build_year, ORIGINAL_BASE_YEAR, ORIGINAL_MAX_YEAR) - ORIGINAL_BASE_YEAR;... Also ich hab keine Ahnung, was in der Dokumentation gemeint war...
(Nachtrag: mal auseinanderklamüsert: 0x44 steht für 0x80+0x44, also 0xC4. v->build_year ist die Variable, die ausgelesen wird, das wird dann auf den Bereich von 1920 bis 2090 umgerechnet.)
RE: VarAction2 C4 (year built) - mb - 13.02.2011 16:06
Eddi schrieb:also in src/newgrf_engine.cpp steht:
Code:
case 0x44: return Clamp(v->build_year, ORIGINAL_BASE_YEAR, ORIGINAL_MAX_YEAR) - ORIGINAL_BASE_YEAR;
... Also ich hab keine Ahnung, was in der Dokumentation gemeint war...
Dann gehe ich mal davon aus dass die OTTD-spezifische Dokumentation dort mittlerweile überholt, bzw falsch ist:
49 - OTTD r13376 (current date)
C4 - OTTD r4611 (current date)
F2 - OTTD r15542 (currently always zero)
Der letzte Punkt ist nämlich definitiv falsch, denn "F2" funktioniert genauso wie spezifiziert, gibt also nicht "always zero" zurück.
Nun kenne ich natürlich nicht die korrekten OTTD-Versionen die dort stattdessen eingetragen werden sollten, sonst könnte ich das selber machen.
Gruß
Michael
RE: VarAction2 C4 (year built) - Eddi - 13.02.2011 16:08
Das solltest du dann wohl besser den Entwicklern über die üblichen Kanäle mitteilen.
RE: VarAction2 C4 (year built) - planetmaker - 13.02.2011 17:49
Die Wiki sagt nach (nicht nur) meiner Interpretation lediglich aus, dass OpenTTD im "Purchase Menu" das aktuelle Datum zurückliefert (siehe Kopfzeile der entsprechenden Spalte) und ansonsten so funktioniert wie allgemein beschrieben.
RE: VarAction2 C4 (year built) - mb - 13.02.2011 20:44
planetmaker schrieb:Die Wiki sagt nach (nicht nur) meiner Interpretation lediglich aus, dass OpenTTD im "Purchase Menu" das aktuelle Datum zurückliefert (siehe Kopfzeile der entsprechenden Spalte) [...]
Ah! Diese verdammt langen Tabellen immer ...
Falscher Alarm.
Gruß
Michael