Test decimal breakpoints - #16081
#16081Closed
ubaldot wants to merge 9 commits into
vim:mastervim/vim:masterfrom
ubaldot:test_decimal_breakpointsubaldot/vim:test_decimal_breakpointsCopy head branch name to clipboard
Closed
Test decimal breakpoints#16081ubaldot wants to merge 9 commits intovim:mastervim/vim:masterfrom ubaldot:test_decimal_breakpointsubaldot/vim:test_decimal_breakpointsCopy head branch name to clipboard
ubaldot wants to merge 9 commits into
vim:mastervim/vim:masterfrom
ubaldot:test_decimal_breakpointsubaldot/vim:test_decimal_breakpointsCopy head branch name to clipboard
Conversation
ubaldot
commented
Nov 18, 2024
Contributor
- Added tests related to PR: runtime: add option for decimal breakpoint signs in termdebug #16011
- Adding a new test may not be super-easy for new contributors. I added a tip in the README.txt to help new contributors to tests their own tests within a running Vim instance.
zeertzjq
reviewed
Nov 18, 2024
| endfunc | ||
|
|
||
| func Test_termdebug_decimal_breakpoints() | ||
| let bin_name = 'XTD_basic' |
Member
There was a problem hiding this comment.
Shouldn't a different bin_name be used?
Contributor
Author
There was a problem hiding this comment.
Point. let in legacy script define global variables, correct? How do you suggest to solve the issue then? Just by using a different name than bin_name?
Member
There was a problem hiding this comment.
No, it sets function local variables in a function. I was suggesting changing XTD_basic to something else.
Contributor
Author
There was a problem hiding this comment.
No, it sets function local variables in a function.
Auch! I am familiar with Vim9 but with the legacy language I have some problem, sorry!
Btw, I changed the name of the file.
Member
There was a problem hiding this comment.
when you change it here, you should also change it a few lines below, in the Termdebug ./bin_name, no?
Contributor
Author
There was a problem hiding this comment.
Usch! Sorry. I also run the test locally and for some reason they passed... fixed now. Please, take a look. :)
Member
|
I still see some failures: |
Contributor
Author
|
Ok! I see the issue. But I am bit puzzled: why the tests passed when I executed them locally?! |
zeertzjq
reviewed
Nov 18, 2024
Contributor
Author
I was actually running |
Member
|
Still some failures: Can you rename the test to |
Contributor
Author
|
Thanks @chrisbra for your time! |
Member
|
thanks! |
RestorerZ
added a commit
to RestorerZ/fork_vim-orig
that referenced
this pull request
Nov 24, 2024
CI: Bump codecov/codecov-action from 4 to 5 Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 4 to 5. - [Release notes](https://github.com/codecov/codecov-action/releases) - [Changelog](https://github.com/codecov/codecov-action/blob/main/CHANGELOG.md) - [Commits](codecov/codecov-action@v4...v5) --- updated-dependencies: - dependency-name: codecov/codecov-action dependency-type: direct:production update-type: version-update:semver-major ... closes: vim#16079 Signed-off-by: dependabot[bot] <support@github.com> Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0870: too many strlen() calls in eval.c Problem: too many strlen() calls in eval.c Solution: Refactor eval.c to remove calls to STRLEN() (John Marriott) closes: vim#16066 Signed-off-by: John Marriott <basilisk@internode.on.net> Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0871: getcellpixels() can be further improved Problem: getcellpixels() can be further improved Solution: Fix floating point exception, implement getcellpixels() in the UI (mikoto2000) closes: vim#16059 Signed-off-by: mikoto2000 <mikoto2000@gmail.com> Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0872: No test for W23 message Problem: No test for W23 message Solution: Check for W23 message when accessing the clipboard fails (after v9.1.0868) closes: vim#16076 Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0873: filetype: Vivado files are not recognized Problem: filetype: Vivado files are not recognized Solution: detect '*.mss' files as 'mss' filetype (Wu, Zhenyu) references: https://docs.amd.com/r/2020.2-English/ug1400-vitis-embedded/Microprocessor-Software-Specification-MSS closes: vim#15907 Signed-off-by: Wu, Zhenyu <wuzhenyu@ustc.edu> Signed-off-by: Christian Brabandt <cb@256bit.org> CI: join codecov array flags, instead of accessing it directly closes: vim#16082 Signed-off-by: rhysd <lin90162@yahoo.co.jp> Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0874: filetype: karel files are not detected Problem: filetype: karel files are not detected Solution: detect '*.kl' files as karel filetype, include syntax and filetype plugin (Kirill Morozov) closes: vim#16075 Co-authored-by: KnoP-01 <knosowski@graeffrobotics.de> Signed-off-by: Kirill Morozov <kirill@robotix.pro> Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0875: filetype: hyprlang detection can be improved Problem: filetype: hyprlang detection can be improved Solution: detect '/hypr/*.conf' files as hyprlang filetype, include basic syntax highlighting (Luca Saccarola) fixes: vim#15875 closes: vim#16064 Signed-off-by: Luca Saccarola <github.e41mv@aleeas.com> Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0876: filetype: openCL files are not recognized Problem: filetype: openCL files are not recognized Solution: detect '*.cl' files as opencl or lisp filetype, include a opencl syntax and filetype plugin (Wu, Zhenyu) closes: vim#15825 Signed-off-by: Wu, Zhenyu <wuzhenyu@ustc.edu> Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0877: tests: missing test for termdebug + decimal signs Problem: tests: missing test for termdebug + decimal signs Solution: Add a termdebug test (Ubaldo Tiberi) closes: vim#16081 Signed-off-by: Ubaldo Tiberi <ubaldo.tiberi@volvo.com> Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0878: termdebug: cannot enable DEBUG mode Problem: termdebug: cannot enable DEBUG mode Solution: Allow to specify DEBUG mode (Ubaldo Tiberi) closes: vim#16080 Signed-off-by: Ubaldo Tiberi <ubaldo.tiberi@volvo.com> Signed-off-by: Christian Brabandt <cb@256bit.org> runtime(compiler): fix escaping of arguments passed to :CompilerSet See newly added help entry referring to option-backslash closes: vim#16084 Signed-off-by: Konfekt <Konfekt@users.noreply.github.com> Signed-off-by: Christian Brabandt <cb@256bit.org> Add clang-format config file This is used in preparation to enable automatic code-formatting in the following commits. For now let's just add a clang-format config file, formatting of source files will follow. related: vim#16019 Signed-off-by: Luca Saccarola <github.e41mv@aleeas.com> Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0879: source is not consistently formatted Problem: source is not consistently formatted Solution: reformat sign.c and sound.c (Luca Saccarola) closes: vim#16019 Signed-off-by: Luca Saccarola <github.e41mv@aleeas.com> Signed-off-by: Christian Brabandt <cb@256bit.org> git: ignore re-formatting commit v9.1.0879 for blame Signed-off-by: Christian Brabandt <cb@256bit.org> runtime(doc): add helptag for :HelpToc command Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0880: filetype: C3 files are not recognized Problem: filetype: C3 files are not recognized Solution: detect '*.c3*' files as c3 filetype (Turiiya) closes: vim#16087 Signed-off-by: Turiiya <34311583+ttytm@users.noreply.github.com> Signed-off-by: Christian Brabandt <cb@256bit.org> runtime(misc): add Italian LICENSE and (top-level) README file related: vim#16061 Signed-off-by: Antonio Giovanni Colombo <azc100@gmail.com> Signed-off-by: Christian Brabandt <cb@256bit.org> runtime(apache): Update syntax keyword definition closes: vim#16105 Signed-off-by: nisbet-hubbard <87453615+nisbet-hubbard@users.noreply.github.com> Signed-off-by: Christian Brabandt <cb@256bit.org> runtime(netrw): update netrw's decompress logic Detect a few more default archive types, correctly handle file extensions with digits in it. fixes: vim#16099 closes: vim#16104 Signed-off-by: Konfekt <Konfekt@users.noreply.github.com> Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0881: GUI: message dialog may not get focus Problem: GUI: message dialog may not get focus Solution: add window manager hint to give focus to the dialog (Chris White) Tell the window manager that message dialogs should be given focus when the user switches from another application back to Vim. This can happen, e.g., when the user has a file open in Vim and then edits it in another program. fixes: vim#172 closes: vim#16100 Signed-off-by: Chris White <christopher.white@crowdstrike.com> Signed-off-by: Christian Brabandt <cb@256bit.org> patch 9.1.0882: too many strlen() calls in insexpand.c Problem: too many strlen() calls in insexpand.c Solution: Refactor insexpand.c and reduce number of calls to STRLEN(), fix a warning get_next_filename_completion(), add new function ins_compl_leader_len() (John Marriott) closes: vim#16095 Signed-off-by: John Marriott <basilisk@internode.on.net> Signed-off-by: Christian Brabandt <cb@256bit.org> runtime(netrw): Fixing powershell execution issues on Windows closes: vim#16094 Signed-off-by: GuyBrush <miguel.barro@live.com> runtime(doc): Expand docs on :! vs. :term fixes: vim#16071 closes: vim#16089 Signed-off-by: matveyt <matthewtarasov@yandex.ru> Signed-off-by: Christian Brabandt <cb@256bit.org> rename the files of the chapter one to tutor1 The numbering of lessons in the files of chapter one begins with 1 the files of chapter one
zeertzjq
added a commit
to zeertzjq/neovim
that referenced
this pull request
Jul 7, 2025
Problem: tests: missing test for termdebug + decimal signs Solution: Add a termdebug test (Ubaldo Tiberi) closes: vim/vim#16081 vim/vim@b5c1557 Co-authored-by: Ubaldo Tiberi <ubaldo.tiberi@volvo.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.