Fix condition in Debug.Assert in TarEntry.cs (#100860)

* Fix condition in Debug.Assert

The precedence of `not` is higher than of `or`, so without parentheses the actual condition was `((not RegularFile) or V7RegularFile or ContigousFile)`, which is not what this function actually expects (see switch cases and the condition in caller function).

* Fix assert condition in UnconditionalSuppressMessageAttributeState.cs

Similar to previous commit

---------

Co-authored-by: Stephen Toub <[email protected]>

Author: usr-sse2

src/libraries/System.Formats.Tar/src/System/Formats/Tar/TarEntry.cs

@@ -439,7 +439,7 @@ private Task ExtractToFileInternalAsync(string filePath, string? linkTargetPath,
 
         private void CreateNonRegularFile(string filePath, string? linkTargetPath)
         {
-            Debug.Assert(EntryType is not TarEntryType.RegularFile or TarEntryType.V7RegularFile or TarEntryType.ContiguousFile);
+            Debug.Assert(EntryType is not (TarEntryType.RegularFile or TarEntryType.V7RegularFile or TarEntryType.ContiguousFile));
 
             switch (EntryType)
             {

src/tools/illink/src/linker/Linker/UnconditionalSuppressMessageAttributeState.cs

@@ -257,7 +257,7 @@ static bool TryDecodeSuppressMessageAttributeData (CustomAttribute attribute, ou
 
 		IEnumerable<Suppression> DecodeSuppressions (ICustomAttributeProvider provider)
 		{
-			Debug.Assert (provider is not ModuleDefinition or AssemblyDefinition);
+			Debug.Assert (provider is not (ModuleDefinition or AssemblyDefinition));
 
 			if (!_context.CustomAttributes.HasAny (provider))
 				yield break;