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

Escape backslashes #2546

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Escape backslashes #2546

wants to merge 1 commit into from

Conversation

mattn
Copy link
Contributor

@mattn mattn commented Oct 21, 2019

Completion of :GoImport does not work since the path is not escaped.

@bhcleek
Copy link
Collaborator

bhcleek commented Oct 22, 2019

I'm having a hard time find a failure mode for the current implementation. How can I duplicate the problem that this PR solves?

@mattn
Copy link
Contributor Author

mattn commented Oct 22, 2019

As far as I walked issues, there are no filed issues will be fixed by this PR.

@bhcleek
Copy link
Collaborator

bhcleek commented Oct 22, 2019

Have you seen a problem in the wild? I'm trying to figure out how to duplicate the problem that this would fix 🤔

@bhcleek
Copy link
Collaborator

bhcleek commented Mar 22, 2020

@mattn Is this trying to solve a problem on Windows?

Is '\\' necessary or can it be shortened to '\'?

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