Add MessageCollector#hasErrors, get rid of MessageSeverityCollector
Also fix duplicated wrapping of a message collector into a message severity
collector (in CLICompiler and in the beginning of doExecute in
K2JVMCompiler/K2JSCompiler)
Original commit: 3c81bb4bfc
This commit is contained in:
@@ -689,8 +689,10 @@ class KotlinBuilder : ModuleLevelBuilder(BuilderCategory.SOURCE_PROCESSOR) {
|
|||||||
}
|
}
|
||||||
|
|
||||||
class MessageCollectorAdapter(private val context: CompileContext) : MessageCollector {
|
class MessageCollectorAdapter(private val context: CompileContext) : MessageCollector {
|
||||||
|
private var hasErrors = false
|
||||||
|
|
||||||
override fun report(severity: CompilerMessageSeverity, message: String, location: CompilerMessageLocation) {
|
override fun report(severity: CompilerMessageSeverity, message: String, location: CompilerMessageLocation) {
|
||||||
|
hasErrors = hasErrors or severity.isError
|
||||||
var prefix = ""
|
var prefix = ""
|
||||||
if (severity == EXCEPTION) {
|
if (severity == EXCEPTION) {
|
||||||
prefix = INTERNAL_ERROR_PREFIX
|
prefix = INTERNAL_ERROR_PREFIX
|
||||||
@@ -705,6 +707,8 @@ class KotlinBuilder : ModuleLevelBuilder(BuilderCategory.SOURCE_PROCESSOR) {
|
|||||||
))
|
))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
override fun hasErrors(): Boolean = hasErrors
|
||||||
|
|
||||||
private fun renderLocationIfNeeded(location: CompilerMessageLocation): String {
|
private fun renderLocationIfNeeded(location: CompilerMessageLocation): String {
|
||||||
if (location == CompilerMessageLocation.NO_LOCATION) return ""
|
if (location == CompilerMessageLocation.NO_LOCATION) return ""
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user