From: Kyle Meyer <firstname.lastname@example.org>
To: Samuel Wales <email@example.com>
Subject: Re: [bug] args out of range upon meta-return in body of list item
Date: Wed, 05 Aug 2020 00:51:26 -0400 [thread overview]
Message-ID: <firstname.lastname@example.org> (raw)
Samuel Wales writes:
> recent maint on emacs 25.
> i know this is an imperfect bug report with no mce that you can
> probably repro. but perhaps it can trigger an idea that could lead
> directly to a fix. my brain and computer limitations cannot do better
> at this time.
The example, instructions, and backtrace you provided are all very
> when i run m-ret at the caret i get this bt. my intent is to make a
> new list item containing prefer as its header.
> 1) [ ] towels
> 1) [ ] 12 very large (asdfasdf asdfasdf) solid color,
> like the ones i use, thick -- asdafsdf -- ^prefer
> to get them slightly wider
> - not any ...
> - not any pattern (something like different color on
> end might be ok, dunno) or anything garish
> 2) [ ] 6 dark aasdfa dfnkajs dkfaskdf kasdn
> fkasdnkfanksdfn kasdfn
> 2) asdfasdfasdf
> Debugger entered--Lisp error: (args-out-of-range -1 3)
> replace-match("1) " nil nil nil 1)
> #[771 ...
> funcall-interactively(org-insert-item nil)
> funcall-interactively(org-meta-return nil)
> call-interactively(org-meta-return nil nil)
I don't have more time to spend on this at the moment, but some quick
* The error you show happens on maint, but doesn't on master. It
looks like it went away with 07a4a7286 (list: Fix regression when
inserting items, 2020-07-06), which was prompted by this report:
I think this can be cherry picked to maint.
* Although the error is gone, the result still looks off. In
particular, the "2) [ ] 6 dark" line loses its bullet.
I suspect that the remaining issue is related to the error you show
above. Here's a bit of context from org-list-struct-apply-struct:
;; a. Replace bullet
(unless (equal old-bul new-bul)
(replace-match new-bul nil nil nil 1))
;; b. Replace checkbox.
((equal (match-string 3) new-box))
((and (match-string 3) new-box)
(replace-match new-box nil nil nil 3))
The code downstream of (looking-at ...) assumes a match, but it seems
that's not valid, even after 07a4a7286. The patch below guards the
downstream code with (when (looking-at ...)) and fixes this particular
"lost bullet" case I mentioned above.
The change doesn't make any tests fail, but I'd need to look into the
code a bit more to be comfortable with the change. (For instance, is
the fact that there isn't a match here a sign that something's going
wrong upstream?) Also, the patch should include a test that is
distilled from your example.
diff --git a/lisp/org-list.el b/lisp/org-list.el
index 727d89c36..61d5def5b 100644
@@ -1866,26 +1866,26 @@ (defun org-list-struct-apply-struct (struct old-struct)
(org-list-get-bullet item struct)))
(old-bul (org-list-get-bullet item old-struct))
(new-box (org-list-get-checkbox item struct)))
- (looking-at org-list-full-item-re)
- ;; a. Replace bullet
- (unless (equal old-bul new-bul)
- (replace-match new-bul nil nil nil 1))
- ;; b. Replace checkbox.
- ((equal (match-string 3) new-box))
- ((and (match-string 3) new-box)
- (replace-match new-box nil nil nil 3))
- ((match-string 3)
- (looking-at ".*?\\([ \t]*\\[[ X-]\\]\\)")
- (replace-match "" nil nil nil 1))
- (t (let ((counterp (match-end 2)))
- (goto-char (if counterp (1+ counterp) (match-end 1)))
- (insert (concat new-box (unless counterp " "))))))
- ;; c. Indent item to appropriate column.
- (unless (= new-ind old-ind)
- (delete-region (goto-char (point-at-bol))
- (progn (skip-chars-forward " \t") (point)))
- (indent-to new-ind)))))))
+ (when (looking-at org-list-full-item-re)
+ ;; a. Replace bullet
+ (unless (equal old-bul new-bul)
+ (replace-match new-bul nil nil nil 1))
+ ;; b. Replace checkbox.
+ ((equal (match-string 3) new-box))
+ ((and (match-string 3) new-box)
+ (replace-match new-box nil nil nil 3))
+ ((match-string 3)
+ (looking-at ".*?\\([ \t]*\\[[ X-]\\]\\)")
+ (replace-match "" nil nil nil 1))
+ (t (let ((counterp (match-end 2)))
+ (goto-char (if counterp (1+ counterp) (match-end 1)))
+ (insert (concat new-box (unless counterp " "))))))
+ ;; c. Indent item to appropriate column.
+ (unless (= new-ind old-ind)
+ (delete-region (goto-char (point-at-bol))
+ (progn (skip-chars-forward " \t") (point)))
+ (indent-to new-ind))))))))
;; 1. First get list of items and position endings. We maintain
;; two alists: ITM-SHIFT, determining indentation shift needed
;; at item, and END-LIST, a pseudo-alist where key is ending
next prev parent reply other threads:[~2020-08-05 4:52 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-08-04 22:41 [bug] args out of range upon meta-return in body of list item Samuel Wales
2020-08-05 4:51 ` Kyle Meyer [this message]
2020-08-05 5:18 ` Samuel Wales
2020-08-16 9:45 ` Nicolas Goaziou
2020-08-16 15:36 ` Kyle Meyer
2020-08-16 22:12 ` Samuel Wales
2020-08-20 20:55 ` Samuel Wales
2020-08-20 21:16 ` Nicolas Goaziou
2020-08-20 21:47 ` Samuel Wales
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
List information: https://www.orgmode.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).