Skip to content

Commit 13798f8

Browse files
leecampbell-codeagentLeeCampbellclaude
authored
chore(#161): Drop netstandard2.0 target, support net8.0+ only (#164)
* plan(#161): initial brief from issue * plan(#161): review brief * plan(#161): apply brief review feedback * plan(#161): review brief * plan(#161): create task breakdown * feat(#161): implement tasks * feat(#161): complete implementation * fix(#161): address PR #164 review feedback - Remove unused `using System;` from Bitwise.cs - Convert BitwiseTests to FluentAssertions for consistency - Add negative value test case (-1L, expected: 0) - Fix build-system.md spec to match actual csproj (bin\Release\...) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Lee Campbell <lee.ryan.campbell@gmail.com> Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent fdea5d3 commit 13798f8

6 files changed

Lines changed: 55 additions & 123 deletions

File tree

‎HdrHistogram.Benchmarking/LeadingZeroCount/LeadingZeroCountBenchmarkBase.cs‎

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ public void OneOffValidationOfImplementations()
5353
var functions = new Dictionary<string, Func<long, int>>
5454
{
5555
{"CurrentImpl", Bitwise.NumberOfLeadingZeros},
56-
{"Imperative", Bitwise.Imperative.NumberOfLeadingZeros},
5756
{"IfAndShift", LeadingZeroCount.IfAndShift.GetLeadingZeroCount},
5857
//{"MathLog", LeadingZeroCount.MathLog.GetLeadingZeroCount},
5958
{"StringManipulation", LeadingZeroCount.StringManipulation.GetLeadingZeroCount},
@@ -121,17 +120,6 @@ public int CurrentImplementation()
121120
return sum;
122121
}
123122

124-
[Benchmark(OperationsPerInvoke = TEST_VALUE_LENGTH)]
125-
public int ImperativeImplementation()
126-
{
127-
var sum = 0;
128-
for (int i = 0; i < _testValues.Length; i++)
129-
{
130-
sum += Bitwise.Imperative.NumberOfLeadingZeros(_testValues[i]);
131-
}
132-
return sum;
133-
}
134-
135123
[Benchmark(OperationsPerInvoke = TEST_VALUE_LENGTH)]
136124
public int IfAndShift()
137125
{

‎HdrHistogram.UnitTests/Utilities/BitwiseTests.cs‎

Lines changed: 47 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,47 @@
1+
/*
2+
* This is a .NET port of the original Java version, which was written by
3+
* Gil Tene as described in
4+
* https://github.com/HdrHistogram/HdrHistogram
5+
* and released to the public domain, as explained at
6+
* http://creativecommons.org/publicdomain/zero/1.0/
7+
*/
8+
9+
using FluentAssertions;
10+
using HdrHistogram.Utilities;
11+
using Xunit;
12+
13+
namespace HdrHistogram.UnitTests.Utilities
14+
{
15+
public class BitwiseTests
16+
{
17+
[Theory]
18+
[InlineData(0L, 64)]
19+
[InlineData(1L, 63)]
20+
[InlineData(2L, 62)]
21+
[InlineData(4L, 61)]
22+
[InlineData(8L, 60)]
23+
[InlineData(16L, 59)]
24+
[InlineData(32L, 58)]
25+
[InlineData(64L, 57)]
26+
[InlineData(128L, 56)]
27+
[InlineData(256L, 55)]
28+
[InlineData(512L, 54)]
29+
[InlineData(1024L, 53)]
30+
[InlineData(2048L, 52)]
31+
[InlineData(4096L, 51)]
32+
[InlineData(8192L, 50)]
33+
[InlineData(16384L, 49)]
34+
[InlineData(32768L, 48)]
35+
[InlineData(65536L, 47)]
36+
[InlineData(1L << 30, 33)]
37+
[InlineData(1L << 31, 32)]
38+
[InlineData(1L << 32, 31)]
39+
[InlineData(1L << 62, 1)]
40+
[InlineData(long.MaxValue, 1)]
41+
[InlineData(-1L, 0)]
42+
public void NumberOfLeadingZeros_ReturnsCorrectValue(long value, int expected)
43+
{
44+
Bitwise.NumberOfLeadingZeros(value).Should().Be(expected);
45+
}
46+
}
47+
}

‎HdrHistogram/HdrHistogram.csproj‎

Lines changed: 5 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
1-
<Project Sdk="Microsoft.NET.Sdk">
1+
<Project Sdk="Microsoft.NET.Sdk">
22

33
<PropertyGroup>
4-
<TargetFrameworks>net10.0;net9.0;net8.0;netstandard2.0</TargetFrameworks>
4+
<TargetFrameworks>net10.0;net9.0;net8.0</TargetFrameworks>
55
<Description>HdrHistogram supports low latency recording and analyzing of sampled data value counts across a configurable integer value range with configurable value precision within the range.</Description>
66
<Authors>Gil Tene, Lee Campbell</Authors>
77
<PackageReleaseNotes>Net 8.0 release</PackageReleaseNotes>
@@ -21,21 +21,8 @@
2121
<None Include="../README.md" Pack="true" PackagePath="" />
2222
</ItemGroup>
2323

24-
<PropertyGroup Condition="'$(Configuration)|$(TargetFramework)|$(Platform)'=='Release|net8.0|AnyCPU'">
25-
<DocumentationFile>bin\Release\net8.0\HdrHistogram.xml</DocumentationFile>
24+
<PropertyGroup Condition="'$(Configuration)' == 'Release'">
25+
<DocumentationFile>bin\Release\$(TargetFramework)\HdrHistogram.xml</DocumentationFile>
2626
</PropertyGroup>
2727

28-
<PropertyGroup Condition="'$(Configuration)|$(TargetFramework)|$(Platform)'=='Release|net9.0|AnyCPU'">
29-
<DocumentationFile>bin\Release\net9.0\HdrHistogram.xml</DocumentationFile>
30-
</PropertyGroup>
31-
32-
<PropertyGroup Condition="'$(Configuration)|$(TargetFramework)|$(Platform)'=='Release|net10.0|AnyCPU'">
33-
<DocumentationFile>bin\Release\net10.0\HdrHistogram.xml</DocumentationFile>
34-
</PropertyGroup>
35-
36-
<PropertyGroup Condition="'$(Configuration)|$(TargetFramework)|$(Platform)'=='Release|netstandard2.0|AnyCPU'">
37-
<DocumentationFile>bin\Release\netstandard2.0\HdrHistogram.xml</DocumentationFile>
38-
<DefineConstants>RELEASE;NETSTANDARD2_0</DefineConstants>
39-
</PropertyGroup>
40-
41-
</Project>
28+
</Project>

‎HdrHistogram/HistogramLogReader.cs‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -240,11 +240,7 @@ private IEnumerable<string> ReadLines()
240240

241241
private static bool IsComment(string line)
242242
{
243-
#if NETSTANDARD2_0
244-
return line.StartsWith("#", StringComparison.Ordinal);
245-
#else
246243
return line.StartsWith('#');
247-
#endif
248244
}
249245

250246
private static bool IsStartTime(string line)

‎HdrHistogram/Utilities/Bitwise.cs‎

Lines changed: 1 addition & 86 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,6 @@
66
* http://creativecommons.org/publicdomain/zero/1.0/
77
*/
88

9-
using System;
10-
119
namespace HdrHistogram.Utilities
1210
{
1311
/// <summary>
@@ -22,90 +20,7 @@ public static class Bitwise
2220
/// <returns>The number of leading zeros.</returns>
2321
public static int NumberOfLeadingZeros(long value)
2422
{
25-
#if NET5_0_OR_GREATER
26-
return IntrinsicNumberOfLeadingZeros(value);
27-
#else
28-
return Imperative.NumberOfLeadingZeros(value);
29-
#endif
30-
}
31-
32-
#if NET5_0_OR_GREATER
33-
/// <summary>
34-
/// Returns the Leading Zero Count (lzc) of the <paramref name="value"/> for its binary representation.
35-
/// </summary>
36-
/// <param name="value">The value to find the number of leading zeros</param>
37-
/// <returns>The number of leading zeros.</returns>
38-
private static int IntrinsicNumberOfLeadingZeros(long value)
39-
{
40-
ulong testValue = (ulong)value;
41-
return System.Numerics.BitOperations.LeadingZeroCount(testValue);
42-
43-
}
44-
#endif
45-
46-
//Code has been tested and taken from :
47-
//http://stackoverflow.com/questions/9543410/i-dont-think-numberofleadingzeroslong-i-in-long-java-is-based-floorlog2x/9543537#9543537
48-
//http://stackoverflow.com/questions/21888140/de-bruijn-algorithm-binary-digit-count-64bits-c-sharp/21888542#21888542
49-
//http://stackoverflow.com/questions/15967240/fastest-implementation-of-log2int-and-log2float
50-
//http://graphics.stanford.edu/~seander/bithacks.html#IntegerLogObvious
51-
52-
/// <summary>
53-
/// Imperative implementation of the LeadingZeroCount, when access to the <c>System.Numerics.BitOperations.LeadingZeroCount(ulong)</c> is not available.
54-
/// </summary>
55-
public static class Imperative
56-
{
57-
private static readonly int[] Lookup;
58-
59-
static Imperative()
60-
{
61-
Lookup = new int[256];
62-
for (int i = 1; i < 256; ++i)
63-
{
64-
Lookup[i] = (int)(Math.Log(i) / Math.Log(2));
65-
}
66-
}
67-
68-
/// <summary>
69-
/// Returns the Leading Zero Count (lzc) of the <paramref name="value"/> for its binary representation.
70-
/// </summary>
71-
/// <param name="value">The value to find the number of leading zeros</param>
72-
/// <returns>The number of leading zeros.</returns>
73-
public static int NumberOfLeadingZeros(long value)
74-
{
75-
//Optimisation for 32 bit values. So values under 00:16:41.0 when measuring with Stopwatch.GetTimestamp()*, we will hit a fast path.
76-
// * as at writing on Win10 .NET 4.6
77-
if (value < int.MaxValue)
78-
return 63 - Log2((int)value);
79-
return NumberOfLeadingZerosLong(value);
80-
}
81-
82-
private static int NumberOfLeadingZerosLong(long value)
83-
{
84-
// Code from http://stackoverflow.com/questions/9543410/i-dont-think-numberofleadingzeroslong-i-in-long-java-is-based-floorlog2x/9543537#9543537
85-
86-
//--Already checked that values here are over int.MaxValue, i.e. !=0
87-
// HD, Figure 5-6
88-
//if (value == 0)
89-
// return 64;
90-
var n = 1;
91-
// >>> in Java is a "unsigned bit shift", to do the same in C# we use >> (but it HAS to be an unsigned int)
92-
var x = (uint)(value >> 32);
93-
if (x == 0) { n += 32; x = (uint)value; }
94-
if (x >> 16 == 0) { n += 16; x <<= 16; }
95-
if (x >> 24 == 0) { n += 8; x <<= 8; }
96-
if (x >> 28 == 0) { n += 4; x <<= 4; }
97-
if (x >> 30 == 0) { n += 2; x <<= 2; }
98-
n -= (int)(x >> 31);
99-
return n;
100-
}
101-
102-
private static int Log2(int i)
103-
{
104-
if (i >= 0x1000000) { return Lookup[i >> 24] + 24; }
105-
if (i >= 0x10000) { return Lookup[i >> 16] + 16; }
106-
if (i >= 0x100) { return Lookup[i >> 8] + 8; }
107-
return Lookup[i];
108-
}
23+
return System.Numerics.BitOperations.LeadingZeroCount((ulong)value);
10924
}
11025
}
11126
}

‎spec/tech-standards/build-system.md‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,15 +22,14 @@ HdrHistogram.sln
2222
### Main Library (HdrHistogram.csproj)
2323

2424
```xml
25-
<TargetFrameworks>net10.0;net9.0;net8.0;netstandard2.0</TargetFrameworks>
25+
<TargetFrameworks>net10.0;net9.0;net8.0</TargetFrameworks>
2626
```
2727

2828
| Target | Description |
2929
|--------|-------------|
3030
| `net10.0` | Modern .NET (current LTS target) |
3131
| `net9.0` | Modern .NET (STS target) |
3232
| `net8.0` | Modern .NET (LTS target) |
33-
| `netstandard2.0` | Broad compatibility (.NET Framework 4.6.1+, .NET Core 2.0+) |
3433

3534
### Test Project
3635

@@ -87,7 +86,7 @@ XML documentation is generated for Release builds:
8786

8887
```xml
8988
<PropertyGroup Condition="'$(Configuration)' == 'Release'">
90-
<DocumentationFile>bin\$(Configuration)\$(TargetFramework)\HdrHistogram.xml</DocumentationFile>
89+
<DocumentationFile>bin\Release\$(TargetFramework)\HdrHistogram.xml</DocumentationFile>
9190
</PropertyGroup>
9291
```
9392

0 commit comments

Comments
 (0)