Apply review comments

Author: jchyb

compiler/src/dotty/tools/dotc/sbt/ExtractDependencies.scala

@@ -171,24 +171,20 @@ private class ExtractDependenciesCollector(rec: DependencyRecorder) extends tpd.
   lazy val allowsDependencyByMacroExpansion =
     classOf[DependencyContext].getFields().exists(_.getName() == "DependencyByMacroExpansion")
 
-  private def addMacroDependency(sym: Symbol)(using Context): Unit =
+  private def addMacroDependency(trees: List[Tree])(using Context): Unit =
     if (allowsDependencyByMacroExpansion) {
-      if (!ignoreDependency(sym)) {
-        if (!sym.is(Package)) {
-          val enclOrModuleClass = if (sym.is(ModuleVal)) sym.moduleClass else sym.enclosingClass
-          assert(enclOrModuleClass.isClass, s"$enclOrModuleClass, $sym")
+      val traverser = new TypeDependencyTraverser {
+        def addDependency(symbol: Symbol) =
+          if (!ignoreDependency(symbol)) {
+            val enclOrModuleClass = if (symbol.is(ModuleVal)) symbol.moduleClass else symbol.enclosingClass
+            assert(enclOrModuleClass.isClass, s"$enclOrModuleClass, $symbol")
 
-          rec.addClassDependency(enclOrModuleClass, DependencyByMacroExpansion)
-        }
+            rec.addClassDependency(enclOrModuleClass, DependencyByMacroExpansion)
+          }
       }
+      trees.foreach(tree => traverser.traverse(tree.tpe))
     }
 
-  private def addMacroDependency(trees: List[Tree])(using Context): Unit =
-    val traverser = new TypeDependencyTraverser {
-      def addDependency(symbol: Symbol) = addMacroDependency(symbol)
-    }
-    trees.foreach(tree => traverser.traverse(tree.tpe))
-
   private def depContextOf(cls: Symbol)(using Context): DependencyContext =
     if cls.isLocal then LocalDependencyByInheritance
     else DependencyByInheritance