epic/OpenRA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix CA2265

Browse Source
This commit is contained in:
RoosterDragon
2024-11-13 19:07:09 +00:00
committed by Pavel Penev
parent e2296ad2d1
commit cf7f57252e
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
.editorconfig
View File

@@ -1049,6 +1049,9 @@ dotnet_diagnostic.CA2263.severity = none # TODO: Change to warning once mono is
# Do not pass a non-nullable value to 'ArgumentNullException.ThrowIfNull'.
dotnet_diagnostic.CA2264.severity = warning
# Do not compare 'Span<T>' to 'null' or 'default'.
dotnet_diagnostic.CA2265.severity = warning
### Roslynator.Analyzers
### https://josefpihrt.github.io/docs/roslynator/analyzers

2
OpenRA.Game/MiniYaml.cs
View File

@@ -333,7 +333,7 @@ namespace OpenRA
// Note: We need to support empty comments here to ensure that empty comments
// (i.e. a lone # at the end of a line) can be correctly re-serialized
var commentString = comment == default ? null : comment.ToString();
var commentString = comment == ReadOnlySpan<char>.Empty ? null : comment.ToString();
keyString = keyString == null ? null : stringPool.GetOrAdd(keyString);
valueString = valueString == null ? null : stringPool.GetOrAdd(valueString);