diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-26 19:58:21 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-26 19:58:21 -0500 |
| commit | 7eece407772d7c5cfba93ba914439094f0d9fbf2 (patch) | |
| tree | 4b25307c895ab134312cd31141c48c0aea3c530e /.github/workflows | |
| parent | 5c8d908a943470e3e4738c090cf8eaa1deee5a1f (diff) | |
| download | org-drill-7eece407772d7c5cfba93ba914439094f0d9fbf2.tar.gz org-drill-7eece407772d7c5cfba93ba914439094f0d9fbf2.zip | |
refactor: dedupe presenters, group defcustoms, and fill in docstrings
A cleanup pass over org-drill internals, squashed from the refactor/wave3-cleanup branch. No behavior change. Each step kept the existing tests green and added its own.
I shared two duplicated helpers across the language card getters: org-drill--read-property-string and org-drill--face-from-alist.
I factored the cloze body-scan out of the two multicloze presenters into org-drill--cloze-body-bounds, org-drill--count-cloze-matches, and org-drill--hide-cloze-by-index, so each presenter just picks which indices to hide.
I pulled the presenter resolution and the four-way result classification out of org-drill-entry-f into org-drill--resolve-presenter and org-drill--classify-presentation-result, untangling the pivot of every drill iteration.
I split the 37 defcustoms (and the three cloze faces) into four customize sub-groups (display, algorithm, session, leech) so customize-group org-drill is navigable. There's no leitner group because the Leitner settings are defvars.
I documented the 22 defuns that had no docstring, rewrote the corrupted org-drill-presentation-prompt-in-mini-buffer docstring, and switched eleven docstrings to the imperative "Return" (issue #2).
Diffstat (limited to '.github/workflows')
0 files changed, 0 insertions, 0 deletions
