Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix split repl lines issue #233

Closed
wants to merge 1 commit into from
Closed

fix split repl lines issue #233

wants to merge 1 commit into from

Conversation

y1rn
Copy link
Contributor

@y1rn y1rn commented Jul 11, 2021

image

In this debug step by step case expect:
(
initial
1 2
true
0
apple
)

@mfussenegger
Copy link
Owner

I assume this is partially related to the require('dap.repl').append(chunk) from #232 ?

I think instead of trimming the last newline within the repl append code, this should rather be done on the call-site already.

You could wrap the chunk in an vim.trim() call.

@y1rn
Copy link
Contributor Author

y1rn commented Jul 11, 2021

i have been debug rust file, still has the same issue.

@mfussenegger
Copy link
Owner

Ah, I see.

I've merged the change with 55f2737

@y1rn
Copy link
Contributor Author

y1rn commented Jul 11, 2021

i work with vim.trim() well, i'v amend my fork https://github.com/y1rn/nvim-dap/commit/d2bf364952eea328fcea33671e6654e59c736c26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants