Codecov Report

Merging #6499 into master will decrease coverage by 0.15%.
The diff coverage is 91.17%.

@@            Coverage Diff             @@
##           master    #6499      +/-   ##
==========================================
- Coverage   88.26%   88.10%   -0.16%     
==========================================
  Files         144      144              
  Lines      159516   156786    -2730     
==========================================
- Hits       140803   138144    -2659     
+ Misses      18713    18642      -71     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b146e01...b7fc5f6. Read the comment docs.

Thank you. Allowing a lambda for the global 'quickfixtextfunc' is inconsistent with other global options that support only global functions, or am I wrong on this?

Thank you very much for working on this.

I can confirm that you can now use a lambda, and even a funcref. Tested in 2 different plugins; both work as expected.

If an empty list is returned, then the default format is used to display all the entries.

This works as documented.

If an item in the returned list is an empty string, then the default format is used to display
the corresponding entry.

However, this doesn't seem to work:

set efm=%f
fu Func()
    call setqflist([], ' ', {
        \ 'lines': v:oldfiles,
        \ 'efm': '%f',
        \ 'quickfixtextfunc': 'Qftf',
        \ })
    cw
endfu

fu Qftf(info)
    let qfl = getqflist(#{id: a:info.id, items: 0}).items
    let l = []
    for idx in range(a:info.start_idx - 1, a:info.end_idx - 1)
        let e = qfl[idx]
        if idx % 3 == 2
            call add(l, '')
        else
            call add(l, bufname(e.bufnr))
        endif
    endfor
    return l
endfu

call Func()

I would expect every third line to be formatted with the default format, which here is %f. IOW, I would expect to see a filename in the quickfix window. Instead, every third line is empty. Maybe I'm doing something wrong though. I'll try to re-read the documentation and make more tests.

Here's a shell one-liner to illustrate:

./src/vim -Nu NONE +'set efm=%f' +"call setqflist([], ' ', {'lines': v:oldfiles, 'efm': '%f', 'quickfixtextfunc': {d -> repeat([''], d.end_idx - d.start_idx + 1)}})" +cw

Can confirm that the latest commit fixes the issue; thank you again.