On top of the class name, it would be nice to display where it was loaded from

# eap

[cedric]

On top of the class name, it would be nice to display where it was loaded from

[udalov]

cedric: could you please look at AnalyzerWithCompilerReport.Companion.reportDiagnostics and what is the `hasIncompatibleClassErrors`‘s value there?

[udalov]

in the debugger

[cedric]

Oh it’s false

[cedric]

Ah found a true one

[cedric]

Can’t paste screenshot in a thread apparently

[udalov]

The reportDiagnostics which returns ReportDiagnosticsResult — is hasIncompatibleClassErrors true there? If yes, this means, there’s at least one other relevant diagnostic (INCOMPATIBLE_CLASS / PRE_RELEASE_CLASS), need to find out why is it not reported

[cedric]

It is being reported, but the class name is never displayed

[udalov]

I don’t see neither INCOMPATIBLE_CLASS nor PRE_RELEASE_CLASS in the K2JVMCompiler output you’ve pasted earlier

[cedric]

Yes that was the one where the error was false. The next one it was true and had that. But either way, I don’t see any code that will display the class that is out of date, isn’t that simply the problem?

[udalov]

Let me clarify. The error “Incompatible classes were found in dependencies. <…>” is not supposed to contain the name of the problematic class. This is a “master” error which is output only if there’s another error which does contain the name of the class. This other error reads: “Class XXX is compiled by a pre-release version of Kotlin and cannot be loaded by this version of the compiler”. I’m trying to find out now why is there no such error in the output you’ve posted, since judging by the code, there should be at least one such error

[udalov]

I think I might have an idea — could you please evaluate diagnostic.isValid on the PRE_RELEASE_CLASS diagnostic you’ve posted on a screenshot above?

[udalov]

My suspicion is that diagnostic.isValid returns false, and so the PRE_RELEASE_CLASS diagnostic is not reported, but its existence is incorrectly taken into account and then the master error is reported

[cedric]

Ah ok got it. Yes, never saw any of these previous error messages.

[cedric]

Ok let me see if I can make this happen again (I’ve upgraded all the classes now)