Purpose of nsc.Reporter/xsbti.Reporter

[Simon Schäfer]

I'm wondering what the purpose of these interfaces is. They seem to
allow reporting build information from sbt/scalac back to the IDE but I
noticed that a lot of there features are unused (not clear to me if
intentionally).

For example there exists the method Reporter.comment whose meaning
doesn't open up to me. On the IDE side (in BuildReporter) it is
implemented to report task tags, which does not work because the code
gets never called.

[Iulian Dragos]

> On 13.06.2014, at 14:30, Simon Schäfer <ma...@antoras.de> wrote:
>
> I'm wondering what the purpose of these interfaces is. They seem to allow reporting build information from sbt/scalac back to the IDE but I noticed that a lot of there features are unused (not clear to me if intentionally).

That's correct, that's the way to get error reporting from the compiler/Sbt. I think the Sbt interface might be not doing much right now, but we should probably change that and at least report errors.

>
> For example there exists the method Reporter.comment whose meaning doesn't open up to me. On the IDE side (in BuildReporter) it is implemented to report task tags, which does not work because the code gets never called.

That's a regression in 2.11 (the code used to be called). However, we might want to rewrite that feature without relying on the compiler itself. We could get there by simply instantiating the scalac scanner and looking at comment tokens.

Cheers,
Iulian

>
> --
> You received this message because you are subscribed to the Google Groups "Scala IDE Dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to scala-ide-de...@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/msgid/scala-ide-dev/539AEEC8.7020206%40antoras.de.
> For more options, visit https://groups.google.com/d/optout.

[Som Snytt]

FWIW, Adriaan's current refactor of Reporter says that Reporter.comment will disappear in the future interface.

The current comment method has a comment noting the IDE uses it.

To view this discussion on the web visit https://groups.google.com/d/msgid/scala-ide-dev/D072F6A4-182E-4D2A-B1E5-03436634CD13%40gmail.com.

[Adriaan Moors]

Yep! Please review and add feedback at 
https://github.com/scala/scala/pull/3820

To view this discussion on the web visit https://groups.google.com/d/msgid/scala-ide-dev/CAPaMnL7jha6Mo9Siz3Fi3pkspYujEVJyJoB%3D6Rbbb%3DGmNmEezw%40mail.gmail.com.

For more options, visit https://groups.google.com/d/optout.

--

SEE YOU IN BERLIN!

Scala

Days

June 16th-18th,

Berlin