I think changes in commit 94dec95 (bug#69943) broke `widget-move' in a 
customize buffer when trying to move to the first widget in a buffer when that 
first widget starts at the second character in the buffer.  Here's some code to 
reproduce (tested in IELM):

(let* ((first-wid (progn (widget-forward 1) (point))))
  (print (format "First widget at %S" first-wid))
  (cl-assert (and (numberp first-wid) (>= first-wid 1)))
  (with-current-buffer (customize-group 'editing)
    (narrow-to-region (1- first-wid) (point-max))
    (goto-char (point-min))
    (widget-forward 1)
    (print (format "Expected to be at %S, point=%S" first-wid (point)))))

On my Emacs I get:

"First widget at 33"

"Expected to be at 33, point=32"

I think this happens because of this code near the end of `widget-move' (which 
is called by `widget-forward'):

    (let ((new (widget-tabable-at)))
      (while (and (eq (widget-tabable-at) new) (not (bobp)))
        (backward-char)))
    (unless (bobp) (forward-char)))

In my test case, as we enter the while loop point is at the start of the first 
widget (AKA "new").  We are not yet at beginning of buffer, so it moves point 
back one character.  Now we are at beginning of buffer, but that doesn't 
matter: the `eq' test fails first, and the loop ends.

However, the `forward-char' never runs because we are indeed at beginning of 
buffer now.  I think this `forward-char' should have been run to put point back 
on the start of the widget.

Bug#70594 also recently modified code around here, but I don't *think* that's 
relevant.

In case you're wondering, this comes up because I use link-hint[1], which 
narrows a customize buffer in exactly the way shown above.

[1]: https://github.com/noctuid/link-hint.el

Please let me know if I can provide any more information!

Best regards,
Dale


In GNU Emacs 31.0.50 (build 1, aarch64-apple-darwin23.6.0, NS
appkit-2487.70 Version 14.6.1 (Build 23G93)) of 2024-09-01 built on
mymacRepository revision: 0f98515cec1298bc82c589b5f8565503900b806d
Repository branch: master
Windowing system distributor 'Apple', version 10.3.2487
System Description:  macOS 14.6.1

Configured using:
 'configure --without-x --with-xwidgets --with-json --with-tree-sitter 
--without-imagemagick --with-xpm --with-jpeg --with-tiff --with-gif --with-png 
--with-rsvg --with-webp --with-sqlite3 --with-cairo --with-xml2 --with-gnutls 
--with-zlib --with-modules --with-threads --with-native-compilation --with-ns 
--enable-ns-self-contained 'CFLAGS= -D_DARWIN_UNLIMITED_SELECT=1 
-DFD_SETSIZE=10240''




  • bug#72995:... Dale
    • bug#7... Bug reports for GNU Emacs, the Swiss army knife of text editors
    • bug#7... Eli Zaretskii

Reply via email to