diff options
| author | Craig Jennings <c@cjennings.net> | 2026-04-29 10:06:33 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-04-29 10:06:33 -0500 |
| commit | 720d6597bf6751ee6d75bb224dd468aa2a78d7ee (patch) | |
| tree | c8df5ced5b90e08eaef22ab29e9e531d631b1f9d | |
| parent | 5c68f1ed1f59075ac1d4c0262884a5cd7aa25ef6 (diff) | |
| download | org-drill-720d6597bf6751ee6d75bb224dd468aa2a78d7ee.tar.gz org-drill-720d6597bf6751ee6d75bb224dd468aa2a78d7ee.zip | |
fix: increment totaln on Simple8 failure path
The Simple8 failure branch was missing (cl-incf totaln) while SM2 and SM5 both increment total-repeats on failure. After this change, DRILL_TOTAL_REPEATS counts every review attempt regardless of which scheduling algorithm produced it, including failures.
Going-forward only. Historical totaln values for Simple8 failures stay under-counted by one. Correct counting starts with the next failed review.
Paired with the test commit 5c68f1e, which captured the new expected behavior first. Full suite at 214 of 214.
| -rw-r--r-- | org-drill.el | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/org-drill.el b/org-drill.el index 892bc29..4d7479c 100644 --- a/org-drill.el +++ b/org-drill.el @@ -1340,6 +1340,7 @@ See the documentation for `org-drill-get-item-data' for a description of these." (cond ((<= quality org-drill-failure-quality) (cl-incf failures) + (cl-incf totaln) (setf repeats 0 next-interval -1)) ((or (zerop repeats) |