fix(previewer): improve file_maker line splitting and timeouts

[jamestrew]

Previous method of splitting files into lines and incorporating a timer is sub-optimal in two ways

• not that fast
• since it's only checking for timeouts between each line split, huge lines can and does lead to exceeding the timeout potentially significantly (Freezes on single line files, ignoring preview timeout #3032)

By avoiding using vim.gsplit, we can avoid some overheads it needs to account for patterns and options, and we can check for timeouts more incrementally (every 200 bytes) to avoid getting caught up in huge singular lines.

Based on quick and dirty benchmarks against large-ish files in the issue above, I'm seeing about a 4-5x speed increase.

[clason]

@jamestrew rebase?

[Conni2461]

thanks :)

[clason]

Sorry for not testing this earlier, but this triggers an error with file previews (on Nvim nightly):

vim.schedule callback: ...m/lazy/telescope.nvim/lua/telescope/previewers/utils.lua:267: attempt to compare boolean with number
stack traceback:
        ...m/lazy/telescope.nvim/lua/telescope/previewers/utils.lua:267: in function 'timed_split_lines'
        ...scope.nvim/lua/telescope/previewers/buffer_previewer.lua:189: in function 'fn'
        [string "vim/_core/editor"]:404: in function <[string "vim/_core/editor"]:403>

This is with opts.preview.timeout = false, which is a valid option to not time out.

Workaround: Set opts.preview.timeout = math.huge to disable the timeout.

[Conni2461]

does this help? #3662

[clason]

Yes! (I would also have considered adapting the documentation, but this is less breaking.)