Hi!
This is my proposal for fixing #12718
I could have done something even more robust, like
if empty(glob('gdb'))
file gdb
elseif empty(glob(' Termdebug-gdb-console'))
file Termdebug-gdb-console
else
Echoerr("You have a file/folder named 'gdb' or 'Termdebug-gdb-console'. Termdebug cannot start. Please rename them. ")
endif
Shall the same if-then-else
condition be placed in other lines of the form file <hard-coded name>
?
https://github.com/vim/vim/pull/14792
(1 file)
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Thanks, are there other places where this could clash? I think the warning approach is good, although you have a leading white space inside your glob()
function. Can you fix this up and add a test to https://github.com/vim/vim/blob/master/src/testdir/test_termdebug.vim ?
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Sure thing! Yes, there are some other places where this could happen: pretty much everywhere where file
command is used. You could /file
in the source and see that you have it in other 2-3 places. I can use the same solution there as well. Regarding the tests: I never wrote a test for vimscripts, if there is some guideline it would be great to read! Finally: I will try to fix it during the weekend if I will find some time or during next week. Would that work?
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
that sounds perfect. Regarding the tests, just check the existing termdebug test. It's not so hard, you basically need to create a gdb
file and then check that either the plugin is not loaded (so you are still in a clean state, e.g. only one single buffer opened), or check if you can see the error message in the messages list. Or something like this :) Thanks for looking into this!
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 2 commits.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
@k-takata commented on this pull request.
In runtime/pack/dist/opt/termdebug/plugin/termdebug.vim:
> @@ -415,7 +415,17 @@ func s:StartDebug_prompt(dict) let s:promptbuf = bufnr('') call prompt_setprompt(s:promptbuf, 'gdb> ') set buftype=prompt - file gdb + + if empty(glob('gdb')) + file gdb + elseif empty(glob(' Termdebug-gdb-console'))⬇️ Suggested change
- elseif empty(glob(' Termdebug-gdb-console')) + elseif empty(glob('Termdebug-gdb-console'))
Is this space intentional?
In runtime/pack/dist/opt/termdebug/plugin/termdebug.vim:
> @@ -415,7 +415,17 @@ func s:StartDebug_prompt(dict) let s:promptbuf = bufnr('') call prompt_setprompt(s:promptbuf, 'gdb> ') set buftype=prompt - file gdb + + if empty(glob('gdb')) + file gdb + elseif empty(glob(' Termdebug-gdb-console')) + file Termdebug-gdb-console⬇️ Suggested change
- file Termdebug-gdb-console + file Termdebug-gdb-console
wrong indent
In runtime/pack/dist/opt/termdebug/plugin/termdebug.vim:
> + Echoerr("You have a file/folder named 'gdb' + \ or 'Termdebug-gdb-console'. + \ Termdebug cannot start. Please rename them. ")⬇️ Suggested change
- Echoerr("You have a file/folder named 'gdb' - \ or 'Termdebug-gdb-console'. - \ Termdebug cannot start. Please rename them. ") + call s:Echoerr("You have a file/folder named 'gdb' + \ or 'Termdebug-gdb-console'. + \ Termdebug cannot start. Please rename them.")
In runtime/pack/dist/opt/termdebug/plugin/termdebug.vim:
> + Echoerr("You have a file/folder named Termdebug-asm-listing'. + \ Termdebug cannot start. Please rename such a file/folder. ")⬇️ Suggested change
- Echoerr("You have a file/folder named Termdebug-asm-listing'. - \ Termdebug cannot start. Please rename such a file/folder. ") + call s:Echoerr("You have a file/folder named Termdebug-asm-listing'. + \ Termdebug cannot start. Please rename such a file/folder.")
In runtime/pack/dist/opt/termdebug/plugin/termdebug.vim:
> + Echoerr("You have a file/folder named Termdebug-variables-listing'. + \ Termdebug cannot start. Please rename such a file/folder. ")⬇️ Suggested change
- Echoerr("You have a file/folder named Termdebug-variables-listing'. - \ Termdebug cannot start. Please rename such a file/folder. ") + call s:Echoerr("You have a file/folder named Termdebug-variables-listing'. + \ Termdebug cannot start. Please rename such a file/folder.")
In src/testdir/test_termdebug.vim:
> + call writefile(['This', 'is', 'a', 'test'], filename) + " Throw away the file once the test has done. + execute 'defer ' .. filename⬇️ Suggested change
- call writefile(['This', 'is', 'a', 'test'], filename) - " Throw away the file once the test has done. - execute 'defer ' .. filename + call writefile(['This', 'is', 'a', 'test'], filename, 'D')
You can use the 'D' flag.
In src/testdir/test_termdebug.vim:
> + call writefile(['This', 'is', 'a', 'test'], filename) + " Throw away the file once the test has done. + execute 'defer delete(filename)'⬇️ Suggested change
- call writefile(['This', 'is', 'a', 'test'], filename) - " Throw away the file once the test has done. - execute 'defer delete(filename)' + call writefile(['This', 'is', 'a', 'test'], filename, 'D')
You can use the 'D' flag.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
.. working on it. Please ignore all the pushes until I notify.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
Try now! :)
Are the tests enough?
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
@k-takata commented on this pull request.
In src/testdir/test_termdebug.vim:
> @@ -300,4 +300,21 @@ func Test_termdebug_mapping() %bw! endfunc +func Test_termdebug_bufnames() + " Test if user has filename/folders named gdb, Termdebug-gdb-console, + " etc. in the current directory + let filename = 'gdb' + let replacement_filename = 'Termdebug-gdb-console' + + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Throw away the file once the test has done. + execute 'Termdebug' + " A file named filename already exists in the working directory, + " hence you must call the newly created buffer differently + call WaitForAssert({-> assert_false(bufexists(filename))}) + call WaitForAssert({-> assert_true(bufexists(bureplacement_filename))})
Mistake?
⬇️ Suggested change- call WaitForAssert({-> assert_true(bufexists(bureplacement_filename))}) + call WaitForAssert({-> assert_true(bufexists(replacement_filename))})
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
Check it now!
(yes, the previous bureplacement_filename
was a typo!)
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
It seems some tests failed due to the following:
Failures:
From test_termdebug.vim:
Found errors in Test_termdebug_bufnames():
Caught exception in Test_termdebug_bufnames(): Vim(bwipeout):E517: No buffers were wiped out: bwipe! 12 @ command line..script /home/runner/work/vim/vim/src/testdir/runtest.vim[607]..function
Most likely it is due to lines 319 and 332 of [src/testdir/test_termdebug.vim](https://github.com/vim/vim/pull/14792/files#diff-07567861d8244d8d0078cb5d13045a8fb5d438ec20af2d2341f648ca6d3b05f7)
and it is not super clear to me how to solve it... any hints?
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
RunTheTest[57]..Test_termdebug_bufnames[20]..9_EndPromptDebug[9]..9_EndDebugCommon, line 4
The error occurs here:
https://github.com/vim/vim/blob/648aedf7c2e89ad2fb55e2f4d0c4aa81808acd50/runtime/pack/dist/opt/termdebug/plugin/termdebug.vim#L741
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
> @@ -300,4 +300,42 @@ func Test_termdebug_mapping() %bw! endfunc +func Test_termdebug_bufnames() + " Test if user has filename/folders named gdb, Termdebug-gdb-console, + " etc. in the current directory + let g:termdebug_config = {} + let g:termdebug_config['use_prompt'] = 1 + let filename = 'gdb' + let replacement_filename = 'Termdebug-gdb-console' + + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Throw away the file once the test has done. + execute 'Termdebug'⬇️ Suggested change
- execute 'Termdebug' + Termdebug
No need to use "execute" if you don't use parameters.
In src/testdir/test_termdebug.vim:
> + sleep 2 + " A file named filename already exists in the working directory, + " hence you must call the newly created buffer differently + call WaitForAssert({-> assert_false(bufexists(filename))}) + call WaitForAssert({-> assert_true(bufexists(replacement_filename))}) + %bw! + + " Check if error message is in :message + " Delay for securing that Termdebug is shutoff + sleep 1 + let g:termdebug_config['disasm_window'] = 1 + let filename = 'Termdebug-asm-listing' + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Check only the head of the error message + let error_message = "You have a file/folder named '" .. filename .. "'" + execute 'Termdebug'⬇️ Suggested change
- execute 'Termdebug' + Termdebug
In src/testdir/test_termdebug.vim:
> + " hence you must call the newly created buffer differently + call WaitForAssert({-> assert_false(bufexists(filename))}) + call WaitForAssert({-> assert_true(bufexists(replacement_filename))}) + %bw! + + " Check if error message is in :message + " Delay for securing that Termdebug is shutoff + sleep 1 + let g:termdebug_config['disasm_window'] = 1 + let filename = 'Termdebug-asm-listing' + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Check only the head of the error message + let error_message = "You have a file/folder named '" .. filename .. "'" + execute 'Termdebug' + sleep 2 + call WaitForAssert({->assert_true(stridx(execute('messages'), error_message) != -1 )})⬇️ Suggested change
- call WaitForAssert({->assert_true(stridx(execute('messages'), error_message) != -1 )}) + call WaitForAssert({-> assert_notequal(-1, stridx(execute('messages'), error_message))})
In src/testdir/test_termdebug.vim:
> + unlet g:termdebug_config + unlet filename + unlet replacement_filename + unlet error_message⬇️ Suggested change
- unlet g:termdebug_config - unlet filename - unlet replacement_filename - unlet error_message + unlet g:termdebug_config
No need to unlet local variables.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
Working on it... ignore push!
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
Ready for review. :)
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
It seems that the following patch fixes the CI failure:
--- a/src/testdir/test_termdebug.vim +++ b/src/testdir/test_termdebug.vim @@ -317,10 +317,9 @@ func Test_termdebug_bufnames() call WaitForAssert({-> assert_false(bufexists(filename))}) call WaitForAssert({-> assert_true(bufexists(replacement_filename))}) quit! + call WaitForAssert({-> assert_equal(1, winnr('$'))})
" Check if error message is in :message
- " Delay for securing that Termdebug is shutoff - sleep 1 let g:termdebug_config['disasm_window'] = 1 let filename = 'Termdebug-asm-listing' call writefile(['This', 'is', 'a', 'test'], filename, 'D') @@ -328,10 +327,11 @@ func Test_termdebug_bufnames() let error_message = "You have a file/folder named '" .. filename .. "'" Termdebug sleep 2 - call WaitForAssert({->assert_notequal(-1, stridx(execute('messages'), error_message))})
+ call WaitForAssert({-> assert_notequal(-1, stridx(execute('messages'), error_message))})
quit! - wincmd t + wincmd b wincmd q + call WaitForAssert({-> assert_equal(1, winnr('$'))}) unlet g:termdebug_config endfunc
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
Changes implemented.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Hum... still does not pass some CI tests.
I am wondering if the following shall be changed somewhere as well...
--- a/src/testdir/test_termdebug.vim
+++ b/src/testdir/test_termdebug.vim
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
Hmm, it passed in my repository...
https://github.com/k-takata/vim/actions/runs/9173423210
Commit: k-takata@f9523a2
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
> + " etc. in the current directory + let g:termdebug_config = {} + let g:termdebug_config['use_prompt'] = 1 + let filename = 'gdb' + let replacement_filename = 'Termdebug-gdb-console' + + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Throw away the file once the test has done. + Termdebug + sleep 2 + " A file named filename already exists in the working directory, + " hence you must call the newly created buffer differently + call WaitForAssert({-> assert_false(bufexists(filename))}) + call WaitForAssert({-> assert_true(bufexists(replacement_filename))}) + quit! + call WaitForAssert({-> assert_equal(1, winnr('$'))}⬇️ Suggested change
- call WaitForAssert({-> assert_equal(1, winnr('$'))} + call WaitForAssert({-> assert_equal(1, winnr('$'))})
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Hum... could you double check my changes to check if I did everything right?
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
Usch! Hopefully one day someone will make a Vim/Vim9 language server to catch these typos :D
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
@yegappan commented on this pull request.
In src/testdir/test_termdebug.vim:
> @@ -300,4 +300,41 @@ func Test_termdebug_mapping() %bw! endfunc +func Test_termdebug_bufnames() + " Test if user has filename/folders named gdb, Termdebug-gdb-console, + " etc. in the current directory + let g:termdebug_config = {} + let g:termdebug_config['use_prompt'] = 1 + let filename = 'gdb' + let replacement_filename = 'Termdebug-gdb-console' + + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Throw away the file once the test has done. + Termdebug + sleep 2
Using hard-coded sleeps in tests will increase the time it takes to run the tests. Can you change these
to WaitForAssert() calls?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot commented on this pull request.
In src/testdir/test_termdebug.vim:
> @@ -300,4 +300,41 @@ func Test_termdebug_mapping() %bw! endfunc +func Test_termdebug_bufnames() + " Test if user has filename/folders named gdb, Termdebug-gdb-console, + " etc. in the current directory + let g:termdebug_config = {} + let g:termdebug_config['use_prompt'] = 1 + let filename = 'gdb' + let replacement_filename = 'Termdebug-gdb-console' + + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Throw away the file once the test has done. + Termdebug + sleep 2
Ok, I replaced the sleep 2
with call WaitForAssert({-> assert_equal(3, winnr('$'))}
, which, in my understanding, wait until the third window is available (in this case, given the termdebug setup, it means that termdebug is up and running).
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@yegappan commented on this pull request.
In src/testdir/test_termdebug.vim:
> + call WaitForAssert({-> assert_equal(3, winnr('$'))} + " A file named filename already exists in the working directory, + " hence you must call the newly created buffer differently + call WaitForAssert({-> assert_false(bufexists(filename))}) + call WaitForAssert({-> assert_true(bufexists(replacement_filename))}) + quit! + call WaitForAssert({-> assert_equal(1, winnr('$'))}) + + " Check if error message is in :message + let g:termdebug_config['disasm_window'] = 1 + let filename = 'Termdebug-asm-listing' + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Check only the head of the error message + let error_message = "You have a file/folder named '" .. filename .. "'" + Termdebug + sleep 2
Can you replace this hard-coded sleep with WaitForAssert()?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot pushed 1 commit.
—
View it on GitHub.
You are receiving this because you are subscribed to this thread.
@ubaldot commented on this pull request.
In src/testdir/test_termdebug.vim:
> + call WaitForAssert({-> assert_equal(3, winnr('$'))} + " A file named filename already exists in the working directory, + " hence you must call the newly created buffer differently + call WaitForAssert({-> assert_false(bufexists(filename))}) + call WaitForAssert({-> assert_true(bufexists(replacement_filename))}) + quit! + call WaitForAssert({-> assert_equal(1, winnr('$'))}) + + " Check if error message is in :message + let g:termdebug_config['disasm_window'] = 1 + let filename = 'Termdebug-asm-listing' + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Check only the head of the error message + let error_message = "You have a file/folder named '" .. filename .. "'" + Termdebug + sleep 2
Done! Sorry for so many commits, it's my first time. I wish I could test the CI on my fork first... shall I cancel the PR if the CI still fails?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@chrisbra commented on this pull request.
In src/testdir/test_termdebug.vim:
> + call WaitForAssert({-> assert_equal(3, winnr('$'))} + " A file named filename already exists in the working directory, + " hence you must call the newly created buffer differently + call WaitForAssert({-> assert_false(bufexists(filename))}) + call WaitForAssert({-> assert_true(bufexists(replacement_filename))}) + quit! + call WaitForAssert({-> assert_equal(1, winnr('$'))}) + + " Check if error message is in :message + let g:termdebug_config['disasm_window'] = 1 + let filename = 'Termdebug-asm-listing' + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Check only the head of the error message + let error_message = "You have a file/folder named '" .. filename .. "'" + Termdebug + sleep 2
No worries. After all, that's what we have CI for ;) BTW: you can also run the tests locally: https://github.com/vim/vim/blob/master/src/testdir/README.txt
Just need to run make test_termdebug.res
and then check the test.log
and messages
file.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@ubaldot commented on this pull request.
In src/testdir/test_termdebug.vim:
> + call WaitForAssert({-> assert_equal(3, winnr('$'))} + " A file named filename already exists in the working directory, + " hence you must call the newly created buffer differently + call WaitForAssert({-> assert_false(bufexists(filename))}) + call WaitForAssert({-> assert_true(bufexists(replacement_filename))}) + quit! + call WaitForAssert({-> assert_equal(1, winnr('$'))}) + + " Check if error message is in :message + let g:termdebug_config['disasm_window'] = 1 + let filename = 'Termdebug-asm-listing' + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Check only the head of the error message + let error_message = "You have a file/folder named '" .. filename .. "'" + Termdebug + sleep 2
AWESOME! I am going to give it a shot straight away. :)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@ubaldot commented on this pull request.
In src/testdir/test_termdebug.vim:
> + call WaitForAssert({-> assert_equal(3, winnr('$'))} + " A file named filename already exists in the working directory, + " hence you must call the newly created buffer differently + call WaitForAssert({-> assert_false(bufexists(filename))}) + call WaitForAssert({-> assert_true(bufexists(replacement_filename))}) + quit! + call WaitForAssert({-> assert_equal(1, winnr('$'))}) + + " Check if error message is in :message + let g:termdebug_config['disasm_window'] = 1 + let filename = 'Termdebug-asm-listing' + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Check only the head of the error message + let error_message = "You have a file/folder named '" .. filename .. "'" + Termdebug + sleep 2
From the README
RUNNING THE TESTS:
To run a single test from the src directory:
$ make test_<name>
The below commands should be run from the src/testdir directory.
To run a single test:
$ make test_<name>.res
I am in ./vim/src/testdir
but when I run make test_termdebug.res
I get make: *** No rule to make target '../vim', needed by 'test_termdebug.res'. Stop.
Perhaps there is some requirement that I am missing?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@chrisbra commented on this pull request.
In src/testdir/test_termdebug.vim:
> + call WaitForAssert({-> assert_equal(3, winnr('$'))} + " A file named filename already exists in the working directory, + " hence you must call the newly created buffer differently + call WaitForAssert({-> assert_false(bufexists(filename))}) + call WaitForAssert({-> assert_true(bufexists(replacement_filename))}) + quit! + call WaitForAssert({-> assert_equal(1, winnr('$'))}) + + " Check if error message is in :message + let g:termdebug_config['disasm_window'] = 1 + let filename = 'Termdebug-asm-listing' + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Check only the head of the error message + let error_message = "You have a file/folder named '" .. filename .. "'" + Termdebug + sleep 2
go up and directory and make vim first (cd .. && make)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
@ubaldot commented on this pull request.
In src/testdir/test_termdebug.vim:
> + call WaitForAssert({-> assert_equal(3, winnr('$'))} + " A file named filename already exists in the working directory, + " hence you must call the newly created buffer differently + call WaitForAssert({-> assert_false(bufexists(filename))}) + call WaitForAssert({-> assert_true(bufexists(replacement_filename))}) + quit! + call WaitForAssert({-> assert_equal(1, winnr('$'))}) + + " Check if error message is in :message + let g:termdebug_config['disasm_window'] = 1 + let filename = 'Termdebug-asm-listing' + call writefile(['This', 'is', 'a', 'test'], filename, 'D') + " Check only the head of the error message + let error_message = "You have a file/folder named '" .. filename .. "'" + Termdebug + sleep 2
Thanks!
I built the vim
executable. The test run but it seems that my tests are not run:
$ make test_termdebug.res
VIMRUNTIME=../../runtime ../vim -f -u unix.vim --gui-dialog-file guidialog -U NONE --noplugin --not-a-term -S runtest.vim test_termdebug.vim --cmd 'au SwapExists * let v:swapchoice = "e"' | LC_ALL=C LANG=C LANGUAGE=C awk '/Executing Test_/{match($0, "([0-9][0-9]:[0-9][0-9] *)?Executing Test_[^\\)]*\\)"); print substr($0, RSTART, RLENGTH) "\r"; fflush()}'
02:05 Executing Test_termdebug_basic()
02:09 Executing Test_termdebug_bufnames()
02:09 Executing Test_termdebug_mapping()
02:09 Executing Test_termdebug_tbreak()
It is missing Test_termdebug_bufnames()
. Shall I add it to some other file?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
thanks, looks good now.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.
—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.