Simplify StringSpan initialization

JunaMeinhold · JunaMeinhold · commit e2ac266b2f97 · 2025-12-07T15:54:05.000+01:00
Remove unnecessary cast to `(int)` in `StringSpan` constructor. The `length - 1` value is now passed directly, improving code clarity and maintainability. This change ensures type consistency and eliminates redundant operations.
diff --git a/Hexa.NET.Utilities/StringPool.cs b/Hexa.NET.Utilities/StringPool.cs
@@ -14,7 +14,7 @@ public StringSpan Take(ReadOnlySpan<byte> span)
             byte* ptr = (byte*)allocator.Alloc(length, 1);
             span.CopyTo(new Span<byte>(ptr, span.Length));
             ptr[span.Length] = 0;
-            StringSpan stringSpan = new(ptr, (int)length - 1);
+            StringSpan stringSpan = new(ptr, length - 1);
             var res = strings.AddIt(stringSpan);
             if (!res.Added)
             {

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ public StringSpan Take(ReadOnlySpan<byte> span)`
`14`	`14`	`byte* ptr = (byte*)allocator.Alloc(length, 1);`
`15`	`15`	`span.CopyTo(new Span<byte>(ptr, span.Length));`
`16`	`16`	`ptr[span.Length] = 0;`
`17`		`- StringSpan stringSpan = new(ptr, (int)length - 1);`
	`17`	`+ StringSpan stringSpan = new(ptr, length - 1);`
`18`	`18`	`var res = strings.AddIt(stringSpan);`
`19`	`19`	`if (!res.Added)`
`20`	`20`	`{`