Revert "Create DoNotCompareSpanToNullAnalyzer (dotnet#6838)"

This reverts commit 3e60082.
CollinAlpert · Mar 6, 2024 · 4bb858c · 4bb858c
1 parent aad8699
commit 4bb858c
Show file tree

Hide file tree

Showing 24 changed files with 2 additions and 853 deletions.
diff --git a/...tAnalyzers/CSharp/Microsoft.NetCore.Analyzers/Usage/CSharpDoNotCompareSpanToNull.Fixer.cs b/...tAnalyzers/CSharp/Microsoft.NetCore.Analyzers/Usage/CSharpDoNotCompareSpanToNull.Fixer.cs
diff --git a/src/NetAnalyzers/Core/AnalyzerReleases.Unshipped.md b/src/NetAnalyzers/Core/AnalyzerReleases.Unshipped.md
@@ -10,4 +10,3 @@ CA1871 | Performance | Info | DoNotPassNonNullableValueToArgumentNullExceptionTh
 CA2262 | Usage | Info | ProvideHttpClientHandlerMaxResponseHeaderLengthValueCorrectly, [Documentation](https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2262)
 CA2263 | Usage | Info | PreferGenericOverloadsAnalyzer, [Documentation](https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2263)
 CA2264 | Usage | Warning | DoNotPassNonNullableValueToArgumentNullExceptionThrowIfNull, [Documentation](https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2264)
-CA2265 | Usage | Warning | DoNotCompareSpanToNullAnalyzer, [Documentation](https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2265)
diff --git a/src/NetAnalyzers/Core/Microsoft.NetCore.Analyzers/MicrosoftNetCoreAnalyzersResources.resx b/src/NetAnalyzers/Core/Microsoft.NetCore.Analyzers/MicrosoftNetCoreAnalyzersResources.resx
@@ -2132,19 +2132,4 @@ Widening and user defined conversions are not supported with generic types.</val
   <data name="DoNotPassNullableStructToArgumentNullExceptionThrowIfNullCodeFixTitle" xml:space="preserve">
     <value>Replace the 'ArgumentNullException.ThrowIfNull' call with a conditional</value>
   </data>
-  <data name="DoNotCompareSpanToNullOrDefaultDescription" xml:space="preserve">
-    <value>Comparing a span to 'null' or 'default' might not do what you intended. 'default' and the 'null' literal are implicitly converted to 'Span&lt;T&gt;.Empty'. Remove the redundant comparison or make the code more explicit by using 'IsEmpty'.</value>
-  </data>
-  <data name="DoNotCompareSpanToNullMessage" xml:space="preserve">
-    <value>Comparing a span to 'null' might be redundant, the 'null' literal will be implicitly converted to a 'Span&lt;T&gt;.Empty'</value>
-  </data>
-  <data name="DoNotCompareSpanToDefaultMessage" xml:space="preserve">
-    <value>Comparing a span to 'default' might not do what you intended, make the code more explicit by checking 'IsEmpty'</value>
-  </data>
-  <data name="DoNotCompareSpanToNullOrDefaultTitle" xml:space="preserve">
-    <value>Do not compare Span&lt;T&gt; to 'null' or 'default'</value>
-  </data>
-  <data name="DoNotCompareSpanToNullIsEmptyCodeFixTitle" xml:space="preserve">
-    <value>Use 'IsEmpty'</value>
-  </data>
 </root>
diff --git a/src/NetAnalyzers/Core/Microsoft.NetCore.Analyzers/Usage/DoNotCompareSpanToNull.Fixer.cs b/src/NetAnalyzers/Core/Microsoft.NetCore.Analyzers/Usage/DoNotCompareSpanToNull.Fixer.cs
diff --git a/src/NetAnalyzers/Core/Microsoft.NetCore.Analyzers/Usage/DoNotCompareSpanToNull.cs b/src/NetAnalyzers/Core/Microsoft.NetCore.Analyzers/Usage/DoNotCompareSpanToNull.cs
diff --git a/...tAnalyzers/Core/Microsoft.NetCore.Analyzers/xlf/MicrosoftNetCoreAnalyzersResources.cs.xlf b/...tAnalyzers/Core/Microsoft.NetCore.Analyzers/xlf/MicrosoftNetCoreAnalyzersResources.cs.xlf
@@ -623,31 +623,6 @@ Obecné přetypování (IL unbox.any) používané sekvencí vrácenou metodou E
         <target state="translated">Nevolejte ToImmutableCollection pro hodnotu ImmutableCollection</target>
         <note />
       </trans-unit>
-      <trans-unit id="DoNotCompareSpanToDefaultMessage">
-        <source>Comparing a span to 'default' might not do what you intended, make the code more explicit by checking 'IsEmpty'</source>
-        <target state="new">Comparing a span to 'default' might not do what you intended, make the code more explicit by checking 'IsEmpty'</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullIsEmptyCodeFixTitle">
-        <source>Use 'IsEmpty'</source>
-        <target state="new">Use 'IsEmpty'</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullMessage">
-        <source>Comparing a span to 'null' might be redundant, the 'null' literal will be implicitly converted to a 'Span&lt;T&gt;.Empty'</source>
-        <target state="new">Comparing a span to 'null' might be redundant, the 'null' literal will be implicitly converted to a 'Span&lt;T&gt;.Empty'</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullOrDefaultDescription">
-        <source>Comparing a span to 'null' or 'default' might not do what you intended. 'default' and the 'null' literal are implicitly converted to 'Span&lt;T&gt;.Empty'. Remove the redundant comparison or make the code more explicit by using 'IsEmpty'.</source>
-        <target state="new">Comparing a span to 'null' or 'default' might not do what you intended. 'default' and the 'null' literal are implicitly converted to 'Span&lt;T&gt;.Empty'. Remove the redundant comparison or make the code more explicit by using 'IsEmpty'.</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullOrDefaultTitle">
-        <source>Do not compare Span&lt;T&gt; to 'null' or 'default'</source>
-        <target state="new">Do not compare Span&lt;T&gt; to 'null' or 'default'</target>
-        <note />
-      </trans-unit>
       <trans-unit id="DoNotCreateTaskCompletionSourceWithWrongArgumentsDescription">
         <source>TaskCompletionSource has constructors that take TaskCreationOptions that control the underlying Task, and constructors that take object state that's stored in the task.  Accidentally passing a TaskContinuationOptions instead of a TaskCreationOptions will result in the call treating the options as state.</source>
         <target state="translated">TaskCompletionSource má konstruktory, které přijímají možnosti TaskCreationOptions určující příslušnou úlohu, a konstruktory, které přijímají stav objektu uložený v úloze. Když se místo TaskCreationOptions omylem předá TaskContinuationOptions, způsobí to, že volání bude možnosti považovat za stav.</target>

diff --git a/...tAnalyzers/Core/Microsoft.NetCore.Analyzers/xlf/MicrosoftNetCoreAnalyzersResources.de.xlf b/...tAnalyzers/Core/Microsoft.NetCore.Analyzers/xlf/MicrosoftNetCoreAnalyzersResources.de.xlf
@@ -623,31 +623,6 @@ Erweiterungen und benutzerdefinierte Konvertierungen werden bei generischen Type
         <target state="translated">"ToImmutableCollection" nicht für einen ImmutableCollection-Wert aufrufen</target>
         <note />
       </trans-unit>
-      <trans-unit id="DoNotCompareSpanToDefaultMessage">
-        <source>Comparing a span to 'default' might not do what you intended, make the code more explicit by checking 'IsEmpty'</source>
-        <target state="new">Comparing a span to 'default' might not do what you intended, make the code more explicit by checking 'IsEmpty'</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullIsEmptyCodeFixTitle">
-        <source>Use 'IsEmpty'</source>
-        <target state="new">Use 'IsEmpty'</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullMessage">
-        <source>Comparing a span to 'null' might be redundant, the 'null' literal will be implicitly converted to a 'Span&lt;T&gt;.Empty'</source>
-        <target state="new">Comparing a span to 'null' might be redundant, the 'null' literal will be implicitly converted to a 'Span&lt;T&gt;.Empty'</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullOrDefaultDescription">
-        <source>Comparing a span to 'null' or 'default' might not do what you intended. 'default' and the 'null' literal are implicitly converted to 'Span&lt;T&gt;.Empty'. Remove the redundant comparison or make the code more explicit by using 'IsEmpty'.</source>
-        <target state="new">Comparing a span to 'null' or 'default' might not do what you intended. 'default' and the 'null' literal are implicitly converted to 'Span&lt;T&gt;.Empty'. Remove the redundant comparison or make the code more explicit by using 'IsEmpty'.</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullOrDefaultTitle">
-        <source>Do not compare Span&lt;T&gt; to 'null' or 'default'</source>
-        <target state="new">Do not compare Span&lt;T&gt; to 'null' or 'default'</target>
-        <note />
-      </trans-unit>
       <trans-unit id="DoNotCreateTaskCompletionSourceWithWrongArgumentsDescription">
         <source>TaskCompletionSource has constructors that take TaskCreationOptions that control the underlying Task, and constructors that take object state that's stored in the task.  Accidentally passing a TaskContinuationOptions instead of a TaskCreationOptions will result in the call treating the options as state.</source>
         <target state="translated">TaskCompletionSource verfügt über Konstruktoren, die TaskCreationOptions zum Steuern der zugrunde liegenden Aufgabe akzeptieren, und Konstruktoren, die den in der Aufgabe gespeicherten Objektzustand übernehmen. Das versehentliche Übergeben von TaskContinuationOptions anstelle von TaskCreationOptions führt dazu, dass der Aufruf die Optionen als Zustand behandelt.</target>

diff --git a/...tAnalyzers/Core/Microsoft.NetCore.Analyzers/xlf/MicrosoftNetCoreAnalyzersResources.es.xlf b/...tAnalyzers/Core/Microsoft.NetCore.Analyzers/xlf/MicrosoftNetCoreAnalyzersResources.es.xlf
@@ -623,31 +623,6 @@ La ampliación y las conversiones definidas por el usuario no se admiten con tip
         <target state="translated">No llame a ToImmutableCollection en un valor ImmutableCollection</target>
         <note />
       </trans-unit>
-      <trans-unit id="DoNotCompareSpanToDefaultMessage">
-        <source>Comparing a span to 'default' might not do what you intended, make the code more explicit by checking 'IsEmpty'</source>
-        <target state="new">Comparing a span to 'default' might not do what you intended, make the code more explicit by checking 'IsEmpty'</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullIsEmptyCodeFixTitle">
-        <source>Use 'IsEmpty'</source>
-        <target state="new">Use 'IsEmpty'</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullMessage">
-        <source>Comparing a span to 'null' might be redundant, the 'null' literal will be implicitly converted to a 'Span&lt;T&gt;.Empty'</source>
-        <target state="new">Comparing a span to 'null' might be redundant, the 'null' literal will be implicitly converted to a 'Span&lt;T&gt;.Empty'</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullOrDefaultDescription">
-        <source>Comparing a span to 'null' or 'default' might not do what you intended. 'default' and the 'null' literal are implicitly converted to 'Span&lt;T&gt;.Empty'. Remove the redundant comparison or make the code more explicit by using 'IsEmpty'.</source>
-        <target state="new">Comparing a span to 'null' or 'default' might not do what you intended. 'default' and the 'null' literal are implicitly converted to 'Span&lt;T&gt;.Empty'. Remove the redundant comparison or make the code more explicit by using 'IsEmpty'.</target>
-        <note />
-      </trans-unit>
-      <trans-unit id="DoNotCompareSpanToNullOrDefaultTitle">
-        <source>Do not compare Span&lt;T&gt; to 'null' or 'default'</source>
-        <target state="new">Do not compare Span&lt;T&gt; to 'null' or 'default'</target>
-        <note />
-      </trans-unit>
       <trans-unit id="DoNotCreateTaskCompletionSourceWithWrongArgumentsDescription">
         <source>TaskCompletionSource has constructors that take TaskCreationOptions that control the underlying Task, and constructors that take object state that's stored in the task.  Accidentally passing a TaskContinuationOptions instead of a TaskCreationOptions will result in the call treating the options as state.</source>
         <target state="translated">TaskCompletionSource tiene constructores que toman elementos TaskCreationOption para controlar la tarea subyacente y constructores que toman el estado del objeto que se almacena en la tarea. Si se pasa accidentalmente un elemento TaskContinuationOption en lugar de un objeto TaskCreationOption, la llamada trata las opciones como estado.</target>