Details

      Description

      With 2.10.0-M5 -explaintypes got much more verbose, compared to (for instance) Scala 2.9.x - it produces extra output even when compilation does not fail.
      While this might be done to help debugging, I don't think this is justified even for an intermediate release - debugging stuff should have separate options.

      This problem was already briefly discussed on scala-language; Daniel Sobral suggested a bug report.
      https://groups.google.com/forum/?fromgroups#!topic/scala-language/XG5SsELdwAg

        Issue Links

          Activity

          Hide
          Paolo G. Giarrusso added a comment - - edited

          Testing that -explaintypes works at all requires that partest inspects the standard output of the programs it runs. Hence I'm marking this as blocked by SI-7003.

          Show
          Paolo G. Giarrusso added a comment - - edited Testing that -explaintypes works at all requires that partest inspects the standard output of the programs it runs. Hence I'm marking this as blocked by SI-7003 .
          Hide
          Adriaan Moors added a comment -

          Un-assigning to foster work stealing, as announced in https://groups.google.com/forum/?fromgroups=#!topic/scala-internals/o8WG4plpNkw

          Show
          Adriaan Moors added a comment - Un-assigning to foster work stealing, as announced in https://groups.google.com/forum/?fromgroups=#!topic/scala-internals/o8WG4plpNkw
          Hide
          Paolo G. Giarrusso added a comment -

          https://github.com/scala/scala/pull/2259 is my current fix - the only problem is that it's hard to test correctly, because of limitations/bugs in partest (SI-7003 among others).

          Show
          Paolo G. Giarrusso added a comment - https://github.com/scala/scala/pull/2259 is my current fix - the only problem is that it's hard to test correctly, because of limitations/bugs in partest ( SI-7003 among others).
          Hide
          Paolo G. Giarrusso added a comment -

          The pull request was merged, but this should still be backported to 2.10.x. So I'm not closing this just yet.

          Show
          Paolo G. Giarrusso added a comment - The pull request was merged, but this should still be backported to 2.10.x. So I'm not closing this just yet.
          Hide
          Jason Zaugg added a comment -

          Please open a followup ticket if you want this backported. I'm closing to show this as fixed in 2.11.0-M3.

          Show
          Jason Zaugg added a comment - Please open a followup ticket if you want this backported. I'm closing to show this as fixed in 2.11.0-M3.

            People

            • Assignee:
              Paolo G. Giarrusso
              Reporter:
              Paolo G. Giarrusso
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development