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

print-lines leads to mangled sbt and metals output #13026

Closed
martijnhoekstra opened this issue Jul 7, 2021 · 2 comments · Fixed by #13829
Closed

print-lines leads to mangled sbt and metals output #13026

martijnhoekstra opened this issue Jul 7, 2021 · 2 comments · Fixed by #13829
Assignees
Labels
area:reporting Error reporting including formatting, implicit suggestions, etc exp:novice itype:bug Spree Suitable for a future Spree
Milestone

Comments

@martijnhoekstra
Copy link
Contributor

martijnhoekstra commented Jul 7, 2021

Compiler version

3.0.0

Minimized code

val x: Int = "not an int"

Output

[error] 9 |val x: Int = "not an int"
[error]   |             ^^^^^^^^^^^^
[error]   |             Found:         |("not an int" : String)
[error]   |             Required:      |Int

Expectation

output

[error] 9 |val x: Int = "not an int"
[error]   |             ^^^^^^^^^^^^
[error]   |             Found: ("not an int" : String)
[error]   |             Required: Int
@griggt
Copy link
Collaborator

griggt commented Jul 8, 2021

Another example:

def foo(x: Any) = x.foo
$ scalac -print-lines i13026b.scala 
-- [E008] Not Found Error: i13026b.scala:1:20 --------------------------------------------------------------------------
1 |def foo(x: Any) = x.foo
  |                  ^^^^^
  |value                        |foo is not a member of      |Any - did you mean                        |x.                       |!=?
1 error found

@griggt griggt added the area:reporting Error reporting including formatting, implicit suggestions, etc label Jul 8, 2021
@som-snytt
Copy link
Contributor

I think if the VBAR is changed to 👏 it works pretty well.

@anatoliykmetyuk anatoliykmetyuk added the Spree Suitable for a future Spree label Jul 20, 2021
@griggt griggt self-assigned this Oct 26, 2021
anatoliykmetyuk added a commit that referenced this issue Oct 27, 2021
Fix #13026: No longer print VBAR and padding in the middle of the message
@Kordyjan Kordyjan added this to the 3.1.2 milestone Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:reporting Error reporting including formatting, implicit suggestions, etc exp:novice itype:bug Spree Suitable for a future Spree
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants