cleanup

Author: petesramek

benchmarks/PolylineAlgorithm.Benchmarks/PolylineBenchmark.cs

@@ -17,7 +17,7 @@ namespace PolylineAlgorithm.Benchmarks;
 public class PolylineBenchmark {
     private static readonly Consumer consumer = new();
 
-    [Params(1, 25, 50, 100, 250, 500, 1_000, 5_000, 10_000, 25_000, 50_000, 100_000, 500_000, 1_000_000)]
+    [Params(1, 25, 50, 100, 250, 500, 1_000)]
     public int Count;
 
 #pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.

benchmarks/PolylineAlgorithm.Benchmarks/PolylineDecoderBenchmark.cs

@@ -17,7 +17,7 @@ namespace PolylineAlgorithm.Benchmarks;
 public class PolylineDecoderBenchmark {
     private readonly Consumer _consumer = new();
 
-    [Params(1, 25, 50, 100, 250, 500, 1_000, 5_000, 10_000, 25_000, 50_000, 100_000, 500_000, 1_000_000)]
+    [Params(1, 25, 50, 100, 250, 500, 1_000)]
     public int Count;
 
 #pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.

benchmarks/PolylineAlgorithm.Benchmarks/PolylineEncoderBenchmark.cs

@@ -15,7 +15,7 @@ namespace PolylineAlgorithm.Benchmarks;
 /// </summary>
 [RankColumn]
 public class PolylineEncoderBenchmark {
-    [Params(1, 25, 50, 100, 250, 500, 1_000, 5_000, 10_000, 25_000, 50_000, 100_000, 500_000, 1_000_000)]
+    [Params(1, 25, 50, 100, 250, 500, 1_000)]
     public int Count;
 
 #pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
@@ -35,11 +35,6 @@ public class PolylineEncoderBenchmark {
     /// </summary>
     public PolylineEncoder Encoder = new();
 
-    /// <summary>
-    /// The async polyline encoder instance.
-    /// </summary>
-    //public AsyncPolylineEncoder AsyncEncoder = new();
-
     /// <summary>
     /// Sets up the data for the benchmarks.
     /// </summary>

benchmarks/PolylineAlgorithm.Benchmarks/Program.cs

@@ -5,7 +5,10 @@
 
 namespace PolylineAlgorithm.Benchmarks;
 
+using BenchmarkDotNet.Configs;
+using BenchmarkDotNet.Engines;
 using BenchmarkDotNet.Running;
+using Microsoft.CodeAnalysis;
 
 /// <summary>
 /// The main entry point for the benchmark application.
@@ -16,10 +19,8 @@ internal class Program {
     /// </summary>
     /// <param name="args">The command-line arguments.</param>
     static void Main(string[] args) {
-        //Directory.Delete("C:\\temp\\benchmark", true);
-
         BenchmarkSwitcher
             .FromAssembly(typeof(Program).Assembly)
-            .Run(args/*, new DebugInProcessConfig()*/);
+            .Run(args, new DebugInProcessConfig());
     }
 }

src/PolylineAlgorithm.Abstraction/PolylineEncoder.cs

@@ -56,8 +56,8 @@ public TPolyline Encode(IEnumerable<TCoordinate> coordinates) {
         int consumed = 0;
         int length = count * Defaults.Polyline.MaxEncodedCoordinateLength;
 
-        if (length > Options.MaxCharCount) {
-            length = Options.MaxCharCount;
+        if (length > Options.BufferSize / sizeof(char)) {
+            length = Options.BufferSize / sizeof(char);
         }
 
         Span<char> buffer = stackalloc char[length];

src/PolylineAlgorithm.Abstraction/PolylineEncodingOptions.cs

@@ -5,6 +5,7 @@
 
 namespace PolylineAlgorithm.Abstraction;
 
+using PolylineAlgorithm.Abstraction.Validation;
 using PolylineAlgorithm.Abstraction.Validation.Abstraction;
 
 /// <summary>
@@ -15,16 +16,10 @@ public class PolylineEncodingOptions<TCoordinate> {
     /// <summary>
     /// Gets the maximum buffer size for encoding operations.
     /// </summary>
-    public int BufferSize { get; internal set; }
-
-    /// <summary>
-    /// Gets the maximum number of characters that can be used in the encoding buffer.
-    /// </summary>
-    /// <returns>The maximum character count based on the buffer size.</returns>
-    public int MaxCharCount => BufferSize / sizeof(char);
+    public int BufferSize { get; internal set; } = 64_000;
 
     /// <summary>
     /// Gets the validator used to validate coordinates, latitude, and longitude values.
     /// </summary>
-    public Validator<TCoordinate> Validator { get; internal set; }
+    public Validator<TCoordinate> Validator { get; internal set; } = new NullValidator<TCoordinate>();
 }