Use character position for ranges in treesit_sync_visible_region

* src/treesit.c (treesit_sync_visible_region): Use character position
instead of byte position when comparing to ranges, because the
ranges are in character position.
This commit is contained in:
Yuan Fu 2025-02-22 23:24:38 -08:00
parent 9c1d13c89a
commit 2dbf7d0b1b
No known key found for this signature in database
GPG key ID: 56E19BC57664A442

View file

@ -1072,12 +1072,12 @@ treesit_sync_visible_region (Lisp_Object parser)
ptrdiff_t beg = XFIXNUM (XCAR (range));
ptrdiff_t end = XFIXNUM (XCDR (range));
if (end <= visible_beg)
/* Even the end is before visible_beg, discard this range. */
if (end <= BUF_BEGV (buffer))
/* Even the end is before BUF_BEGV (buffer), discard this range. */
new_ranges_head = XCDR (new_ranges_head);
else if (beg >= visible_end)
else if (beg >= BUF_ZV (buffer))
{
/* Even the beg is after visible_end, discard this range and all
/* Even the beg is after BUF_ZV (buffer), discard this range and all
the ranges after it. */
if (NILP (prev_cons))
new_ranges_head = Qnil;
@ -1090,10 +1090,10 @@ treesit_sync_visible_region (Lisp_Object parser)
/* At this point, the range overlaps with the visible portion of
the buffer in some way (in front / in back / completely
encased / completely encases). */
if (beg < visible_beg)
XSETCAR (range, make_fixnum (visible_beg));
if (end > visible_end)
XSETCDR (range, make_fixnum (visible_end));
if (beg < BUF_BEGV (buffer))
XSETCAR (range, make_fixnum (BUF_BEGV (buffer)));
if (end > BUF_ZV (buffer))
XSETCDR (range, make_fixnum (BUF_ZV (buffer)));
}
prev_cons = lisp_ranges;
}
@ -1103,8 +1103,8 @@ treesit_sync_visible_region (Lisp_Object parser)
options, so just throw the towel: just give the parser a zero
range. (Perfect filling!!) */
if (NILP (new_ranges_head))
new_ranges_head = Fcons (Fcons (make_fixnum (visible_beg),
make_fixnum (visible_beg)),
new_ranges_head = Fcons (Fcons (make_fixnum (BUF_BEGV (buffer)),
make_fixnum (BUF_BEGV (buffer))),
Qnil);
XTS_PARSER (parser)->last_set_ranges = new_ranges_head;