Fix unicode path handling on windows LSPs

[thistle-witch]

Closes #2920

Tested with Rust analyzer. to_file_path() properly converts file:///c:/... style URIs into properly drive rooted windows paths.

[panekj]

This breaks lapce

[thistle-witch]

I'm not sure what you mean by "this breaks lapce"? I tested against the relevant URLs. The handling seemed to be specific to windows file URIs which I tested. No other code was affected.

[panekj]

Using to_file_path() makes lapce stack overflow

[thistle-witch]

I'm curious how you got that result. I ran a build of this myself.

[thistle-witch]

If you have a test project or example URI that causes the stack overflow I can try to find out what is going on.

[panekj]

The test code is what cargo init gives, typing and deleting println! causes overflow but it doesn't happen in release mode

[thistle-witch]

It would be helpful if you could provide the exact URI that is causing this issue for you. I have just run a debug build with a cargo init project and do not get the same results at all. To confirm, which version of windows are you using? Its entirely possible this stack overflow is caused by stack size differences.