Merge pull request #35 from simdutf/clean_benchmark #168
lemireAnnotations
30 warnings
|
Build and test on macos-latest:
src/UTF8.cs#L737
The variable 'TempUtf16CodeUnitCountAdjustment' is assigned but its value is never used
|
|
Build and test on macos-latest:
src/UTF8.cs#L738
The variable 'TempScalarCountAdjustment' is assigned but its value is never used
|
|
Build and test on macos-latest:
src/UTF8.cs#L294
In member SimdUnicode.UTF8.calculateErrorPathadjust(int, int, byte*, int, int, int), remove the underscores from parameter name start_point (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1707)
|
|
Build and test on macos-latest:
src/Ascii.cs#L28
In externally visible method 'bool Ascii.IsAscii(string s)', validate parameter 's' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
|
|
Build and test on macos-latest:
benchmark/Benchmark.cs#L49
Member 'PriorityInCategory' is explicitly initialized to its default value (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1805)
|
|
Build and test on macos-latest:
benchmark/Benchmark.cs#L50
Member 'IsNumeric' is explicitly initialized to its default value (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1805)
|
|
Build and test on macos-latest:
benchmark/Benchmark.cs#L133
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
|
|
Build and test on macos-latest:
benchmark/Benchmark.cs#L134
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
|
|
Build and test on macos-latest:
benchmark/Benchmark.cs#L134
Avoid unnecessary zero-length array allocations. Use Array.Empty<byte>() instead. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1825)
|
|
Build and test on macos-latest:
benchmark/Benchmark.cs#L149
Exception type System.Exception is not sufficiently specific (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2201)
|
|
Build and test on ubuntu-latest:
src/UTF8.cs#L737
The variable 'TempUtf16CodeUnitCountAdjustment' is assigned but its value is never used
|
|
Build and test on ubuntu-latest:
src/UTF8.cs#L738
The variable 'TempScalarCountAdjustment' is assigned but its value is never used
|
|
Build and test on ubuntu-latest:
src/UTF8.cs#L294
In member SimdUnicode.UTF8.calculateErrorPathadjust(int, int, byte*, int, int, int), remove the underscores from parameter name start_point (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1707)
|
|
Build and test on ubuntu-latest:
src/Ascii.cs#L28
In externally visible method 'bool Ascii.IsAscii(string s)', validate parameter 's' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
|
|
Build and test on ubuntu-latest:
benchmark/Benchmark.cs#L49
Member 'PriorityInCategory' is explicitly initialized to its default value (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1805)
|
|
Build and test on ubuntu-latest:
benchmark/Benchmark.cs#L50
Member 'IsNumeric' is explicitly initialized to its default value (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1805)
|
|
Build and test on ubuntu-latest:
benchmark/Benchmark.cs#L134
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
|
|
Build and test on ubuntu-latest:
benchmark/Benchmark.cs#L134
Avoid unnecessary zero-length array allocations. Use Array.Empty<byte>() instead. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1825)
|
|
Build and test on ubuntu-latest:
benchmark/Benchmark.cs#L133
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
|
|
Build and test on ubuntu-latest:
benchmark/Benchmark.cs#L149
Exception type System.Exception is not sufficiently specific (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2201)
|
|
Build and test on windows-latest:
src/UTF8.cs#L737
The variable 'TempUtf16CodeUnitCountAdjustment' is assigned but its value is never used
|
|
Build and test on windows-latest:
src/UTF8.cs#L738
The variable 'TempScalarCountAdjustment' is assigned but its value is never used
|
|
Build and test on windows-latest:
src/UTF8.cs#L294
In member SimdUnicode.UTF8.calculateErrorPathadjust(int, int, byte*, int, int, int), remove the underscores from parameter name start_point (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1707)
|
|
Build and test on windows-latest:
src/Ascii.cs#L28
In externally visible method 'bool Ascii.IsAscii(string s)', validate parameter 's' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
|
|
Build and test on windows-latest:
benchmark/Benchmark.cs#L49
Member 'PriorityInCategory' is explicitly initialized to its default value (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1805)
|
|
Build and test on windows-latest:
benchmark/Benchmark.cs#L50
Member 'IsNumeric' is explicitly initialized to its default value (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1805)
|
|
Build and test on windows-latest:
benchmark/Benchmark.cs#L133
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
|
|
Build and test on windows-latest:
benchmark/Benchmark.cs#L134
Do not declare visible instance fields (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1051)
|
|
Build and test on windows-latest:
benchmark/Benchmark.cs#L134
Avoid unnecessary zero-length array allocations. Use Array.Empty<byte>() instead. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1825)
|
|
Build and test on windows-latest:
benchmark/Benchmark.cs#L31
In externally visible method 'string Speed.GetValue(Summary summary, BenchmarkCase benchmarkCase)', validate parameter 'benchmarkCase' is non-null before using it. If appropriate, throw an 'ArgumentNullException' when the argument is 'null'. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1062)
|