refactor: Fix linter warnings

jcoliz · Jul 19, 2024 · 296fa36 · 296fa36
1 parent 7e823d4
commit 296fa36
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/LogoSlideMaker.Lib/Export/Renderer.cs b/LogoSlideMaker.Lib/Export/Renderer.cs
@@ -49,10 +49,10 @@ public void Render(Presentation pres, SlideLayout layout, string? dataVersion, I
         }
     }
 
-    private void SetDescription(SlideLayout layout, ISlideShapes target)
+    private static void SetDescription(SlideLayout layout, ISlideShapes target)
     {
         // Fill in description field
-        var num_description_lines = layout.Variant.Description.Count();
+        var num_description_lines = layout.Variant.Description.Count;
         if (num_description_lines > 0)
         {
             var description_box = target.TryGetByName<IShape>("Description");

diff --git a/LogoSlideMaker.WinUi/MainWindow.xaml.cs b/LogoSlideMaker.WinUi/MainWindow.xaml.cs
@@ -1,8 +1,6 @@
-using DocumentFormat.OpenXml.Bibliography;
 using LogoSlideMaker.Primitives;
 using LogoSlideMaker.WinUi.Services;
 using LogoSlideMaker.WinUi.ViewModels;
-using Microsoft.AspNetCore.Routing.Template;
 using Microsoft.Extensions.Logging;
 using Microsoft.Graphics.Canvas;
 using Microsoft.Graphics.Canvas.Brushes;
@@ -14,6 +12,7 @@
 using System;
 using System.ComponentModel;
 using System.Diagnostics;
+using System.Diagnostics.CodeAnalysis;
 using System.IO;
 using System.Linq;
 using System.Runtime.CompilerServices;
@@ -287,12 +286,13 @@ private async void ExportSlides(object _, RoutedEventArgs __)
 
     private async void ShowAboutDialog(object sender, RoutedEventArgs e)
     {
-        var result = await aboutDialog.ShowAsync();
+        await aboutDialog.ShowAsync();
 
         logOk();
     }
 
-    private void OpenLogsFolder(ContentDialog sender, ContentDialogButtonClickEventArgs args)
+    [SuppressMessage("Performance", "CA1822:Mark members as static", Justification = "UI system requires calling with instance")]
+    private void OpenLogsFolder(ContentDialog _, ContentDialogButtonClickEventArgs __)
     {
         Process.Start("explorer.exe", MainViewModel.LogsFolder);
     }