Reverse Engineering der NLT II

[siebenstreich]

Im Rahmen der Fehlersuche für den Patch habe ich mir eine lokale Kopie von Bright Eyes angelegt und darin -- hauptsächlich in seg038.cpp -- Variablen umbenannt und Bugfixes eingebaut. Es wäre schön, die Erkenntnisse wieder im offiziellen Bright Eyes drinzuhaben. Deswegen habe ich einen Pull Request erstellt.

Meine Fragen:
1) Gibt es die Aussicht, dass der Pull Request angenommen wird? Passiert hier noch etwas?
2) Vermutlich habe ich sämtliche Konventionen verletzt. Ich bin aber lernwillig und würde mich über kritische Rückmeldungen freuen.
3) Seit dem letzten Commit läuft 'make' nicht mehr durch, und ich verstehe nicht so recht warum. Eine richtige Fehlermeldung sehe ich nicht. Kann mir hier jemand helfen? Die letzten Ausgabezeilen lauten

Code:

In file included from seg002.cpp:26:
datseg.cpp:50:23: warning: ‘M302de::g_weapons_table’ defined but not used [-Wunused-variable]
  50 | struct{char unkn[7];} g_weapons_table[65] = { { 0x01, 0x04, 0x0e, 0x02, -0x01, 0x00, 0x00 }, { 0x01, 0x01, 0x0e, 0x06, -0x01, -0x01, -0x03 }, { 0x01, 0x03, 0x0f, 0x02, -0x01, 0x00, 0x00 }, { 0x01, 0x00, 0x10, 0x04, -0x01, -0x03, -0x04 }, { 0x01, 0x03, 0x63, 0x05, 0x01, 0x00, -0x03 }, { 0x01, 0x02, 0x0f, 0x01, -0x01, 0x00, -1 }, { 0x01, 0x04, 0x0e, 0x05, -0x01, 0x00, -0x03 }, { 0x01, 0x03, 0x63, 0x00, 0x03, 0x00, 0x00 }, { 0x02, 0x04, 0x0e, 0x03, -0x01, -0x01, -0x04 }, { 0x01, 0x06, 0x63, 0x00, 0x05, 0x00, 0x00 }, { 0x01, 0x01, 0x0f, 0x03, -0x01, -0x02, -0x03 }, { 0x01, 0x03, 0x63, 0x04, 0x02, -0x01, -0x03 }, { 0x01, 0x01, 0x63, 0x00, 0x00, 0x00, 0x00 }, { 0x02, 0x04, 0x0e, 0x03, -0x01, -0x02, -0x03 }, { 0x01, 0x04, 0x63, 0x00, 0x04, 0x00, 0x00 }, { 0x01, 0x05, 0x0f, 0x05, -0x01, -0x01, -0x03 }, { 0x01, 0x00, 0x10, 0x06, -0x01, -0x02, -0x03 }, { 0x01, 0x03, 0x63, 0x04, 0x02, -0x01, -0x04 }, { 0x01, 0x03, 0x13, 0x07, -0x01, -0x01, -0x04 }, { 0x01, 0x04, 0x0d, 0x01, -0x01, 0x00, -0x02 }, { 0x01, 0x03, 0x10, 0x03, -0x01, 0x00, -1 }, { 0x01, 0x03, 0x10, 0x03, -0x01, 0x00, -1 }, { 0x01, 0x01, 0x0f, 0x05, -0x01, 0x00, -1 }, { 0x01, 0x00, 0x13, 0x02, -0x01, 0x00, -0x06 }, { 0x01, 0x00, 0x63, 0x04, 0x00, -0x03, -0x04 }, { 0x01, 0x02, 0x11, 0x05, -0x01, -0x03, -0x04 }, { 0x01, 0x03, 0x11, 0x06, -0x01, -0x03, -0x04 }, { 0x02, 0x03, 0x0f, 0x02, -0x01, -0x02, -0x04 }, { 0x01, 0x03, 0x0f, 0x03, -0x01, 0x00, -0x03 }, { 0x01, 0x04, 0x0f, 0x05, -0x01, -0x01, -0x03 }, { 0x01, 0x02, 0x0d, 0x06, -0x01, -0x02, -0x03 }, { 0x01, 0x03, 0x12, 0x04, -0x01, -0x01, -1 }, { 0x03, 0x03, 0x11, 0x04, -0x01, -0x03, -0x04 }, { 0x01, 0x01, 0x10, 0x04, -0x01, -0x02, -0x03 }, { 0x01, 0x02, 0x0f, 0x04, -0x01, -0x02, -0x03 }, { 0x01, 0x01, 0x10, 0x07, -0x01, -0x03, -0x04 }, { 0x01, 0x02, 0x0f, 0x02, -0x01, -0x01, -0x02 }, { 0x02, 0x02, 0x0f, 0x03, -0x01, -0x02, -0x02 }, { 0x01, 0x03, 0x0f, 0x02, -0x01, 0x00, -1 }, { 0x01, 0x05, 0x0e, 0x02, -0x01, -0x01, -0x02 }, { 0x01, 0x06, 0x0f, 0x01, -0x01, -0x02, -0x02 }, { 0x01, 0x04, 0x10, 0x03, -0x01, -0x01, -0x03 }, { 0x01, 0x05, 0x0e, 0x01, -0x01, -0x01, -0x02 }, { 0x01, 0x03, 0x10, 0x04, -0x01, 0x00, -1 }, { 0x01, 0x04, 0x0e, 0x02, -0x01, 0x00, -1 }, { 0x01, 0x06, 0x0f, 0x03, -0x01, -0x01, -0x03 }, { 0x01, 0x02, 0x63, 0x00, 0x01, 0x00, 0x00 }, { 0x01, 0x03, 0x0d, 0x04, -0x01, 0x00, -0x02 }, { 0x01, 0x04, 0x0d, 0x02, -0x01, -0x02, -0x04 }, { 0x01, 0x05, 0x0e, 0x02, -0x01, -0x01, -0x03 }, { 0x01, 0x04, 0x63, 0x00, -0x01, 0x00, 0x00 }, { 0x01, 0x03, 0x0f, 0x02, -0x01, 0x00, 0x00 }, { 0x01, 0x03, 0x0f, 0x02, -0x01, 0x00, -1 }, { 0x00, 0x00, 0x63, 0x00, -0x01, 0x00, 0x00 }, { 0x01, 0x05, 0x0e, -0x05, -0x01, -0x02, -0x03 }, { 0x01, 0x0a, 0x0e, -0x63, -0x01, -0x02, -0x08 }, { 0x01, 0x02, 0x0f, -0x63, -0x01, 0x02, -1 }, { 0x01, 0x03, 0x0f, 0x00, -0x01, 0x00, 0x00 }, { 0x02, 0x04, 0x0e, -0x63, -0x01, -0x03, -0x04 }, { 0x01, 0x04, 0x0e, -0x63, -0x01, 0x02, 0x02 }, { 0x01, 0x04, 0x0e, 0x00, -0x01, 0x02, 0x00 }, { 0x01, 0x03, 0x63, 0x05, 0x06, 0x01, 0x01 }, { 0x01, 0x00, 0x63, 0x04, 0x07, -0x03, -0x04 }, { 0x01, 0x01, 0x0f, -0x63, -0x01, 0x00, 0x00 }, { -0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 } }; // ds:0x06b0
      |                      ^~~~~~~~~~~~~~~
In file included from seg002.cpp:26:
datseg.cpp:49:23: warning: ‘M302de::g_ranged_weapons_table’ defined but not used [-Wunused-variable]
  49 | struct{char unkn[8];} g_ranged_weapons_table[9] = { { 0x01, 0x00, 0x00, -0x01, -0x63, -0x63, -0x63, 0x06 }, { 0x01, 0x01, 0x00, 0x00, -0x01, -0x63, -0x63, 0x06 }, { 0x02, 0x01, 0x00, 0x00, 0x00, -0x63, -0x63, 0x07 }, { 0x01, 0x01, 0x00, 0x00, 0x00, -0x01, -0x63, 0x04 }, { 0x02, 0x02, 0x01, 0x00, 0x00, -0x01, -0x02, 0x04 }, { 0x02, 0x02, 0x01, 0x00, -0x01, -0x02, -0x03, 0x03 }, { 0x02, 0x02, 0x01, 0x01, 0x00, 0x00, -0x01, 0x04 }, { 0x09, 0x09, 0x09, 0x09, 0x09, 0x09, 0x09, 0x02 }, { -0x01, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 } }; // ds:0x0668
      |                      ^~~~~~~~~~~~~~~~~~~~~~
In file included from seg002.cpp:20:
../../../../include/dos_inc.h:641:14: warning: ‘Bit8u RealHandle(Bit16u)’ defined but not used [-Wunused-function]
  641 | static Bit8u RealHandle(Bit16u handle) {
      |              ^~~~~~~~~~
make[6]: *** [Makefile:714: seg002.o] Error 1
make[6]: *** Waiting for unfinished jobs....
mv -f .deps/seg004.Tpo .deps/seg004.Po
mv -f .deps/seg006.Tpo .deps/seg006.Po
mv -f .deps/seg005.Tpo .deps/seg005.Po
make[6]: Leaving directory '/<...>/Bright Eyes/git_branch/src/custom/schick/rewrite_m302de'
make[5]: *** [Makefile:734: all-recursive] Error 1
make[5]: Leaving directory '/<...>/Bright Eyes/git_branch/src/custom/schick/rewrite_m302de'
make[4]: *** [Makefile:411: all-recursive] Error 1
make[4]: Leaving directory '/<...>/Bright Eyes/git_branch/src/custom/schick'
make[3]: *** [Makefile:413: all-recursive] Error 1
make[3]: Leaving directory '/<...>/Bright Eyes/git_branch/src/custom'
make[2]: *** [Makefile:448: all-recursive] Error 1
make[2]: Leaving directory '/<...>/Bright Eyes/git_branch/src'
make[1]: *** [Makefile:377: all-recursive] Error 1
make[1]: Leaving directory '/<...>/Bright Eyes/git_branch'