Issue #12998 ctx.docCtx.get -> ctx.docCtx.foreach

GavinRay97 · GavinRay97 · commit 2f4ca43a2c42 · 2021-07-05T12:45:41.000-04:00
Changes based on suggestion in the issue and the implementation currently seen here: https://github.com/lampepfl/dotty/blob/79fae194b80e136a243e7c7f949268166d169e1e/compiler/src/dotty/tools/dotc/typer/Namer.scala#L449
diff --git a/compiler/src/dotty/tools/dotc/core/Definitions.scala b/compiler/src/dotty/tools/dotc/core/Definitions.scala
@@ -1781,7 +1781,7 @@ class Definitions {
   }
 
   def addSyntheticSymbolsComments(using Context): Unit =
-    def add(sym: Symbol, doc: String) = ctx.docCtx.get.addDocstring(sym, Some(Comment(NoSpan, doc)))
+    def add(sym: Symbol, doc: String) = ctx.docCtx.foreach(_.addDocstring(sym, Some(Comment(NoSpan, doc))))
 
     add(AnyClass,
     """/** Class `Any` is the root of the Scala class hierarchy.  Every class in a Scala

Original file line number	Diff line number	Diff line change
`@@ -1781,7 +1781,7 @@ class Definitions {`
`1781`	`1781`	`}`
`1782`	`1782`
`1783`	`1783`	`def addSyntheticSymbolsComments(using Context): Unit =`
`1784`		`- def add(sym: Symbol, doc: String) = ctx.docCtx.get.addDocstring(sym, Some(Comment(NoSpan, doc)))`
	`1784`	`+ def add(sym: Symbol, doc: String) = ctx.docCtx.foreach(_.addDocstring(sym, Some(Comment(NoSpan, doc))))`
`1785`	`1785`
`1786`	`1786`	`add(AnyClass,`
`1787`	`1787`	"""/** Class `Any` is the root of the Scala class hierarchy. Every class in a Scala