ahaha

Author: petesramek

src/PolylineAlgorithm/Abstraction/AbstractPolylineDecoder.cs

@@ -4,7 +4,6 @@
 //
 
 using Microsoft.Extensions.Logging;
-using PolylineAlgorithm.Diagnostics;
 using PolylineAlgorithm.Internal.Diagnostics;
 using System.Runtime.CompilerServices;
 

src/PolylineAlgorithm/Abstraction/AbstractPolylineEncoder.cs

@@ -7,7 +7,6 @@ namespace PolylineAlgorithm.Abstraction;
 
 using Microsoft.Extensions.Logging;
 using PolylineAlgorithm;
-using PolylineAlgorithm.Diagnostics;
 using PolylineAlgorithm.Internal;
 using PolylineAlgorithm.Internal.Diagnostics;
 using System;

src/PolylineAlgorithm/Extensions/PolylineEncoderExtensions.cs

@@ -1,7 +1,7 @@
 namespace PolylineAlgorithm.Extensions;
 
 using PolylineAlgorithm.Abstraction;
-using PolylineAlgorithm.Diagnostics;
+using PolylineAlgorithm.Internal.Diagnostics;
 using System;
 using System.Collections.Generic;
 using System.Diagnostics.CodeAnalysis;
@@ -14,7 +14,7 @@ namespace PolylineAlgorithm.Extensions;
 /// </summary>
 public static class PolylineEncoderExtensions {
     /// <summary>
-    /// Encodes a collection of <see cref="Coordinate"/> instances into an encoded polyline.
+    /// Encodes a collection of <see cref="TValue"/> instances into an encoded polyline.
     /// </summary>
     /// <param name="encoder">
     /// The <see cref="IPolylineEncoder{TValue, TPolyline}"/> instance used to perform the encoding operation.

…eAlgorithm/Diagnostics/ExceptionGuard.cs …m/Internal/Diagnostics/ExceptionGuard.cssrc/PolylineAlgorithm/Diagnostics/ExceptionGuard.cs renamed to src/PolylineAlgorithm/Internal/Diagnostics/ExceptionGuard.cs src/PolylineAlgorithm/Diagnostics/ExceptionGuard.cs renamed to src/PolylineAlgorithm/Internal/Diagnostics/ExceptionGuard.cs

@@ -1,4 +1,4 @@
-namespace PolylineAlgorithm.Diagnostics;
+namespace PolylineAlgorithm.Internal.Diagnostics;
 
 using PolylineAlgorithm;
 using PolylineAlgorithm.Properties;
@@ -218,7 +218,7 @@ public static void ThrowInvalidPolylineBlockTerminator() {
     /// resource strings. This class is public because messages are only used by the guard methods.
     /// </remarks>
     [SuppressMessage("Style", "IDE1006:Naming Styles", Justification = "Internal use only.")]
-    public static class ExceptionMessage {
+    internal static class ExceptionMessage {
 #if NET8_0_OR_GREATER
         private static readonly CompositeFormat StackAllocLimitMustBeEqualOrGreaterThanFormat = CompositeFormat.Parse(ExceptionMessageResource.StackAllocLimitMustBeEqualOrGreaterThanFormat);
         private static readonly CompositeFormat PolylineCannotBeShorterThanFormat = CompositeFormat.Parse(ExceptionMessageResource.PolylineCannotBeShorterThanFormat);

src/PolylineAlgorithm/Internal/Diagnostics/LogDebugExtensions.cs

@@ -9,7 +9,7 @@ namespace PolylineAlgorithm.Internal.Diagnostics;
 
 internal static partial class LogDebugExtensions {
     private const LogLevel LOG_LEVEL = LogLevel.Debug;
-    private const int EVENT_ID_BASE = (int)LOG_LEVEL * 100;
+    private const int EVENT_ID_BASE = (int)LOG_LEVEL * Defaults.Logging.LogLevelMultiplier;
 
     [LoggerMessage(EVENT_ID_BASE + 1, LOG_LEVEL, "Operation {operationName} has started.")]
     internal static partial void LogOperationStartedDebug(this ILogger logger, string operationName);

src/PolylineAlgorithm/Internal/Diagnostics/LogWarningExtensions.cs

@@ -27,7 +27,7 @@ internal static partial class LogWarningExtensions {
     /// Base value for event ids used by the <see cref="LoggerMessageAttribute"/> annotations.
     /// Computed as <c>(int)LOG_LEVEL * 100</c>.
     /// </summary>
-    private const int EVENT_ID_BASE = (int)LOG_LEVEL * 100;
+    private const int EVENT_ID_BASE = (int)LOG_LEVEL * Defaults.Logging.LogLevelMultiplier;
 
     /// <summary>
     /// Logs a warning when a method argument is <c>null</c>.

src/PolylineAlgorithm/Internal/Pow10.cs

@@ -1,4 +1,4 @@
-namespace PolylineAlgorithm.Internal;
+namespace PolylineAlgorithm.Internal;
 
 using System;
 
@@ -13,7 +13,7 @@ internal static class Pow10 {
     /// <summary>
     /// Pre-computed powers of 10 from 10^0 to 10^9.
     /// </summary>
-    private static readonly uint[] _pow10 = { 1, 10, 100, 1000, 10000, 100000, 1000000, 10000000, 100000000, 1000000000 };
+    private static readonly uint[] _pow10 = [1, 10, 100, 1000, 10000, 100000, 1000000, 10000000, 100000000, 1000000000];
 
     /// <summary>
     /// Returns the power of 10 for the specified precision level.

src/PolylineAlgorithm/PolylineEncoding.cs

@@ -5,9 +5,8 @@
 
 namespace PolylineAlgorithm;
 
-using PolylineAlgorithm.Diagnostics;
 using PolylineAlgorithm.Internal;
-
+using PolylineAlgorithm.Internal.Diagnostics;
 using System;
 using System.Numerics;
 using System.Runtime.InteropServices;
@@ -79,7 +78,7 @@ public static int Normalize(double value, uint precision = 5) {
     /// This is the inverse of <see cref="Normalize"/>. The method divides <paramref name="value"/> by 10^<paramref name="precision"/>
     /// and returns a <see cref="double"/>. If <paramref name="precision"/> is zero the integer value is returned as a double unchanged.
     /// </para>
-    /// <para>Arithmetic is performed in a <c>checked</c> context so that any overflow is surfaced as an <see cref="OverflowException"/>.</para>
+    /// <para>Arithmetic is performed in a <see langword="checked"/> context so that any overflow is surfaced as an <see cref="OverflowException"/>.</para>
     /// </remarks>
     /// <param name="value">Integer normalized coordinate (typically produced by <see cref="Normalize"/>).</param>
     /// <param name="precision">Number of decimal places that were used when normalizing. Default is 5.</param>
@@ -114,14 +113,14 @@ public static double Denormalize(int value, uint precision = 5) {
     /// </para>
     /// <para>
     /// The <paramref name="position"/> argument is advanced to the character after the last character consumed for the value.
-    /// If the buffer ends before a complete encoded value is available the method returns <c>false</c> and <paramref name="position"/>
+    /// If the buffer ends before a complete encoded value is available the method returns <see langword="false"/> and <paramref name="position"/>
     /// will point to the buffer end.
     /// </para>
     /// </remarks>
     /// <param name="delta">Reference to the accumulator; the decoded value will be added to this parameter.</param>
     /// <param name="buffer">Read-only memory containing polyline-encoded characters.</param>
     /// <param name="position">Reference to the current index in <paramref name="buffer"/>; advanced as characters are read.</param>
-    /// <returns><c>true</c> if a full value was decoded; <c>false</c> if the buffer ended before the value was complete.</returns>
+    /// <returns><see langword="true"/> if a full value was decoded; <see langword="false"/> if the buffer ended before the value was complete.</returns>
     public static bool TryReadValue(ref int delta, ReadOnlyMemory<char> buffer, ref int position) {
         // Validate that the position is within the bounds of the buffer.
         if (position >= buffer.Length) {
@@ -164,14 +163,14 @@ public static bool TryReadValue(ref int delta, ReadOnlyMemory<char> buffer, ref
     /// </para>
     /// <para>
     /// Before writing the method checks that there is sufficient space in the destination span by calling
-    /// <see cref="GetRequiredBufferSize(int)"/>. If there is not enough space the method returns <c>false</c> without modifying
+    /// <see cref="GetRequiredBufferSize(int)"/>. If there is not enough space the method returns <see langword="false"/> without modifying
     /// the buffer or position.
     /// </para>
     /// </remarks>
     /// <param name="delta">The integer value to encode (typically a coordinate delta).</param>
     /// <param name="buffer">Destination character span to write the encoded characters into.</param>
     /// <param name="position">Reference to the current write index in <paramref name="buffer"/>; advanced as characters are written.</param>
-    /// <returns><c>true</c> if the value was encoded and written; <c>false</c> if the buffer did not have sufficient space.</returns>
+    /// <returns><see langword="true"/> if the value was encoded and written; <see langword="false"/> if the buffer did not have sufficient space.</returns>
     public static bool TryWriteValue(int delta, Span<char> buffer, ref int position) {
         // Validate that the position and required space for write is within the bounds of the buffer.
         if (buffer[position..].Length < GetRequiredBufferSize(delta)) {

src/PolylineAlgorithm/PolylineEncodingOptionsBuilder.cs

@@ -7,7 +7,7 @@ namespace PolylineAlgorithm;
 
 using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Logging.Abstractions;
-using PolylineAlgorithm.Diagnostics;
+using PolylineAlgorithm.Internal.Diagnostics;
 
 /// <summary>
 /// Provides a builder for configuring options for polyline encoding operations.

tests/PolylineAlgorithm.Tests/Internal/Diagnostics/ExceptionGuardTests.cs

@@ -5,7 +5,7 @@
 
 namespace PolylineAlgorithm.Tests.Internal.Diagnostics;
 
-using PolylineAlgorithm.Diagnostics;
+using PolylineAlgorithm.Internal.Diagnostics;
 using System;
 
 /// <summary>