Skip to content

Commit

Permalink
Dispose AnalysisEnvironment
Browse files Browse the repository at this point in the history
  • Loading branch information
vmishenev committed Dec 1, 2022
1 parent f7691fe commit 2491d73
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ internal fun createEnvironmentAndFacade(
val environment = createCoreEnvironment()

val (facade, _) = createResolutionFacade(environment, analysisConfiguration.ignoreCommonBuiltIns)
EnvironmentAndFacade(environment, facade)
EnvironmentAndFacade(environment, facade, this)
}

class DokkaMessageCollector(private val logger: DokkaLogger) : MessageCollector {
Expand All @@ -51,7 +51,10 @@ class DokkaMessageCollector(private val logger: DokkaLogger) : MessageCollector
}

// It is not data class due to ill-defined equals
class EnvironmentAndFacade(val environment: KotlinCoreEnvironment, val facade: DokkaResolutionFacade) {
class EnvironmentAndFacade(val environment: KotlinCoreEnvironment, val facade: DokkaResolutionFacade, private val analysisEnvironment: AnalysisEnvironment) {
operator fun component1() = environment
operator fun component2() = facade
protected fun finalize() {
analysisEnvironment.dispose()
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,7 @@ abstract class SamplesTransformer(val context: DokkaContext) : PageTransformer {

val environment = createCoreEnvironment()
val (facade, _) = createResolutionFacade(environment)
EnvironmentAndFacade(environment, facade)
EnvironmentAndFacade(environment, facade, this)
}
}

Expand Down

0 comments on commit 2491d73

Please sign in to comment.