Extension is generating URIs which are causing vscode-csharp to error in the diff view.

[makzimus]

Please see the issue submitted for vscode-csharp here.

[mjcrouch]

thanks for reporting - it was quite a while ago but I seem to remember this being a rather hacky way to pass around the details we needed for the diff view & view of historic files in general (inspired by how gitlens worked at the time) - the information like @=1234 is really not an 'authority' but I recall there were some very annoying edge cases in vs code that made it impossible to pass the data in e.g. the query string where it would make more sense. (I don't remember the details at-all but iirc it was something like: when navigating between different versions of a file, the perforce api didn't have access to the query string for the open editor so we had no way of working out exactly what we should be displaying - may be totally off the mark)

So, perhaps there have been improvements in the core vs code that make it possible to do it in a better way, or perhaps it would be simpler to just modify the encoding to meet the URL spec whithout changing where we put it.