Refactor parameter names and simplify iteration

Changed parameter name `innerResult` to `methodInnerResult` in `OnInnerResultCompleted` method across multiple files in the `Caliburn.Micro` namespace for consistency:
- `ContinueResultDecorator.cs`
- `OverrideCancelResultDecorator.cs`
- `RescueResultDecorator.cs`
- `ResultDecoratorBase.cs`

Removed unnecessary variable `keys` in `HttpUtility.cs` within the `Caliburn.Micro.Maui` namespace, directly iterating over `this.Keys` in the `foreach` loop to simplify the code.

Closes Caliburn-Micro#940
Closes Caliburn-Micro#939

Author: vb2ae

src/Caliburn.Micro.Core/ContinueResultDecorator.cs

@@ -30,17 +30,17 @@ public ContinueResultDecorator(IResult result, Func<IResult> coroutine)
         /// Called when the execution of the decorated result has completed.
         /// </summary>
         /// <param name="context">The context.</param>
-        /// <param name="innerResult">The decorated result.</param>
+        /// <param name="methodInnerResult">The decorated result.</param>
         /// <param name="args">The <see cref="ResultCompletionEventArgs" /> instance containing the event data.</param>
-        protected override void OnInnerResultCompleted(CoroutineExecutionContext context, IResult innerResult, ResultCompletionEventArgs args)
+        protected override void OnInnerResultCompleted(CoroutineExecutionContext context, IResult methodInnerResult, ResultCompletionEventArgs args)
         {
             if (args.Error != null || !args.WasCancelled)
             {
                 OnCompleted(new ResultCompletionEventArgs { Error = args.Error });
             }
             else
             {
-                Log.Info(string.Format("Executing coroutine because {0} was cancelled.", innerResult.GetType().Name));
+                Log.Info(string.Format("Executing coroutine because {0} was cancelled.", methodInnerResult.GetType().Name));
                 Continue(context);
             }
         }

src/Caliburn.Micro.Core/OverrideCancelResultDecorator.cs

@@ -18,13 +18,13 @@ public OverrideCancelResultDecorator(IResult result)
         /// Called when the execution of the decorated result has completed.
         /// </summary>
         /// <param name="context">The context.</param>
-        /// <param name="innerResult">The decorated result.</param>
+        /// <param name="methodInnerResult">The decorated result.</param>
         /// <param name="args">The <see cref="ResultCompletionEventArgs" /> instance containing the event data.</param>
-        protected override void OnInnerResultCompleted(CoroutineExecutionContext context, IResult innerResult, ResultCompletionEventArgs args)
+        protected override void OnInnerResultCompleted(CoroutineExecutionContext context, IResult methodInnerResult, ResultCompletionEventArgs args)
         {
             if (args.WasCancelled)
             {
-                Log.Info(string.Format("Overriding WasCancelled from {0}.", innerResult.GetType().Name));
+                Log.Info(string.Format("Overriding WasCancelled from {0}.", methodInnerResult.GetType().Name));
             }
 
             OnCompleted(new ResultCompletionEventArgs { Error = args.Error });

src/Caliburn.Micro.Core/RescueResultDecorator.cs

@@ -28,9 +28,9 @@ public RescueResultDecorator(IResult result, Func<TException, IResult> coroutine
         /// Called when the execution of the decorated result has completed.
         /// </summary>
         /// <param name="context">The context.</param>
-        /// <param name="innerResult">The decorated result.</param>
+        /// <param name="methodInnerResult">The decorated result.</param>
         /// <param name="args">The <see cref="ResultCompletionEventArgs" /> instance containing the event data.</param>
-        protected override void OnInnerResultCompleted(CoroutineExecutionContext context, IResult innerResult, ResultCompletionEventArgs args)
+        protected override void OnInnerResultCompleted(CoroutineExecutionContext context, IResult methodInnerResult, ResultCompletionEventArgs args)
         {
             var error = args.Error as TException;
             if (error == null)
@@ -40,7 +40,7 @@ protected override void OnInnerResultCompleted(CoroutineExecutionContext context
             else
             {
                 Log.Error(error);
-                Log.Info(string.Format("Executing coroutine because {0} threw an exception.", innerResult.GetType().Name));
+                Log.Info(string.Format("Executing coroutine because {0} threw an exception.", methodInnerResult.GetType().Name));
                 Rescue(context, error);
             }
         }

src/Caliburn.Micro.Core/ResultDecoratorBase.cs

@@ -50,9 +50,9 @@ private void InnerResultCompleted(object sender, ResultCompletionEventArgs args)
         /// Called when the execution of the decorated result has completed.
         /// </summary>
         /// <param name="context">The context.</param>
-        /// <param name="innerResult">The decorated result.</param>
+        /// <param name="methodInnerResult">The decorated result.</param>
         /// <param name="args">The <see cref="ResultCompletionEventArgs"/> instance containing the event data.</param>
-        protected abstract void OnInnerResultCompleted(CoroutineExecutionContext context, IResult innerResult, ResultCompletionEventArgs args);
+        protected abstract void OnInnerResultCompleted(CoroutineExecutionContext context, IResult methodInnerResult, ResultCompletionEventArgs args);
 
         /// <summary>
         /// Occurs when execution has completed.

src/Caliburn.Micro.Platform/Platforms/Maui/HttpUtility.cs

@@ -46,7 +46,6 @@ public override string ToString()
                 if (count == 0)
                     return "";
                 StringBuilder sb = new StringBuilder();
-                var keys = this.Keys;
                 foreach (var key in this.Keys)
                 {
                     sb.AppendFormat("{0}={1}&", key, this[key]);