Remove enumerator usage in AcquireNextTriggers when getting timeTrigg…

…ers min value (#2092)

changelog.md

@@ -8,10 +8,12 @@
 
   * Mark UseJsonSerializer as obsolete, one should use UseNewtonsoftJsonSerializer (#2077)
   * Removed obsolete UseMicrosoftDependencyInjectionScopedJobFactory(), mark UseMicrosoftDependencyInjectionJobFactory() obsolete (#2085)
+  * Change QuartzHostedService from internal to public (#2090)
 
 * FIXES
 
   * Now omitting UseMicrosoftDependencyInjectionJobFactory() should actually work as it will be the default (#2085)
+  * Remove enumerator usage in AcquireNextTriggers when getting timeTriggers min value (#2092)
 
 
 ## Release 3.6.3, Jun 25 2023

src/Quartz/Simpl/RAMJobStore.cs

@@ -1634,7 +1634,7 @@ public virtual Task<IReadOnlyCollection<IOperableTrigger>> AcquireNextTriggers(
 
                 while (true)
                 {
-                    var tw = timeTriggers.FirstOrDefault();
+                    var tw = timeTriggers.Min;
                     if (tw == null)
                     {
                         break;

src/Quartz/Util/SortedSetExtensions.cs

@@ -1,5 +1,4 @@
-using System;
-using System.Collections.Generic;
+using System.Collections.Generic;
 
 namespace Quartz.Util
 {
@@ -9,30 +8,5 @@ internal static SortedSet<int> TailSet(this SortedSet<int> set, int value)
         {
             return set.GetViewBetween(value, 9999999);
         }
-
-        /// <summary>
-        /// Returns the first element of the specified <see cref="SortedSet{TSource}"/>, or a default
-        /// value if no element is found.
-        /// </summary>
-        /// <typeparam name="TSource">The type of the elements of <paramref name="source"/>.</typeparam>
-        /// <param name="source">The <see cref="SortedSet{TSource}"/> to return the first element of.</param>
-        /// <returns>
-        /// The default value for <typeparamref name="TSource"/> if <paramref name="source"/> is empty;
-        /// otherwise, the first element in <paramref name="source"/>.
-        /// </returns>
-        /// <exception cref="ArgumentNullException"><paramref name="source"/> is <see langword="null"/>.</exception>
-        internal static TSource? FirstOrDefault<TSource>(this SortedSet<TSource> source) where TSource : class
-        {
-            if (source == null)
-                throw new ArgumentNullException(nameof(source));
-
-            using var enumerator = source.GetEnumerator();
-            if (!enumerator.MoveNext())
-            {
-                return null;
-            }
-
-            return enumerator.Current;
-        }
     }
 }