From a9a4d8c7148c115a242a7b35d16dd536f9c0c700 Mon Sep 17 00:00:00 2001 From: Craig Jennings Date: Sat, 16 May 2026 02:48:18 -0500 Subject: refactor(custom-editing): five hygiene fixes from the module-by-module re-review - Guard `cj/duplicate-line-or-region' when COMMENT is non-nil but the current mode has no `comment-start' (e.g. fundamental-mode). Previously the function silently produced malformed output via `comment-region'; now it signals a clear `user-error'. - Factor the `find-file' advice install in external-open.el into `cj/external-open-install-advice'. Same idempotent shape (remove-then-add) but the intent is named. - Add `cj/--validate-decoration-char' in custom-comments.el and wire it into all six divider / border / box helpers. Rejects multi-char strings, empty strings, and control characters like newline/tab that would corrupt subsequent `M-q' flows. Updated the five nil-decoration ERT tests from `:type 'wrong-type-argument' (the old crash signal from `string-to-char' on nil) to `:type 'user-error', since the validator produces a clear message instead of a deep crash. - Extract `cj/--require-spell-checker' in flyspell-and-abbrev.el. Both `cj/flyspell-toggle' and `cj/flyspell-then-abbrev' now call the shared helper; the checker list lives in `cj/--spell-checker-executables', so adding nuspell or any other checker is a one-line edit. - Preserve trailing newlines in custom-ordering output. Both `cj/--arrayify' and `cj/--unarrayify' now detect a trailing newline on the input region and re-append it to the result, matching the pattern custom-text-enclose.el already uses. --- tests/test-custom-comments-comment-inline-border.el | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'tests/test-custom-comments-comment-inline-border.el') diff --git a/tests/test-custom-comments-comment-inline-border.el b/tests/test-custom-comments-comment-inline-border.el index ca2bef06..78e86035 100644 --- a/tests/test-custom-comments-comment-inline-border.el +++ b/tests/test-custom-comments-comment-inline-border.el @@ -190,10 +190,12 @@ Returns the buffer string for assertions." :type 'error)) (ert-deftest test-inline-border-elisp-nil-decoration () - "Should error when decoration-char is nil." + "Should error when decoration-char is nil. +The decoration-char validator signals `user-error' for any non-printable +or non-single-character input." (should-error (test-inline-border-at-column 0 ";;" "" nil "Header" 70) - :type 'wrong-type-argument)) + :type 'user-error)) (ert-deftest test-inline-border-elisp-non-integer-length () "Should error when length is not an integer." -- cgit v1.2.3