Skip to content

Conversation

@stm2
Copy link
Contributor

@stm2 stm2 commented Sep 25, 2023

HP of seeded units where not scaled rendering them wounded

ennorehling and others added 29 commits August 27, 2023 20:38
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>
* narrow down failing test'

* on failure, print the filename
let install script use cmake --install
Co-authored-by: Enno Rehling <enno@eressea.de>
* upgrade to python3

* get rid of python2 compatibility

---------

Co-authored-by: Enno Rehling <enno@eressea.de>
@ennorehling
Copy link
Member

Dafür konnte man keinen Test schreiben?

@stm2
Copy link
Contributor Author

stm2 commented Sep 25, 2023

Wie sich herausstellt konnte man. Gute Sache.


#include <CuTest.h>

#include <stb_ds.h>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dieser header wird glaube ich nicht gebraucht.


#include <stb_ds.h>

#include <stdbool.h>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ditto - dein Test benutzt keine booleans.

},
}

EQUIP_SKILLS = 2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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??

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ja, was in bindings.c mit den directions gemacht wird, ist ein gutes Beispiel, glaube ich.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants