Eintragsdetails ansehen

IDProjektKategorieSichtbarkeitZuletzt aktualisiert
0002355EresseaLERNE/LEHREöffentlich2017-09-04 16:07
ReporterEnnoBearbeitung durchEnno 
PrioritätnormalAuswirkungkleinerer FehlerReproduzierbarnicht getestet
Status erledigtLösungerledigt 
Produktversion3.13 
Zielversion3.13Behoben in Version3.14 
Zusammenfassung0002355: Akademie-assert in LEHRE
Beschreibung

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

Schritte zur Reproduktion

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

Partei0
SpielE2
Report1037

Notizen / Dateien

Enno

Enno

2017-08-06 21: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 21:27

Administrator   ~0007390

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

Solthar

Solthar

2017-09-02 03:33

Entwickler   ~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 16:07

Administrator   ~0007463

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

Eintrags-Historie

Änderungsdatum Benutzername Feld Änderung
2017-08-06 11:10 Enno Neuer Eintrag
2017-08-06 21:07 Enno Bearbeitung durch => Enno
2017-08-06 21:07 Enno Status neu => zugewiesen
2017-08-06 21:12 Enno Notiz hinzugefügt: 0007389
2017-08-06 21:27 Enno Notiz hinzugefügt: 0007390
2017-09-02 03:33 Solthar Notiz hinzugefügt: 0007454
2017-09-04 16:07 Enno Status zugewiesen => erledigt
2017-09-04 16:07 Enno Lösung offen => erledigt
2017-09-04 16:07 Enno Behoben in Version => 3.14
2017-09-04 16:07 Enno Notiz hinzugefügt: 0007463