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

[PL] Remove filters for PL warnings in NativeTestGroupingMessageCollector

Browse Source 
Normally, all PL warning messages should have non-null location. Thus, no need to have a specific filter for them.
This commit is contained in:
Dmitriy Dolovov
2023-03-22 13:59:28 +01:00
committed by Space Team
parent 295c578f44
commit 3e7043e1f5
1 changed files with 0 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
native/native.tests/tests/org/jetbrains/kotlin/konan/blackboxtest/support/compilation/NativeTestGroupingMessageCollector.kt
-10
View File
@@ -51,10 +51,6 @@ internal class NativeTestGroupingMessageCollector(
null null
} }
private val partialLinkageEnabled: Boolean by lazy {
"-Xpartial-linkage" in compilerArgs
}
override fun report(severity: CompilerMessageSeverity, message: String, location: CompilerMessageSourceLocation?) = override fun report(severity: CompilerMessageSeverity, message: String, location: CompilerMessageSourceLocation?) =
super.report(adjustSeverity(severity, message, location), message, location) super.report(adjustSeverity(severity, message, location), message, location)
@@ -72,7 +68,6 @@ internal class NativeTestGroupingMessageCollector(
|| isUnsafeCompilerArgumentsWarning(message) || isUnsafeCompilerArgumentsWarning(message)
|| isLibraryIncludedMoreThanOnceWarning(message) || isLibraryIncludedMoreThanOnceWarning(message)
|| isK2Experimental(message) || isK2Experimental(message)
|| isPartialLinkageWarning(message)
|| isLegacyMMWarning(message) -> { || isLegacyMMWarning(message) -> {
// These warnings are known and should not be reported as errors. // These warnings are known and should not be reported as errors.
severity severity
@@ -112,10 +107,6 @@ internal class NativeTestGroupingMessageCollector(
return libraryPath == pathOfCachedLibraryWithTests return libraryPath == pathOfCachedLibraryWithTests
} }
private fun isPartialLinkageWarning(message: String): Boolean =
partialLinkageEnabled && message.matches(PARTIAL_LINKAGE_WARNING_REGEX)
private fun isK2Experimental(message: String): Boolean = message.startsWith(K2_NATIVE_EXPERIMENTAL_WARNING_PREFIX) private fun isK2Experimental(message: String): Boolean = message.startsWith(K2_NATIVE_EXPERIMENTAL_WARNING_PREFIX)
// Legacy MM is deprecated and will be removed in 1.9.20. Until that moment we still need to run tests with it. // Legacy MM is deprecated and will be removed in 1.9.20. Until that moment we still need to run tests with it.
@@ -128,7 +119,6 @@ internal class NativeTestGroupingMessageCollector(
private const val UNSAFE_COMPILER_ARGS_WARNING_PREFIX = "ATTENTION!\nThis build uses unsafe internal compiler arguments:\n\n" private const val UNSAFE_COMPILER_ARGS_WARNING_PREFIX = "ATTENTION!\nThis build uses unsafe internal compiler arguments:\n\n"
private const val LIBRARY_INCLUDED_MORE_THAN_ONCE_WARNING_PREFIX = "library included more than once: " private const val LIBRARY_INCLUDED_MORE_THAN_ONCE_WARNING_PREFIX = "library included more than once: "
private const val K2_NATIVE_EXPERIMENTAL_WARNING_PREFIX = "Language version 2.0 is experimental" private const val K2_NATIVE_EXPERIMENTAL_WARNING_PREFIX = "Language version 2.0 is experimental"
private val PARTIAL_LINKAGE_WARNING_REGEX = Regex(".+ uses unlinked symbols(:.*)?")
private const val LEGACY_MM_WARNING_PREFIX = "Legacy MM is deprecated and will be removed" private const val LEGACY_MM_WARNING_PREFIX = "Legacy MM is deprecated and will be removed"
private fun parseLanguageFeatureArg(arg: String): String? = private fun parseLanguageFeatureArg(arg: String): String? =