azalea/kotlin-fork
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Kapt: Remove laziness from diagnosticFactory and javacMessages, use factory from supertype (KT-22189)

Browse Source 
writeDiagnostic() may be invoked after the 'context' is cleared, and lazy implementations will fail.
This commit is contained in:
Yan Zhulanow
2018-01-09 19:01:35 +09:00
parent 192489ae66
commit 283c762b8b
1 changed files with 3 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
plugins/kapt3/kapt3-compiler/src/org/jetbrains/kotlin/kapt3/javac/KaptJavaLog.kt
+3 -5
View File
@@ -42,11 +42,9 @@ class KaptJavaLog(
warnWriter: PrintWriter,
noticeWriter: PrintWriter,
val interceptorData: DiagnosticInterceptorData
) : Log(context, errWriter, warnWriter, noticeWriter) {
) : Log(context, errWriter, warnWriter, noticeWriter) {
private val stubLineInfo = KaptStubLineInformation()
private val diagnosticFactory by lazy { JCDiagnostic.Factory.instance(context) }
private val javacMessages by lazy { JavacMessages.instance(context) }
private val javacMessages = JavacMessages.instance(context)
init {
context.put(Log.outKey, noticeWriter)
@@ -104,7 +102,7 @@ class KaptJavaLog(
val kotlinFile = kotlinPosition?.let { getKotlinSourceFile(it) }
if (kotlinPosition != null && kotlinFile != null) {
val locationMessage = "$KOTLIN_LOCATION_PREFIX${kotlinFile.absolutePath}: (${kotlinPosition.line}, ${kotlinPosition.column})"
val locationDiagnostic = diagnosticFactory.note(null as DiagnosticSource?, null, "proc.messager", locationMessage)
val locationDiagnostic = diags.note(null as DiagnosticSource?, null, "proc.messager", locationMessage)
val wrappedDiagnostic = JCDiagnostic.MultilineDiagnostic(diagnostic, JavacList.of(locationDiagnostic))
reportDiagnostic(wrappedDiagnostic)