Skip to content

Color leaks in error messages #3614

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

Closed
OlivierBlanvillain opened this issue Dec 1, 2017 · 2 comments
Closed

Color leaks in error messages #3614

OlivierBlanvillain opened this issue Dec 1, 2017 · 2 comments
Assignees
Labels
area:reporting Error reporting including formatting, implicit suggestions, etc itype:enhancement

Comments

@OlivierBlanvillain
Copy link
Contributor

trait Test[A]

object Test {
  val a = new Test[Int, Int, Int, Int, Int] {}
}
-- [E022] Syntax Error: /home/olivier/colol.scala:38:14 
38 |  val a = new Test[Int, Int, Int, Int, Int] {}
   |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |            Too many type arguments for Test[A]
   |            expected: [A]
   |            actual:   [35mInt0m, 35mInt0m, 35mInt0m, 35mInt0m, 35mInt0m]
@OlivierBlanvillain OlivierBlanvillain added area:reporting Error reporting including formatting, implicit suggestions, etc itype:enhancement prio:medium labels Dec 1, 2017
@rsoeldner
Copy link
Contributor

I can't reproduce this, for me, it looks normal.

-- [E022] Syntax Error: coloor.scala:4:14 --------------------------------------
4 |  val a = new Test[Int, Int, Int, Int, Int] {}
  |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  |              Too many type arguments for Test[A]
  |              expected: [A]
  |              actual:   [Int, Int, Int, Int, Int]

Can this be related to your terminal / OS ?

@allanrenucci
Copy link
Contributor

Reproduced on macOS

nicolasstucki added a commit to dotty-staging/dotty that referenced this issue Dec 1, 2017
nicolasstucki added a commit to dotty-staging/dotty that referenced this issue Dec 7, 2017
nicolasstucki added a commit to dotty-staging/dotty that referenced this issue Dec 7, 2017
nicolasstucki added a commit to dotty-staging/dotty that referenced this issue Dec 7, 2017
nicolasstucki added a commit to dotty-staging/dotty that referenced this issue Dec 7, 2017
nicolasstucki added a commit to dotty-staging/dotty that referenced this issue Dec 7, 2017
nicolasstucki added a commit that referenced this issue Dec 12, 2017
Fix #3614: Do not allow the SyntaxFormatter to recolor code
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 itype:enhancement
Projects
None yet
Development

No branches or pull requests

4 participants