Extra terminal cells styled after end of file

[git-bruh]

Problem

For markdown files containing a hyperlink at the end (no newlines), the underlining style overflows over to the empty cells:

I've not investigated much but there might be an off by one error around this logic (as lua is one indexed): https://github.com/martanne/vis/blob/6f537f300b0b4d7e4a729e28cce7be9241e829a3/lua/vis-std.lua#L67

Seems similar to https://github.com/martanne/vis/issues/1147

Steps to reproduce

No response

vis version (vis -v)

vis 95bf9f5-dirty +curses +lua +lpeg

Terminal name/version

foot 1.16.2

$TERM environment variable

foot

[acidrums4]

Not sure if it's related, but if for example I set a different background color for a certain type of string in the theme file (say, lexers.STYLE_FUNCTION), the background color will overflow over the neighboring empty cells too.

[git-bruh]

diff --git a/view.c b/view.c
index df7906b..6a723a4 100644
--- a/view.c
+++ b/view.c
@@ -1432,10 +1432,10 @@ void view_style(View *view, enum UiStyle style, size_t start, size_t end) {
        col++;

    do {
-       while (pos <= end && col < width) {
+       while (pos < end && col < width) {
            pos += line->cells[col].len;
            view->ui->style_set(view->ui, &line->cells[col++], style);
        }
        col = 0;
-   } while (pos <= end && (line = line->next));
+   } while (pos < end && (line = line->next));
 }

This works, the end position passed from lua is equal to view->end, and we should only iterate till view->end - 1

Otherwise we reach view->end which has empty columns, and adding line->cells[col].len to pos keeps the loop running as length of an empty cell is 0

@acidrums4 could you see if this fixes your issue as well?

EDIT: nvm, this breaks regular highlighting at the last character as well, in case there is more text after it , eg [a](b) xyz, the ) isn't highlighted

[git-bruh]

This might be a more reasonable fix:

diff --git a/lua/vis-std.lua b/lua/vis-std.lua
index 8b5715b..1ccef0d 100644
--- a/lua/vis-std.lua
+++ b/lua/vis-std.lua
@@ -75,7 +75,7 @@ vis.events.subscribe(vis.events.WIN_HIGHLIGHT, function(win)
    local data = win.file:content(viewport)
    local token_styles = lexer._TOKENSTYLES
    local tokens = lexer:lex(data, 1)
-   local token_end = lex_start + (tokens[#tokens] or 1) - 1
+   local token_end = lex_start + ((tokens[#tokens] and tokens[#tokens] - 1) or 1) - 1

    for i = #tokens - 1, 1, -2 do
        local token_start = lex_start + (tokens[i-1] or 1) - 1

Not sure if I'm missing any edge case here