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

CLI produces unicode symbols that don't appear correctly in VSCode #6755

Closed
samuela opened this issue May 16, 2019 · 4 comments
Closed

CLI produces unicode symbols that don't appear correctly in VSCode #6755

samuela opened this issue May 16, 2019 · 4 comments

Comments

@samuela
Copy link

samuela commented May 16, 2019

Moving from fsharp/fsharp#878...

Please provide a succinct description of the issue.

I have a file with a few compile errors. That's all fine. Here's the output from dotnet run:

cloudbuild [master !?] -> dotnet run
/Users/skainswo/dev/kumo/new-world/src/cloudbuild/Program.fs(89,8): error FS0365: No implementation was given for 'Newtonsoft.Json.JsonConverter.WriteJson(writer: Newtonsoft.Json.JsonWriter, value: Status, serializer: Newtonsoft.Json.JsonSerializer) : unit' [/Users/skainswo/dev/kumo/new-world/src/cloudbuild/cloudbuild.fsproj]
/Users/skainswo/dev/kumo/new-world/src/cloudbuild/Program.fs(89,8): error FS0365: No implementation was given for 'Newtonsoft.Json.JsonConverter.ReadJson(reader: Newtonsoft.Json.JsonReader, objectType: System.Type, existingValue: Status, hasExistingValue: bool, serializer: Newtonsoft.Json.JsonSerializer) : Status' [/Users/skainswo/dev/kumo/new-world/src/cloudbuild/cloudbuild.fsproj]
/Users/skainswo/dev/kumo/new-world/src/cloudbuild/Program.fs(89,8): error FS0054: This type is 'abstract' since some abstract members have not been given an implementation. If this is intentional then add the '[<AbstractClass>]' attribute to your type. [/Users/skainswo/dev/kumo/new-world/src/cloudbuild/cloudbuild.fsproj]
/Users/skainswo/dev/kumo/new-world/src/cloudbuild/Program.fs(92,19): error FS3213: The member 'WriteJson<'a0,'a1> : 'a0 * Status * 'a1 -> unit' matches multiple overloads of the same method.�Please restrict it to one of the following:�   WriteJson : Newtonsoft.Json.JsonWriter * obj * Newtonsoft.Json.JsonSerializer -> unit�   WriteJson : Newtonsoft.Json.JsonWriter * Status * Newtonsoft.Json.JsonSerializer -> unit�   WriteJson : Newtonsoft.Json.JsonWriter * obj * Newtonsoft.Json.JsonSerializer -> unit. [/Users/skainswo/dev/kumo/new-world/src/cloudbuild/cloudbuild.fsproj]

The build failed. Please fix the build errors and run again.

Note the question marky things after "method.", "following:" and "-> unit".

Now here's what that looks like in the VSCode integrated terminal on macOS:
screen shot 2018-10-11 at 10 17 31 pm

Now the question marky things are backspace "e"s. Notice "followinge" and "unite".

Related information

Provide any related information

  • Operating system: macOS 10.14
  • Branch: n/a
  • .NET Runtime, CoreCLR or Mono Version: .NET Core 2.1.403
  • Editing Tools (e.g. Visual Studio Version): Version 1.29.0-insider (1.29.0-insider) (f151c46b1783cf4f9a4ca934a85f78846f08688e)
@cartermp
Copy link
Contributor

This is probably the newline characters in the message: https://github.com/dotnet/fsharp/blob/master/src/fsharp/FSComp.txt#L1349

Not sure how to reconcile that with the VSCode terminal.

@auduchinok
Copy link
Member

I've previously reported it in #4516.
The resolution was to leave corrupted output as it was. I'd be glad if it's being revisited.

@samuela
Copy link
Author

samuela commented May 23, 2019

@auduchinok That doesn't really sounds like a resolution haha

@cartermp
Copy link
Contributor

cartermp commented Aug 6, 2020

This one no longer reproduces in VSCode for me in VSCode. Will close as no repro

image

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

No branches or pull requests

3 participants