aboutsummaryrefslogtreecommitdiff
path: root/.gitignore
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-05-03 06:56:10 -0500
committerCraig Jennings <c@cjennings.net>2026-05-03 06:56:10 -0500
commit6936f5b60b61dcf41cbda75813d7f259733eedf2 (patch)
treef058960d56232d8629c6833607a0df287a44063d /.gitignore
parente2f373ff05116d4b2ed48d4ddcaea8c47846f105 (diff)
downloaddotemacs-6936f5b60b61dcf41cbda75813d7f259733eedf2.tar.gz
dotemacs-6936f5b60b61dcf41cbda75813d7f259733eedf2.zip
fix(line-paragraph): join-line-or-region strands space on next line
The region branch's `(while (< (point) end) (join-line 1))` ran one iteration too many. After the final in-region join, point sat just before the end marker, so the loop fired once more. That extra `join-line 1` consumed the next line's preceding newline and replaced it with a space. Then `(goto-char end)` + `(newline)` reinserted a newline at the original end position, before the inserted space, so the space ended up stranded at BOL of the next line. I replaced the position-based loop with `count-lines` + `dotimes` to do exactly the right number of joins. I also swapped the trailing `(newline)` for `(forward-line 1)`. The bullet-list use case now lands directly on the next existing line with no blank gap. The trailing-newline change ripples to `cj/join-paragraph` (which delegates here), so paragraphs now also stop adding a trailing newline when the input lacks one. `require-final-newline` handles file-end discipline on save anyway. I added 3 new tests that fail against the old loop and pass against the fix. I also updated 11 existing tests whose assertions baked in the old trailing-newline behavior. While in there I wrapped the `cj/custom-keymap` defvar stub in `eval-and-compile` in both test files. The bare defvar wasn't evaluated at byte-compile time, so the `require` of `custom-line-paragraph` would hit a void symbol when the validate-el hook ran.
Diffstat (limited to '.gitignore')
0 files changed, 0 insertions, 0 deletions