View Issue Details

IDProjectCategoryView StatusLast Update
0002355EresseaLERNE/LEHREpublic2017-12-05 18:48
ReporterEnno Assigned ToEnno  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version3.13.0 
Target Version3.13.0Fixed in Version3.14.0 
Summary0002355: Akademie-assert in LEHRE
Description

eressea: /home/eressea/eressea/git/src/study.c:486: teach_cmd: Assertion `academy % 30 == 0' failed.

Steps To Reproduce

Das Preview ist heute mit der o.g. assertion abgeschmiert. Ich vermute, dass die Änderungen von @Solthar damit zu tun haben könnten.

TagsNo tags attached.
Faction0
GameE2
Report1037

Activities

Enno

Enno

2017-08-06 19:12

administrator   ~0007389

Die Code-Zeile heisst: assert(academy % STUDYDAYS == 0);

Der Wert von academy ist 230. Leider ist aus den Kommentaren nicht ersichtlich, warum dieses assert hier stimmt. Das assert ist gerade erst neu zu der Funktion hinzu gekommen. @Solthar, hast Du da noch eine Idee, was Du damit meintest?

Enno

Enno

2017-08-06 19:27

administrator   ~0007390

Ich werde das assert für die heutige Preview-Auswertung erst einmal entfernen.

Solthar

Solthar

2017-09-02 01:33

developer   ~0007454

Ich hatte da eine falsche Annahme gemacht, die aufgrund eines alten Bugs bestand. Korrektur: https://github.com/eressea/server/pull/727

Enno

Enno

2017-09-04 14:07

administrator   ~0007463

Danke für den PR, ich glaube, das ist jetzt okay.

Issue History

Date Modified Username Field Change
2017-08-06 09:10 Enno New Issue
2017-08-06 19:07 Enno Assigned To => Enno
2017-08-06 19:07 Enno Status new => assigned
2017-08-06 19:12 Enno Note Added: 0007389
2017-08-06 19:27 Enno Note Added: 0007390
2017-09-02 01:33 Solthar Note Added: 0007454
2017-09-04 14:07 Enno Status assigned => resolved
2017-09-04 14:07 Enno Resolution open => fixed
2017-09-04 14:07 Enno Fixed in Version => 3.14.0
2017-09-04 14:07 Enno Note Added: 0007463
2017-12-05 18:48 Enno Status resolved => closed