From 8c74bb6917805c2bcec6c1f58f9c6b80b6b54bb8 Mon Sep 17 00:00:00 2001
From: Robin Krom <robin.krom@gmail.com>
Date: Sun, 28 Apr 2019 01:16:34 +0200
Subject: [PATCH] Fixing a build error and updating some more documentation to
 reduce warnings.

---
 NuGet.Config                                   |  1 -
 .../Configuration/CommandlineOptions.cs        | 18 ++++++++++++++++++
 .../Configuration/IConfigTranslations.cs       |  6 ++++++
 .../Configuration/IMetroConfiguration.cs       |  5 +++++
 .../Destinations/ClipboardDestination.cs       | 10 ++++++++++
 5 files changed, 39 insertions(+), 1 deletion(-)

diff --git a/NuGet.Config b/NuGet.Config
index cba75eeee..28aaeb53d 100644
--- a/NuGet.Config
+++ b/NuGet.Config
@@ -6,7 +6,6 @@
     <add key="Dapplo.Config" value="https://ci.appveyor.com/nuget/dapplo-config-85fagsv5oequ" />
     <add key="Dapplo.CaliburnMicro" value="https://ci.appveyor.com/nuget/dapplo-caliburnmicro-46kaa8k8ft4i" />
     <add key="Dapplo.HttpExtensions" value="https://ci.appveyor.com/nuget/dapplo-httpextensions-6qa60p3t5ixy" />
-    <add key="Dapplo.Windows" value="https://ci.appveyor.com/nuget/dapplo-windows-pjnkjqjp7t6a" />
     <add key="dotnet-core" value="https://dotnetfeed.blob.core.windows.net/dotnet-core/index.json" />
     <add key="dotnet-windowsdesktop" value="https://dotnetfeed.blob.core.windows.net/dotnet-windowsdesktop/index.json" />
     <add key="aspnet-aspnetcore" value="https://dotnetfeed.blob.core.windows.net/aspnet-aspnetcore/index.json" />
diff --git a/src/Greenshot/Configuration/CommandlineOptions.cs b/src/Greenshot/Configuration/CommandlineOptions.cs
index cd8c41ca2..20b491133 100644
--- a/src/Greenshot/Configuration/CommandlineOptions.cs
+++ b/src/Greenshot/Configuration/CommandlineOptions.cs
@@ -27,21 +27,39 @@ namespace Greenshot.Configuration
     /// </summary>
     public class CommandlineOptions
     {
+        /// <summary>
+        /// A commandline setting to define what files to open
+        /// </summary>
         [Option('f', "file", HelpText = "Files to open.")]
         public IEnumerable<string> ImageFiles { get; set; }
 
+        /// <summary>
+        /// A commandline setting to define that we need to exit
+        /// </summary>
         [Option(Default = false, HelpText = "Exit the running Greenshot instance.")]
         public bool Exit { get; set; }
 
+        /// <summary>
+        /// A commandline setting to define that a reload is needed
+        /// </summary>
         [Option(Default = false, HelpText = "Reload configuration.")]
         public bool Reload { get; set; }
 
+        /// <summary>
+        /// A commandline setting to define the language
+        /// </summary>
         [Option('l', "language", Default = "en-US", HelpText = "Language to use.")]
         public string Language { get; set; }
 
+        /// <summary>
+        /// A commandline setting to define the directory to use for the configuration files
+        /// </summary>
         [Option('i', "inidirectory", HelpText = "The directory to use to locate the ini file(s). This can be used to use Greenshot with different profiles.")]
         public string IniDirectory { get; set; }
 
+        /// <summary>
+        /// A commandline setting to define verbose
+        /// </summary>
         [Option(Default = false, HelpText = "Prints all messages to standard output.")]
         public bool Verbose { get; set; }
     }
diff --git a/src/Greenshot/Configuration/IConfigTranslations.cs b/src/Greenshot/Configuration/IConfigTranslations.cs
index 980a2fd6f..cab23e787 100644
--- a/src/Greenshot/Configuration/IConfigTranslations.cs
+++ b/src/Greenshot/Configuration/IConfigTranslations.cs
@@ -22,9 +22,15 @@ using Dapplo.Config.Language;
 
 namespace Greenshot.Configuration
 {
+    /// <summary>
+    /// This defines the translations for the config
+    /// </summary>
     [Language("Config")]
     public interface IConfigTranslations : Dapplo.CaliburnMicro.Translations.IConfigTranslations, ILanguage
     {
+        /// <summary>
+        /// Theme translation
+        /// </summary>
         [DefaultValue("Theme")]
         string Theme { get; }
     }
diff --git a/src/Greenshot/Configuration/IMetroConfiguration.cs b/src/Greenshot/Configuration/IMetroConfiguration.cs
index a109594ae..97497e89e 100644
--- a/src/Greenshot/Configuration/IMetroConfiguration.cs
+++ b/src/Greenshot/Configuration/IMetroConfiguration.cs
@@ -23,12 +23,17 @@ using Dapplo.Config.Ini;
 
 namespace Greenshot.Configuration
 {
+    /// <summary>
+    /// This specifies the configuration for MahApps.Metro
+    /// </summary>
     [IniSection("Metro")]
     public interface IMetroConfiguration : IIniSection, IMetroUiConfiguration
     {
+        /// <inheritdoc />
         [DefaultValue("Light")]
         new string Theme { get; set; }
 
+        /// <inheritdoc />
         [DefaultValue("Olive")]
         new string ThemeColor{ get; set; }
     }
diff --git a/src/Greenshot/Destinations/ClipboardDestination.cs b/src/Greenshot/Destinations/ClipboardDestination.cs
index d751ed13d..8abf958e6 100644
--- a/src/Greenshot/Destinations/ClipboardDestination.cs
+++ b/src/Greenshot/Destinations/ClipboardDestination.cs
@@ -41,6 +41,12 @@ namespace Greenshot.Destinations
 	{
 	    private readonly ExportNotification _exportNotification;
 
+        /// <summary>
+        /// DI Constructor
+        /// </summary>
+        /// <param name="coreConfiguration">ICoreConfiguration</param>
+        /// <param name="greenshotLanguage">IGreenshotLanguage</param>
+        /// <param name="exportNotification">ExportNotification</param>
 	    public ClipboardDestination(
 	        ICoreConfiguration coreConfiguration,
 	        IGreenshotLanguage greenshotLanguage,
@@ -50,12 +56,16 @@ namespace Greenshot.Destinations
 	        _exportNotification = exportNotification;
 	    }
 
+        /// <inheritdoc />
 	    public override string Description => GreenshotLanguage.SettingsDestinationClipboard;
 
+        /// <inheritdoc />
 	    public override Keys EditorShortcutKeys => Keys.Control | Keys.Shift | Keys.C;
 
+        /// <inheritdoc />
 	    public override IBitmapWithNativeSupport DisplayIcon => GreenshotResources.Instance.GetBitmap("Clipboard.Image");
 
+        /// <inheritdoc />
 	    protected override ExportInformation ExportCapture(bool manuallyInitiated, ISurface surface, ICaptureDetails captureDetails)
 	    {
 			var exportInformation = new ExportInformation(Designation, Description);