From e918686911df9fd8600dee6ee142dee4d86d4ab4 Mon Sep 17 00:00:00 2001
From: Mike Marynowski <mikem@singulink.com>
Date: Thu, 7 Mar 2024 22:19:02 -0500
Subject: [PATCH] Add missing mutable ctors + optimizations

---
 .../MoneySet.cs                               | 22 +++++++++------
 .../SortedMoneySet.cs                         | 28 ++++++++++---------
 2 files changed, 29 insertions(+), 21 deletions(-)

diff --git a/Source/Singulink.Globalization.Currency/MoneySet.cs b/Source/Singulink.Globalization.Currency/MoneySet.cs
index 22dd5e4..d2fec59 100644
--- a/Source/Singulink.Globalization.Currency/MoneySet.cs
+++ b/Source/Singulink.Globalization.Currency/MoneySet.cs
@@ -36,6 +36,19 @@ public MoneySet(CurrencyRegistry registry)
         _registry = registry;
     }
 
+    /// <inheritdoc cref="MoneySet(ReadOnlySpan{Money})"/>
+    public MoneySet(IEnumerable<Money> values) : this(CurrencyRegistry.Default, values) { }
+
+    /// <inheritdoc cref="MoneySet(CurrencyRegistry, ReadOnlySpan{Money})"/>
+    public MoneySet(CurrencyRegistry registry, IEnumerable<Money> values)
+        : this(registry, values, values is not IReadOnlyMoneySet s || s.Registry != registry) { }
+
+    /// <inheritdoc cref="MoneySet(ReadOnlySpan{Money})"/>
+    public MoneySet(params Money[] values) : this(CurrencyRegistry.Default, values) { }
+
+    /// <inheritdoc cref="MoneySet(CurrencyRegistry, ReadOnlySpan{Money})"/>
+    public MoneySet(CurrencyRegistry registry, params Money[] values) : this(registry, values.AsSpan()) { }
+
     /// <summary>
     /// Initializes a new instance of the <see cref="MoneySet"/> class with the <see cref="CurrencyRegistry.Default"/> currency registry and adds all
     /// the specified values.
@@ -43,7 +56,7 @@ public MoneySet(CurrencyRegistry registry)
     /// <exception cref="ArgumentException">
     /// Attempted to add a value with a currency that is not available in the currency registry.
     /// </exception>
-    public MoneySet(IEnumerable<Money> values) : this(CurrencyRegistry.Default, values) { }
+    public MoneySet(ReadOnlySpan<Money> values) : this(CurrencyRegistry.Default, values) { }
 
     /// <summary>
     /// Initializes a new instance of the <see cref="MoneySet"/> class with the specified currency registry and adds all the specified values.
@@ -51,13 +64,6 @@ public MoneySet(IEnumerable<Money> values) : this(CurrencyRegistry.Default, valu
     /// <exception cref="ArgumentException">
     /// Attempted to add a value with a currency that is not available in the currency registry.
     /// </exception>
-    public MoneySet(CurrencyRegistry registry, IEnumerable<Money> values)
-        : this(registry, values, values is not IReadOnlyMoneySet s || s.Registry != registry) { }
-
-    /// <inheritdoc cref="MoneySet(IEnumerable{Money})"/>
-    public MoneySet(ReadOnlySpan<Money> values) : this(CurrencyRegistry.Default, values) { }
-
-    /// <inheritdoc cref="MoneySet(CurrencyRegistry, IEnumerable{Money})"/>
     public MoneySet(CurrencyRegistry registry, ReadOnlySpan<Money> values)
     {
         _registry = registry;
diff --git a/Source/Singulink.Globalization.Currency/SortedMoneySet.cs b/Source/Singulink.Globalization.Currency/SortedMoneySet.cs
index 3cc33f5..8bbac45 100644
--- a/Source/Singulink.Globalization.Currency/SortedMoneySet.cs
+++ b/Source/Singulink.Globalization.Currency/SortedMoneySet.cs
@@ -23,9 +23,7 @@ public sealed partial class SortedMoneySet : IMoneySet
     /// <summary>
     /// Initializes a new instance of the <see cref="SortedMoneySet"/> class with the <see cref="CurrencyRegistry.Default"/> currency registry.
     /// </summary>
-    public SortedMoneySet() : this(CurrencyRegistry.Default)
-    {
-    }
+    public SortedMoneySet() : this(CurrencyRegistry.Default) { }
 
     /// <summary>
     /// Initializes a new instance of the <see cref="SortedMoneySet"/> class with the specified currency registry.
@@ -38,6 +36,19 @@ public SortedMoneySet(CurrencyRegistry registry)
         _registry = registry;
     }
 
+    /// <inheritdoc cref="MoneySet(ReadOnlySpan{Money})"/>
+    public SortedMoneySet(IEnumerable<Money> values) : this(CurrencyRegistry.Default, values) { }
+
+    /// <inheritdoc cref="MoneySet(CurrencyRegistry, ReadOnlySpan{Money})"/>
+    public SortedMoneySet(CurrencyRegistry registry, IEnumerable<Money> values)
+        : this(registry, values, values is not IReadOnlyMoneySet s || s.Registry != registry) { }
+
+    /// <inheritdoc cref="MoneySet(ReadOnlySpan{Money})"/>
+    public SortedMoneySet(params Money[] values) : this(CurrencyRegistry.Default, values) { }
+
+    /// <inheritdoc cref="MoneySet(CurrencyRegistry, ReadOnlySpan{Money})"/>
+    public SortedMoneySet(CurrencyRegistry registry, params Money[] values) : this(registry, values.AsSpan()) { }
+
     /// <summary>
     /// Initializes a new instance of the <see cref="SortedMoneySet"/> class with the <see cref="CurrencyRegistry.Default"/> currency registry and adds all
     /// the specified values.
@@ -45,9 +56,7 @@ public SortedMoneySet(CurrencyRegistry registry)
     /// <exception cref="ArgumentException">
     /// Attempted to add a value with a currency that is not available in the currency registry.
     /// </exception>
-    public SortedMoneySet(IEnumerable<Money> values) : this(CurrencyRegistry.Default, values)
-    {
-    }
+    public SortedMoneySet(ReadOnlySpan<Money> values) : this(CurrencyRegistry.Default, values) { }
 
     /// <summary>
     /// Initializes a new instance of the <see cref="SortedMoneySet"/> class with the specified currency registry and adds all the specified values.
@@ -55,13 +64,6 @@ public SortedMoneySet(IEnumerable<Money> values) : this(CurrencyRegistry.Default
     /// <exception cref="ArgumentException">
     /// Attempted to add a value with a currency that is not available in the currency registry.
     /// </exception>
-    public SortedMoneySet(CurrencyRegistry registry, IEnumerable<Money> values)
-        : this(registry, values, values is not IReadOnlyMoneySet s || s.Registry != registry) { }
-
-    /// <inheritdoc cref="SortedMoneySet(IEnumerable{Money})"/>
-    public SortedMoneySet(ReadOnlySpan<Money> values) : this(CurrencyRegistry.Default, values) { }
-
-    /// <inheritdoc cref="SortedMoneySet(CurrencyRegistry, IEnumerable{Money})"/>
     public SortedMoneySet(CurrencyRegistry registry, ReadOnlySpan<Money> values)
     {
         _registry = registry;