Minor cleanup

Author: jodydonetti

src/ZiggyCreatures.FusionCache/FusionCache_Async.cs

@@ -292,7 +292,6 @@ public partial class FusionCache
 		if (_logger?.IsEnabled(LogLevel.Debug) ?? false)
 			_logger.LogDebug("FUSION (O={CacheOperationId} K={CacheKey}): calling GetOrSetAsync<T> {Options}", operationId, key, options.ToLogString());
 
-		// TODO: MAYBE WE SHOULD AVOID ALLOCATING A LAMBDA HERE, BY CHANGING THE INTERNAL LOGIC OF THE GetOrSetEntryInternalAsync METHOD
 		var entry = await GetOrSetEntryInternalAsync<TValue>(operationId, key, (_, _) => Task.FromResult(defaultValue), false, default, options, token).ConfigureAwait(false);
 
 		if (entry is null)

src/ZiggyCreatures.FusionCache/FusionCache_Sync.cs

@@ -292,7 +292,6 @@ public partial class FusionCache
 		if (_logger?.IsEnabled(LogLevel.Debug) ?? false)
 			_logger.LogDebug("FUSION (O={CacheOperationId} K={CacheKey}): calling GetOrSet<T> {Options}", operationId, key, options.ToLogString());
 
-		// TODO: MAYBE WE SHOULD AVOID ALLOCATING A LAMBDA HERE, BY CHANGING THE INTERNAL LOGIC OF THE GetOrSetEntryInternal METHOD
 		var entry = GetOrSetEntryInternal<TValue>(operationId, key, (_, _) => defaultValue, false, default, options, token);
 
 		if (entry is null)

src/ZiggyCreatures.FusionCache/Internals/FusionCacheInternalUtils.cs

@@ -99,9 +99,9 @@ public static bool IsLogicallyExpired(this IFusionCacheEntry? entry)
 
 	public static Exception GetSingleInnerExceptionOrSelf(this AggregateException exc)
 	{
-		// TODO: NOT SURE ABOUT THIS
-		//if (exc is null)
-		//	return null;
+		if (exc is null)
+			throw new ArgumentNullException(nameof(exc));
+
 		return (exc.InnerException is not null && exc.InnerExceptions?.Count <= 1)
 			? exc.InnerException
 			: exc