GNU bug report logs - #68899
Treesitter's forward-sexp-function

Previous Next

Package: emacs;

Reported by: João Távora <joaotavora <at> gmail.com>

Date: Fri, 2 Feb 2024 21:49:01 UTC

Severity: normal

Fixed in version 30.1

Done: Yuan Fu <casouri <at> gmail.com>

Bug is archived. No further changes may be made.

Full log

View this message in rfc822 format

From: João Távora <joaotavora <at> gmail.com>
To: 68899 <at> debbugs.gnu.org, casouri <at> gmail.com
Subject: bug#68899: Treesitter's forward-sexp-function
Date: Sat, 3 Feb 2024 00:42:44 +0000

On Fri, Feb 2, 2024 at 9:47 PM João Távora <joaotavora <at> gmail.com> wrote:

> Shouldn't the intervening treesit-end-of-thing go to the end of the
> current thing?, i.e. to the '('?  I think it should, at least judging
> from its docstring, and this patch makes that happen:
>
> -              (setq pos (funcall advance (if (> arg 0) next prev)))
> +              (setq pos (funcall advance (or (if (> arg 0) next prev)
> +                                             parent)))
>
> This doesn't seem to break tests, assuming it's not in these 3 there
> were skipped because I don't have the grammar installed.

Despite that, I think it's still wrong :-/ Now it moves too much,
i.e. it never stops moving.
This bug report was last modified 1 year and 102 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.