Fix typos and comment styling

Author: tamasvajk

csharp/ql/src/Linq/Helpers.qll

@@ -27,10 +27,10 @@ predicate isIEnumerableType(ValueOrRefType t) {
 }
 
 /**
- * Holds if `ForeachStmt` could be converted to a `.All()` call. That is,
- * the `ForeachStmt` contains a single `if` with a condition that accesses
- * the loop variable and with a body that assigns `false` to a variable and
- * `break`s out of the `foreach`.
+ * Holds if `foreach` statement `fes` could be converted to a `.All()` call.
+ * That is, the `ForeachStmt` contains a single `if` with a condition that
+ * accesses the loop variable and with a body that assigns `false` to a variable
+ * and `break`s out of the `foreach`.
  */
 predicate missedAllOpportunity(ForeachStmt fes) {
   exists(IfStmt is |
@@ -51,9 +51,10 @@ predicate missedAllOpportunity(ForeachStmt fes) {
 }
 
 /**
- * Holds if `ForeachStmt` could be converted to a `.Cast()` call. That is, the
- * loop variable is accessed only in the first statement of the block, and the
- * access is a cast. The first statement needs to be a `LocalVariableDeclStmt`.
+ * Holds if `foreach` statement `fes` could be converted to a `.Cast()` call.
+ * That is, the loop variable is accessed only in the first statement of the
+ * block, and the access is a cast. The first statement needs to be a
+ * `LocalVariableDeclStmt`.
  */
 predicate missedCastOpportunity(ForeachStmt fes, LocalVariableDeclStmt s) {
   s = firstStmt(fes) and
@@ -67,10 +68,10 @@ predicate missedCastOpportunity(ForeachStmt fes, LocalVariableDeclStmt s) {
 }
 
 /**
- * Holds if `ForeachStmt` could be converted to an `.OfType()` call. That is, the
- * loop variable is accessed only in the first statement of the block, and the
- * access is a cast with the `as` operator. The first statement needs to be a
- * `LocalVariableDeclStmt`.
+ * Holds if `foreach` statement `fes` could be converted to an `.OfType()` call.
+ * That is, the loop variable is accessed only in the first statement of the
+ * block, and the access is a cast with the `as` operator. The first statement
+ * needs to be a `LocalVariableDeclStmt`.
  */
 predicate missedOfTypeOpportunity(ForeachStmt fes, LocalVariableDeclStmt s) {
   s = firstStmt(fes) and
@@ -84,9 +85,10 @@ predicate missedOfTypeOpportunity(ForeachStmt fes, LocalVariableDeclStmt s) {
 }
 
 /**
- * Holds if `ForeachStmt` could be converted to a `.Select()` call. That is, the
- * loop variable is accessed only in the first statement of the block, and the
- * access is not a cast. The first statement needs to be a `LocalVariableDeclStmt`.
+ * Holds if `foreach` statement `fes` could be converted to a `.Select()` call.
+ * That is, the loop variable is accessed only in the first statement of the
+ * block, and the access is not a cast. The first statement needs to be a
+ * `LocalVariableDeclStmt`.
  */
 predicate missedSelectOpportunity(ForeachStmt fes, LocalVariableDeclStmt s) {
   s = firstStmt(fes) and
@@ -97,10 +99,10 @@ predicate missedSelectOpportunity(ForeachStmt fes, LocalVariableDeclStmt s) {
 }
 
 /**
- * Holds if `ForeachStmt` could be converted to a `.Where()` call. That is, first
- * statement of the loop is an `if`, which accesses the loop variable, and the body
- * of the `if` is either a `continue` or there's nothing else in the loop than the
- * `if`.
+ * Holds if `foreach` statement `fes` could be converted to a `.Where()` call.
+ * That is, first statement of the loop is an `if`, which accesses the loop
+ * variable, and the body of the `if` is either a `continue` or there's nothing
+ * else in the loop than the `if`.
  */
 predicate missedWhereOpportunity(ForeachStmt fes, IfStmt is) {
   // The very first thing the foreach loop does is test its iteration variable.

csharp/ql/src/external/CodeDuplication.qll

@@ -276,7 +276,7 @@ predicate mostlyDuplicateClass(Class c, Class other, string message) {
   )
 }
 
-/** Holds if the two files are similar or duplated. */
+/** Holds if the two files are similar or duplicated. */
 predicate fileLevelDuplication(File f, File other) {
   similarFiles(f, other, _) or duplicateFiles(f, other, _)
 }

csharp/ql/src/semmle/code/cil/Access.qll

@@ -31,7 +31,7 @@ class ReadRefAccess extends ReadAccess, ReadRef { }
 
 /** An instruction that writes a variable. */
 class WriteAccess extends VariableAccess, @cil_write_access {
-  /** Gets the `Expr` whose value is used in this variable write. */
+  /** Gets the expression whose value is used in this variable write. */
   Expr getExpr() { none() }
 }
 

csharp/ql/src/semmle/code/csharp/exprs/Expr.qll

@@ -1011,7 +1011,7 @@ class RangeExpr extends Expr, @range_expr {
 
 /** An index expression, for example `^1` meaning "1 from the end". */
 class IndexExpr extends Expr, @index_expr {
-  /** Gets the sub expression of this `IndexExpr`. */
+  /** Gets the sub expression of this index expression. */
   Expr getExpr() { result.getParent() = this }
 
   override string toString() { result = "^..." }

csharp/ql/src/semmle/code/csharp/frameworks/EntityFramework.qll

@@ -8,7 +8,10 @@ private import semmle.code.csharp.frameworks.system.collections.Generic
 private import semmle.code.csharp.frameworks.Sql
 private import semmle.code.csharp.dataflow.LibraryTypeDataFlow
 
-/** Definitions relating to the `System.ComponentModel.DataAnnotations`. */
+/**
+ * Definitions relating to the `System.ComponentModel.DataAnnotations`
+ * namespace.
+ */
 module DataAnnotations {
   /** Class for `NotMappedAttribute`. */
   class NotMappedAttribute extends Attribute {
@@ -22,7 +25,7 @@ module DataAnnotations {
 
 /**
  * Definitions relating to the `Microsoft.EntityFrameworkCore` or
- * `System.Data.Entity`.
+ * `System.Data.Entity` namespaces.
  */
 module EntityFramework {
   /** An EF6 or EFCore namespace. */

csharp/ql/src/semmle/code/csharp/frameworks/system/Linq.qll

@@ -5,7 +5,7 @@
 private import csharp as csharp
 private import semmle.code.csharp.frameworks.System as System
 
-/** Definitions relating to the `System.Linq`. */
+/** Definitions relating to the `System.Linq` namespace. */
 module SystemLinq {
   /** The `System.Linq` namespace. */
   class Namespace extends csharp::Namespace {

csharp/ql/src/semmle/code/csharp/frameworks/system/data/Common.qll

@@ -5,7 +5,7 @@
 private import csharp as csharp
 private import semmle.code.csharp.frameworks.system.Data as Data
 
-/** Definitions relating to the `System.Data.Common`. */
+/** Definitions relating to the `System.Data.Common` namespace. */
 module SystemDataCommon {
   /** The `System.Data.Common` namespace. */
   class Namespace extends csharp::Namespace {

csharp/ql/src/semmle/code/csharp/frameworks/system/data/Entity.qll

@@ -5,7 +5,7 @@
 private import csharp as csharp
 private import semmle.code.csharp.frameworks.system.Data as Data
 
-/** Definitions relating to the `System.Data.Entity`. */
+/** Definitions relating to the `System.Data.Entity` namespace. */
 module SystemDataEntity {
   /** The `System.Data.Entity` namespace. */
   class Namespace extends csharp::Namespace {
@@ -79,7 +79,7 @@ module SystemDataEntity {
   }
 }
 
-/** Definitions relating to the `System.Data.Entity.Infrastructure`. */
+/** Definitions relating to the `System.Data.Entity.Infrastructure` namespace. */
 module SystemDataEntityInfrastructure {
   /** The `System.Data.Entity.Infrastructure` namespace. */
   class Namespace extends csharp::Namespace {

csharp/ql/src/semmle/code/csharp/frameworks/system/linq/Expressions.qll

@@ -5,7 +5,7 @@
 private import csharp as csharp
 private import semmle.code.csharp.frameworks.system.Linq
 
-/** Definitions relating to the `System.Linq.Expressions`. */
+/** Definitions relating to the `System.Linq.Expressions` namespace. */
 module SystemLinqExpressions {
   /** The `System.Linq.Expressions` namespace. */
   class Namespace extends csharp::Namespace {

csharp/ql/src/semmle/code/csharp/frameworks/system/xml/XPath.qll

@@ -3,7 +3,7 @@
 import csharp as csharp
 private import semmle.code.csharp.frameworks.system.Xml as xml
 
-/** Definitions relating to the `System.Xml.XPath`. */
+/** Definitions relating to the `System.Xml.XPath` namespace. */
 module SystemXmlXPath {
   /** The `System.Xml.XPath` namespace. */
   class Namespace extends csharp::Namespace {