Reverse Engineering der NLT II

[siebenstreich]

Hast du auch mal in meine Überarbeitung der seg096.cpp reingeschaut? Du hattest den vorherigen Code dort ja auch gelesen.

sieht ganz gut aus, ich hoffe vorsichtig umgestellt - wie testest du das ganze? 

also wie garantierst du das GUI_name_inflect_with_article sich noch genau so verhält wie GUI_names_grammar?

Eigentlich sind es ja alles nur Umbenennungen, incl. z.B. des Funktionsnamens von GUI_names_grammar zu GUI_name_inflect_with_article, um besser abzubilden was die Funktion tut.

Es ist der Binäräquivalenztest, der mich vor Fehlern bewahrt. Wenn der BCC aus dem geänderten Code nach wie vor exakt die SCHICKM.EXE produziert (bis auf diese 17 Bytes), kann ich nichts falsch gemacht haben.
Dieser Test ist unglaublich mächtig. Bei der Grammatik-Geschichte hat er mich sicher 10 mal davor bewahrt, Mist zu produzieren.

Bei den Bugfixes bin ich jedoch im Blindflug unterwegs. Es sind ja immer nur wenige Code-Zeilen. Ich versuche, mich dabei extra zu konzentrieren, so dass ich darauf hoffe, dabei nichts falsch gemacht zu haben. 

 

Code:

enum {
INFLECT_1ST_CASE            = GRAMMAR_CASE_1ST,                   // 0x0000
INFLECT_2ND_CASE            = GRAMMAR_CASE_2ND,                    // 0x0001
INFLECT_4TH_CASE            = GRAMMAR_CASE_4TH,                    // 0x0002
INFLECT_3RD_CASE            = GRAMMAR_CASE_3RD,                    // 0x0003
INFLECT_CASE_BITMASK        = 0x03,     // 0x0003
INFLECT_SINGULAR            = GRAMMAR_NUMBER_SINGULAR << 2,        // 0x0000
INFLECT_PLURAL              = GRAMMAR_NUMBER_PLURAL << 2,          // 0x0004
INFLECT_NUMBER_BITMASK      = 0x01 << 2,                            // 0x0004
INFLECT_CASE_NUMBER_BITMASK = 0x0f,             // 0x000f (1st nibble, includes bit 3 (unused))
INFLECT_GENDER_UNSPECIFIED  = 0x00 << 12,                          // 0x0000
INFLECT_MASCULINE          = (GRAMMAR_GENDER_MASCULINE + 1) << 12, // 0x1000
INFLECT_FEMININE            = (GRAMMAR_GENDER_FEMININE + 1) << 12,  // 0x2000
INFLECT_NEUTER              = (GRAMMAR_GENDER_NEUTER + 1) << 12,    // 0x3000
INFLECT_GENDER_BITMASK      = 0x03 << 12,                          // 0x3000
INFLECT_INDEFINITE_ARTICLE  = GRAMMAR_ARTICLE_INDEFINITE << 14,    // 0x0000
INFLECT_OMIT_ARTICLE        = GRAMMAR_ARTICLE_NONE << 14,          // 0x4000
INFLECT_DEFINITE_ARTICLE    = GRAMMAR_ARTICLE_DEFINITE << 14,      // 0x8000
INFLECT_ARTICLE_BITMASK    = 0x03 << 14                            // 0xc000
};

die Zusammenstellung sind noch ein bisschen kryptisch aus - könnte man da so ein PACK_XYZ Macro machen um das mehr zu vereinheitlichen?

Damit hast du nicht ganz unrecht, zugegeben.

Ich hatte darüber nachgedacht, die Zahlen 2, 12 und 14 nochmal per enum in INFLECT_BIT_OFFSET_NUMBER, INFLECT_BIT_OFFSET_GENDER und INFLECT_BIT_OFFSET_ARTICLE verpacken. Die Zahlen 0x01 und 0x03 entstehen aus der jeweiligen Bit-Breite (1 oder 2) der Stellen mit der entsprechenden Information. Aber irgendwie will ich mich auch nicht verkünsteln.

Vielleicht besser noch einen Kommentar dazuschreiben, der erklärt, wie diese 'inflect_features' Information aufgebaut ist.