code cleanup

Author: petesramek

samples/PolylineAlgorithm.NetTopologySuite.Sample/NetTopologyPolylineEncoder.cs

@@ -7,8 +7,6 @@ namespace PolylineAlgorithm.NetTopologySuite.Sample;
 
 using global::NetTopologySuite.Geometries;
 using PolylineAlgorithm.Abstraction;
-using System.Buffers;
-using System.Text;
 
 internal class NetTopologyPolylineEncoder : AbstractPolylineEncoder<Point, string> {
 

src/PolylineAlgorithm.Abstraction/AbstractPolylineDecoder.cs

@@ -62,11 +62,11 @@ public IEnumerable<TCoordinate> Decode(TPolyline polyline) {
         int longitude = 0;
 
         while (true) {
-            if(sequence.Length == position) {
+            if (sequence.Length == position) {
                 break; // End of the polyline string
             }
 
-            if(!PolylineEncoding.TryReadValue(ref latitude, ref sequence, ref position)
+            if (!PolylineEncoding.TryReadValue(ref latitude, ref sequence, ref position)
                 || !PolylineEncoding.TryReadValue(ref longitude, ref sequence, ref position)
             ) {
                 Options

src/PolylineAlgorithm.Abstraction/Internal/Logging.cs

@@ -16,7 +16,7 @@ internal static partial class Logging {
 
     [LoggerMessage(3, LogLevel.Error, "Internal buffer has {bufferLength} length. At position {position} is required additional {requiredSpace} length.")]
     internal static partial void LogInternalBufferOverflowError(this ILogger logger, int position, int bufferLength, int requiredSpace);
-    
+
     [LoggerMessage(4, LogLevel.Error, "Cannot write to internal buffer at position {position}. Current coordinate index is {coordinateIndex}.")]
     internal static partial void LogCannotWriteValueToBufferError(this ILogger logger, int position, int coordinateIndex);
 

src/PolylineAlgorithm/Coordinate.cs

@@ -7,7 +7,6 @@ namespace PolylineAlgorithm;
 
 using System;
 using System.Diagnostics;
-using System.Diagnostics.CodeAnalysis;
 using System.Globalization;
 using System.Runtime.InteropServices;
 

src/PolylineAlgorithm/PolylineDecoder.cs

@@ -6,7 +6,6 @@
 namespace PolylineAlgorithm;
 
 using PolylineAlgorithm.Abstraction;
-using System.Buffers;
 using System.Runtime.CompilerServices;
 
 /// <summary>

src/PolylineAlgorithm/PolylineEncoder.cs

@@ -6,7 +6,6 @@
 namespace PolylineAlgorithm;
 
 using PolylineAlgorithm.Abstraction;
-using System.Buffers;
 using System.Runtime.CompilerServices;
 
 /// <summary>

tests/PolylineAlgorithm.Abstraction.Tests/AbstractPolylineDecoderTest.cs

@@ -16,7 +16,7 @@ public class AbstractPolylineDecoderTest {
 
     public static IEnumerable<object[]> CoordinateCount => [[1], [10], [100], [1_000]];
 
-    public static IEnumerable<(double, double)> NotANumberAndInfinityCoordinates => StaticValueProvider.Invalid.GetNotANumberAndInfinityCoordinates() ;
+    public static IEnumerable<(double, double)> NotANumberAndInfinityCoordinates => StaticValueProvider.Invalid.GetNotANumberAndInfinityCoordinates();
 
     public static IEnumerable<(double, double)> MinAndMaxCoordinates => StaticValueProvider.Invalid.GetMinAndMaxCoordinates();
 

tests/PolylineAlgorithm.Abstraction.Tests/AbstractPolylineEncoderTest.cs

@@ -15,7 +15,7 @@ public class AbstractPolylineEncoderTest {
 
     public static IEnumerable<object[]> CoordinateCount => [[1], [10], [100], [1_000]];
 
-    public static IEnumerable<(double, double)> NotANumberAndInfinityCoordinates => StaticValueProvider.Invalid.GetNotANumberAndInfinityCoordinates() ;
+    public static IEnumerable<(double, double)> NotANumberAndInfinityCoordinates => StaticValueProvider.Invalid.GetNotANumberAndInfinityCoordinates();
 
     public static IEnumerable<(double, double)> MinAndMaxCoordinates => StaticValueProvider.Invalid.GetMinAndMaxCoordinates();
 
@@ -86,7 +86,7 @@ public void Encode_EmptyCoordinates_Throws_ArgumentException() {
     [TestMethod]
     public void Encode_BufferTooSmall_Throws_InternalBufferOverflowException() {
         // Arrange
-        PolylineEncoder _encoder = new PolylineEncoder(new PolylineEncodingOptions { BufferSize = 12});
+        PolylineEncoder _encoder = new PolylineEncoder(new PolylineEncodingOptions { BufferSize = 12 });
         IEnumerable<(double Latitude, double Longitude)> coordinates = RandomValueProvider.GetCoordinates(2);
 
         // Act
@@ -100,7 +100,7 @@ public void Encode_BufferTooSmall_Throws_InternalBufferOverflowException() {
     [DynamicData(nameof(NotANumberAndInfinityCoordinates))]
     public void Encode_NotANumberAndInfinityCoordinate_Throws_ArgumentOutOfRangeException((double, double) coordinate) {
         // Arrange
-        
+
         // Act
         var exception = Assert.ThrowsExactly<ArgumentOutOfRangeException>(() => _encoder.Encode([coordinate]));
 

tests/PolylineAlgorithm.Abstraction.Tests/FakeLoggerFactory.cs

@@ -29,7 +29,7 @@ public ILogger CreateLogger(string categoryName) {
     protected virtual void Dispose(bool disposing) {
         if (!_isDisposed) {
             if (disposing) {
-               
+
             }
 
             _isDisposed = true;

tests/PolylineAlgorithm.Abstraction.Tests/Internal/LoggingTest.cs

@@ -117,7 +117,7 @@ public void ILogger_LogPolylineCannotBeShorterThanError_Ok(string value) {
             _loggerProvider.Collector.LatestRecord.Message);
     }
 
-    
+
     [TestMethod]
     public void ILogger_LogRequestedBufferSizeExceedsMaxBufferLengthWarning_Ok() {
         // Arrange