Scale hp autoseed #1029
Scale hp autoseed #1029
Conversation
fix resistance tests
fix null-pointer crash
some mild refactoring, but no real changes.
https://bugs.eressea.de/view.php?id=2980 Ausserdem: Flammenschwerter vor Laenschwertern bevorzugen.
Co-authored-by: Enno Rehling <enno@eressea.de>
Co-authored-by: Enno Rehling <enno@eressea.de>
Sehr schön, sogar mit Tests!
* narrow down failing test' * on failure, print the filename
let install script use cmake --install
this resolves issue eressea#1026
Co-authored-by: Enno Rehling <enno@eressea.de>
* upgrade to python3 * get rid of python2 compatibility --------- Co-authored-by: Enno Rehling <enno@eressea.de>
|
Dafür konnte man keinen Test schreiben? |
|
Wie sich herausstellt konnte man. Gute Sache. |
src/races/races.test.c
Outdated
|
|
||
| #include <CuTest.h> | ||
|
|
||
| #include <stb_ds.h> |
There was a problem hiding this comment.
Dieser header wird glaube ich nicht gebraucht.
src/races/races.test.c
Outdated
|
|
||
| #include <stb_ds.h> | ||
|
|
||
| #include <stdbool.h> |
There was a problem hiding this comment.
ditto - dein Test benutzt keine booleans.
| @@ -124,31 +124,41 @@ local mysets = { | |||
| }, | |||
| } | |||
|
|
|||
| EQUIP_SKILLS = 2 | |||
There was a problem hiding this comment.
Es wäre besser, wir würden solche Konstanten aus C exportieren - die könnten sich ja mal ändern, und zwei Definitionien synchronisiert zu behalten, ist immer doof.
There was a problem hiding this comment.
Ich weiß nur nicht wie. Mache ich das in bindings, so wie du es für directions gemacht hast oder definiere ich dafür irgendwie was eigenes??
There was a problem hiding this comment.
Ja, was in bindings.c mit den directions gemacht wird, ist ein gutes Beispiel, glaube ich.
HP of seeded units where not scaled rendering them wounded