Merge pull request #113 from gslowikowski/no-temp-file

sksamuel · sksamuel · commit 052b9866b9e0 · 2015-03-11T12:05:13.000Z
Don't create unused temporary file on every plugin initialization
diff --git a/scalac-scoverage-plugin/src/main/scala/scoverage/plugin.scala b/scalac-scoverage-plugin/src/main/scala/scoverage/plugin.scala
@@ -47,7 +47,7 @@ class ScoveragePlugin(val global: Global) extends Plugin {
 class ScoverageOptions {
   var excludedPackages: Seq[String] = Nil
   var excludedFiles: Seq[String] = Nil
-  var dataDir: String = File.createTempFile("scoverage_datadir_not_defined", ".tmp").getParent
+  var dataDir: String = IOUtils.getTempPath
 }
 
 class ScoverageInstrumentationComponent(val global: Global)

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ class ScoveragePlugin(val global: Global) extends Plugin {`
`47`	`47`	`class ScoverageOptions {`
`48`	`48`	`var excludedPackages: Seq[String] = Nil`
`49`	`49`	`var excludedFiles: Seq[String] = Nil`
`50`		`- var dataDir: String = File.createTempFile("scoverage_datadir_not_defined", ".tmp").getParent`
	`50`	`+ var dataDir: String = IOUtils.getTempPath`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`class ScoverageInstrumentationComponent(val global: Global)`