Reverse Engineering der NLT II

[siebenstreich]

Wenn ich das richtig verstehe, verwenden alle drei Korrekturen zwei neue Funktionen im Segment Nr. 2.

Original-Bug 19 braucht diese Funktionen nicht, sondern nur eine Zeile zusätzlichen Code. Wobei der Bug wirklich alles andere als kritisch ist. (Ich hatte nicht konkret nach einem Fix gesucht, sondern mir ist die richtige Stelle beim Stöbern von selbst über den Weg gelaufen...)

Diese neuen Funktionen ermitteln die Anzahl der Helden insgesamt, beziehungsweise in der aktuellen Gruppe, ohne den NPC. Neue Funktionen einzubauen ist auf Binärebene mit das aufwändigste überhaupt. Es ist noch aufwändiger als das bloße Einfügen von Code in bestehende Funktionen. Es müssen dazu nicht nur alle Segmentbezüge aktualisiert werden -- dafür habe ich mir bereits vor langer Zeit ein Werkzeug gebastelt --, sondern es muss die Tabelle der öffentlichen Funktionen im Segment des Overlay-Managers ergänzt werden.

Die zusätzlichen Funktionen sind für die Original-Bugs 12, 13, 14 und 15 notwendig. Eine sauberere Lösung wäre eigentlich, den vorhandenen Funktionen count_heroes_available und count_heroes_available_in_group ein zusätzliches Flag <ignore_npc> mit zu übergeben. Ist das vielleicht einfacher umzusetzen in Maschinencode? Ich habe mich für die neuen Funktionen entschieden, weil ich die Signatur der Original-Funktionen nicht verändern wollte.

Dann gäbe es noch die Alternative, den Test aus den vorhandenen Funktionen zusammenzustricken, wie es an manchen Stellen im Originalcode auch gemacht wird. Es sollte

Code:

count_heroes_available_ignore_npc()

dasselbe sein wie

Code:

check_hero(get_hero(6)) ? count_heroes_available() : (count_heroes_available()-1)

und es sollte

Code:

count_heroes_available_in_group_ignore_npc()

dasselbe sein wie

Code:

is_hero_available_in_group(get_hero(6)) ? count_heroes_available_in_group() : (count_heroes_available_in_group()-1)

Mal abgesehen davon, dass ich den Patch jetzt abschließen möchte -- irgendein Kleinkram wird immer wieder noch auftauchen, aber die großen Fehler sollten jetzt eigentlich alle korrigiert sein.

In der Liste der neu gefundenen Bugs sind sehr viele völlig unkritische Kleinigkeiten dabei, das stimmt. Zwei Bugs halte ich schon noch für reparierenswert, aber ich habe mich nicht damit auseinandergesetzt bisher. Vielleicht hast du ja Lust, dem noch nachzugehen.

• Speicherzugriffsfehler nach dem Finalisieren eines lange dauernden (>=1 Tag) Rezepts, für den das brauende Held von der Gruppe abgetrennt wurde. [Genauer: Einen Helden ein lange dauerndes (>= 1 Tag) Rezept in einem Gasthaus brauen lassen und von der Gruppe trennen. Mit der verkleinerten Gruppe irgendwas machen, bis die Zeit verstrichen ist. Dann durch Weiterschalten der aktiven Gruppe auf den Alchemie-Helden im Gasthaus wechseln. Es wird die Meldung über den Erfolg oder Misserfolg der Brauaktion angezeigt. Danach befindet man sich mit diesem einen Helden in der Herberge. In der Statuszeile wird anstelle des Namens der Herberge Datenmüll oder fremde Textteile angezeigt, je nachdem, was die andere Gruppe währenddessen gemacht hat. Z.B. habe ich einfach eine leere Zeile dort gesehen, den Namen einer anderen Herberge (nachdem die anderen Helden in genau dieser Herberge übernachtet hatten um die Zeit abzusitzen), oder die Ortsbeschreibung von Manrin (die natürlich viel zu lang ist und dann in die Heldenbilder hineinhängt).]
  
• AT/PT-Basiswerte bleiben unverändert bei Anlegen des Kraftgürtels (vmtl. auch Gulmond, KK-Elixier, KK-Steigern-Zauber etc.)