Accept review comment - doc clarification

owen-mc · owen-mc · commit b2c2a94866bb · 2024-11-11T23:48:30.000Z
diff --git a/go/ql/lib/semmle/go/dataflow/internal/FlowSummaryImpl.qll b/go/ql/lib/semmle/go/dataflow/internal/FlowSummaryImpl.qll
@@ -149,9 +149,9 @@ module SourceSinkInterpretationInput implements
     )
   }
 
-  // Note that due to embedding, which is currently implemented via some Methods
-  // or Fields having multiple qualified names, a given Method or Field is liable
-  // to have more than one SourceOrSinkElement, one for each of the names it claims.
+  // Note that due to embedding, which is currently implemented via some
+  // Methods having multiple qualified names, a given Method is liable to have
+  // more than one SourceOrSinkElement, one for each of the names it claims.
   private newtype TSourceOrSinkElement =
     TMethodEntityElement(Method m, string pkg, string type, boolean subtypes) {
       m.hasQualifiedName(pkg, type, _) and subtypes = [true, false]

Original file line number	Diff line number	Diff line change
`@@ -149,9 +149,9 @@ module SourceSinkInterpretationInput implements`
`149`	`149`	`)`
`150`	`150`	`}`
`151`	`151`
`152`		`- // Note that due to embedding, which is currently implemented via some Methods`
`153`		`- // or Fields having multiple qualified names, a given Method or Field is liable`
`154`		`- // to have more than one SourceOrSinkElement, one for each of the names it claims.`
	`152`	`+ // Note that due to embedding, which is currently implemented via some`
	`153`	`+ // Methods having multiple qualified names, a given Method is liable to have`
	`154`	`+ // more than one SourceOrSinkElement, one for each of the names it claims.`
`155`	`155`	`private newtype TSourceOrSinkElement =`
`156`	`156`	`TMethodEntityElement(Method m, string pkg, string type, boolean subtypes) {`
`157`	`157`	`m.hasQualifiedName(pkg, type, _) and subtypes = [true, false]`