Skip to content

Commit fe38417

Browse files
author
Felicity Chapman
committed
Update for feedback
1 parent eba8abe commit fe38417

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

change-notes/1.21/analysis-csharp.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@ C# analysis now supports the extraction and analysis of many C# 8 features. For
99
| **Query** | **Expected impact** | **Change** |
1010
|------------------------------|------------------------|-----------------------------------|
1111
| Class defines a field that uses an ICryptoTransform class in a way that would be unsafe for concurrent threads (`cs/thread-unsafe-icryptotransform-field-in-class`) | Fewer false positive results | The criteria for a result has changed to include nested properties, nested fields, and collections. The format of the alert message has changed to highlight the static field. |
12-
| Constant condition (`cs/constant-condition`) | Fewer false positive results | Results where the `null` value is in a conditional expression on the left hand side of a null-coalescing expression are now ignored. For example, in `(a ? b : null) ?? c`, `null` is not considered to be a constant condition. |
13-
| Useless upcast (`cs/useless-upcast`) | Fewer false positive results | Results where the upcast is used to disambiguate the target of a constructor call are now ignored. |
12+
| Constant condition (`cs/constant-condition`) | Fewer false positive results | The query now ignores code where the `null` value is in a conditional expression on the left hand side of a null-coalescing expression. For example, in `(a ? b : null) ?? c`, `null` is not considered to be a constant condition. |
13+
| Useless upcast (`cs/useless-upcast`) | Fewer false positive results | The query now ignores code where the upcast is used to disambiguate the target of a constructor call. |
1414

1515
## Changes to code extraction
1616

0 commit comments

Comments
 (0)