From 720d6597bf6751ee6d75bb224dd468aa2a78d7ee Mon Sep 17 00:00:00 2001
From: Craig Jennings <c@cjennings.net>
Date: Wed, 29 Apr 2026 10:06:33 -0500
Subject: 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.
---
 org-drill.el | 1 +
 1 file changed, 1 insertion(+)

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)
-- 
cgit v1.2.3