Actions rebased#6
Conversation
UNITONSHIPHASMOVED is now a warning fixed itob buffer bug
check teachers with warning level 5
| if (t->ship == i) { | ||
| if (t->hasmoved > 1) { /* schon bewegt! */ | ||
| sprintf(warn_buf, errtxt[UNITONSHIPHASMOVED], uid(t), itob(i)); | ||
| Error(warn_buf, t->line_no, t->long_order); |
There was a problem hiding this comment.
Das hier schient mir der eigentliche Bugfix zu sein, alles andere in diesem commit dient nur der Verwirrung des Reviewers (und macht mir einen Haufen Konflikte beim Cherry-Picking).
ennorehling
left a comment
ennorehling
left a comment
There was a problem hiding this comment.
Was war denn der itob buffer bug? von dem steht in Mantis nichts. Ist der unabhängig davon? Kann der dann sein eigener PR sein, bitte?
ennorehling
left a comment
ennorehling
left a comment
There was a problem hiding this comment.
importiert, war kompilziert zu picken, weil es help.txt nicht mehr gibt.
ennorehling
left a comment
ennorehling
left a comment
There was a problem hiding this comment.
Am Ende habe ich 24 neue commits aus diesem PR extrahiert, und diejenigen kommentiert, die ich nicht mergen kann, und für die ein neuer PR nötig ist.
| int i, j; | ||
|
|
||
| switch (key) { | ||
| case 'b': /* Befehle */ |
There was a problem hiding this comment.
nicht sicher, warum das gebraucht wird, aber ich habe es übernommen.
| ASSUMING200STUDYCOSTS, Assuming learnng costs of 200 silver | ||
| AWARNING, ~a warning | ||
| BEFOREINCOME, before income | ||
| BREEDHORSESORHERBS, BREED HORSES or BREED HERBS |
There was a problem hiding this comment.
Soweit ich sehe, wird diese Meldunge nirgendwo benutzt?
| monument | ||
| dam | ||
| caravanserai | ||
| caravanserei |
There was a problem hiding this comment.
Bitte? Das ist verkehrt. Macht der Server das etwa auch so? Dann sollte es dort gefixt werden.
| remove(OUTPUT_FILE); | ||
| remove(ERROR_FILE); | ||
| set_output(stdout, stderr); | ||
| set_output(stdout, stdout); |
There was a problem hiding this comment.
Die Änderungen an den Tests benötigen einen früheren commit, den ich nicht integriert bekommen habe, deshalb kann ich leider auch diesen nicht integrieren ohne einen neuen PR.
| ISNOTCOMBATSPELL, k | ||
| ITEM, Gegenstand | ||
| LINETOOLONG, Zeile zu lang | ||
| LOCALEMISMATCH, Locales '%s' von ECheck und '%s' der Befehle passen nicht zusammen |
There was a problem hiding this comment.
commit wegen Einführung von neuer Message vorerst nicht integriert.
| @@ -0,0 +1,150 @@ | |||
| name: Automatic build and release | |||
There was a problem hiding this comment.
github workflows scheint mir ein neues feature zu sein, das ich gerne als eigenen PR gesehen hätte, und deshalb heute überspringe.
| to->lives = from->lives; | ||
| to->start_of_orders_line = from->start_of_orders_line; | ||
| to->long_order_line = from->long_order_line; | ||
| if (from->start_of_orders_line) |
There was a problem hiding this comment.
auch dieser commit ist zu komplex, als das ein einfacher cherry-pick möglich gewesen wäre. Die Änderungen am Test-Framework hätten ein separater PR sein sollen, dann wäre vieles einfacher gewesen.
ennorehling
left a comment
ennorehling
left a comment
There was a problem hiding this comment.
Da war ich wohl ein wenig voreilig, das alles so blind zu akzeptieren.
| if (*bp == '"') { | ||
| quote = (bool) ! quote; | ||
| eatwhite = true; | ||
| if ((!quote || quote == *bp) && (*bp == DOUBLE_QUOTE || *bp == SINGLE_QUOTE)) { |
There was a problem hiding this comment.
Diese Änderung musste ich rückgängig machen (commit 4cba246), weil sie nicht mit Kommentaren klar kam, die den Namen einer Einheit wie Patrick O'Connor klar kommt (die also ein einzelnes Anführungszeichen in einem Kommentar haben).
| MISSINGOFFER, Geldgebot fehlt | ||
| MISSINGPARAMETERS, Parameter STUFE oder REGION fehlt | ||
| MISSINGPASSWORD, Kein Passwort angeben | ||
| MISSINGSTART, ERESSEA partei-nummer "Passwort" fehlt |
There was a problem hiding this comment.
Rückgängig gemacht in commit 417a5e9 weil es nicht damit klar kam, dass in der ersten Zeile ein BOM steht.
2 participants
rebased #5