Slightly more cleanup and clarification

This commit is contained in:
Matt Nadareski
2026-01-27 09:13:30 -05:00
parent 3211f1a218
commit a334ffc25a
7 changed files with 24 additions and 26 deletions

View File

@@ -1,5 +1,4 @@
using System;
using System.Collections.Generic;
using System.IO;
using Xunit;
@@ -16,15 +15,15 @@ namespace SabreTools.Hashing.Test
/// <summary>
/// Get an array of all hash types
/// </summary>
public static List<object[]> AllHashTypes
public static TheoryData<HashType> AllHashTypes
{
get
{
var values = Enum.GetValues<HashType>();
var set = new List<object[]>();
var set = new TheoryData<HashType>();
foreach (var value in values)
{
set.Add([value]);
set.Add(value);
}
return set;

View File

@@ -1,5 +1,4 @@
using System;
using System.Collections.Generic;
using System.Linq;
using Xunit;
@@ -10,15 +9,15 @@ namespace SabreTools.Hashing.Test
/// <summary>
/// Get an array of all hash types
/// </summary>
public static List<object[]> AllHashTypes
public static TheoryData<HashType> AllHashTypes
{
get
{
var values = Enum.GetValues<HashType>();
var set = new List<object[]>();
var set = new TheoryData<HashType>();
foreach (var value in values)
{
set.Add([value]);
set.Add(value);
}
return set;

View File

@@ -163,6 +163,7 @@ internal static class Comparisons
secondTraverse[secondIndex + 1] = Math.Min(Math.Min(costA, costD), costR);
}
#pragma warning disable IDE0180 // Use tuple to swap values
#if NETCOREAPP || NETSTANDARD2_0_OR_GREATER
(secondTraverse, firstTraverse) = (firstTraverse, secondTraverse);
#else
@@ -170,6 +171,7 @@ internal static class Comparisons
firstTraverse = secondTraverse;
secondTraverse = tempArray;
#endif
#pragma warning restore IDE0180
}
long score = firstTraverse[second.Length];

View File

@@ -2,6 +2,7 @@ using System;
using System.Text;
using static SabreTools.Hashing.SpamSum.Constants;
#pragma warning disable IDE0059 // Unnecessary assignment of a value
namespace SabreTools.Hashing.SpamSum
{
/// <see href="https://github.com/ssdeep-project/ssdeep/blob/master/fuzzy.c"/>

View File

@@ -1,3 +1,4 @@
#pragma warning disable IDE0051 // Remove unused private members
namespace SabreTools.Hashing.XxHash
{
// https://github.com/Cyan4973/xxHash/blob/dev/xxhash.h
@@ -63,9 +64,7 @@ namespace SabreTools.Hashing.XxHash
/// </summary>
public const int XXH_SECRET_CONSUME_RATE = 8;
#pragma warning disable IDE0051
private const int XXH_ACC_NB = XXH_STRIPE_LEN / sizeof(ulong);
#pragma warning restore IDE0051
#endregion
}

View File

@@ -1,13 +1,12 @@
#pragma warning disable CS0169 // Private field is never used
#pragma warning disable CS0414 // Private field is assigned but its value is never used
#pragma warning disable CS0649 // Field is never assigned to
#pragma warning disable IDE0044 // Add readonly modifier
#pragma warning disable IDE0051 // Remove unused private members
#pragma warning disable IDE0052 // Remove unread private members
#pragma warning disable IDE0060 // Remove unused parameter
namespace SabreTools.Hashing.XxHash
{
// Handle unused private fields
#pragma warning disable CS0169
#pragma warning disable CS0414
#pragma warning disable CS0649
#pragma warning disable IDE0044
#pragma warning disable IDE0051
#pragma warning disable IDE0052
/// <summary>
/// Structure for XXH3 streaming API.
/// </summary>

View File

@@ -1,13 +1,12 @@
#pragma warning disable CS0169 // Private field is never used
#pragma warning disable CS0414 // Private field is assigned but its value is never used
#pragma warning disable CS0649 // Field is never assigned to
#pragma warning disable IDE0044 // Add readonly modifier
#pragma warning disable IDE0051 // Remove unused private members
#pragma warning disable IDE0052 // Remove unread private members
#pragma warning disable IDE0060 // Remove unused parameter
namespace SabreTools.Hashing.XxHash
{
// Handle unused private fields
#pragma warning disable CS0169
#pragma warning disable CS0414
#pragma warning disable CS0649
#pragma warning disable IDE0044
#pragma warning disable IDE0051
#pragma warning disable IDE0052
/// <summary>
/// Structure for XXH3 streaming API.
/// </summary>