View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002355 | Eressea | LERNE/LEHRE | public | 2017-08-06 09:10 | 2017-12-05 18:48 |
| Reporter | Enno | Assigned To | Enno | ||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | 3.13.0 | ||||
| Target Version | 3.13.0 | Fixed in Version | 3.14.0 | ||
| Summary | 0002355: 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. | ||||
| Tags | No tags attached. | ||||
| Faction | 0 | ||||
| Game | E2 | ||||
| Report | 1037 | ||||
|
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? |
|
|
Ich werde das assert für die heutige Preview-Auswertung erst einmal entfernen. |
|
|
Ich hatte da eine falsche Annahme gemacht, die aufgrund eines alten Bugs bestand. Korrektur: https://github.com/eressea/server/pull/727 |
|
|
Danke für den PR, ich glaube, das ist jetzt okay. |
|
| 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 |