sourcepath: don't create a new run for outline parsing

smarter · smarter · commit 35c08046c61d · 2019-12-04T18:19:04.000+01:00
The previous commit broke compilation of the standard library, because
we ended up calling `runContext` before the first context is set, when
it's still `null`. But we don't really need a new run just for creating
the symbol: the completer we set will call Run#lateCompile which will
take care of creating a new run anyway.
diff --git a/compiler/src/dotty/tools/dotc/core/SymbolLoaders.scala b/compiler/src/dotty/tools/dotc/core/SymbolLoaders.scala
@@ -156,7 +156,7 @@ object SymbolLoaders {
       }
 
       val unit = CompilationUnit(ctx.getSource(src.path))
-      enterScanned(unit)(ctx.run.runContext.fresh.setCompilationUnit(unit))
+      enterScanned(unit)(ctx.fresh.setCompilationUnit(unit))
 
   /** The package objects of scala and scala.reflect should always
    *  be loaded in binary if classfiles are available, even if sourcefiles

Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ object SymbolLoaders {`
`156`	`156`	`}`
`157`	`157`
`158`	`158`	`val unit = CompilationUnit(ctx.getSource(src.path))`
`159`		`- enterScanned(unit)(ctx.run.runContext.fresh.setCompilationUnit(unit))`
	`159`	`+ enterScanned(unit)(ctx.fresh.setCompilationUnit(unit))`
`160`	`160`
`161`	`161`	`/** The package objects of scala and scala.reflect should always`
`162`	`162`	`* be loaded in binary if classfiles are available, even if sourcefiles`