From 75386b6dacdd7a16492d4daa9598d6725b77f42f Mon Sep 17 00:00:00 2001
From: alex6dj <alexeiagueroalba@gmail.com>
Date: Thu, 5 Aug 2021 15:11:23 -0400
Subject: [PATCH] Document parameters

Former-commit-id: 6247bf6ea4a0ce4bc74e1f93d4192419c27f3e7a
---
 FFMpegCore/Extend/KeyValuePairExtensions.cs   |  9 ++++
 FFMpegCore/Extend/StringExtensions.cs         |  5 +++
 .../Arguments/SubtitleHardBurnArgument.cs     | 43 +++++++++++++++++++
 3 files changed, 57 insertions(+)

diff --git a/FFMpegCore/Extend/KeyValuePairExtensions.cs b/FFMpegCore/Extend/KeyValuePairExtensions.cs
index 92dbf6d..28cc087 100644
--- a/FFMpegCore/Extend/KeyValuePairExtensions.cs
+++ b/FFMpegCore/Extend/KeyValuePairExtensions.cs
@@ -4,6 +4,15 @@ namespace FFMpegCore.Extend
 {
     internal static class KeyValuePairExtensions
     {
+        /// <summary>
+        ///     Concat the two members of a <see cref="KeyValuePair{TKey,TValue}" />
+        /// </summary>
+        /// <param name="pair">Input object</param>
+        /// <param name="enclose">
+        ///     If true encloses the value part between quotes if contains an space character. If false use the
+        ///     value unmodified
+        /// </param>
+        /// <returns>The formatted string</returns>
         public static string FormatArgumentPair(this KeyValuePair<string, string> pair, bool enclose)
         {
             var key = pair.Key;
diff --git a/FFMpegCore/Extend/StringExtensions.cs b/FFMpegCore/Extend/StringExtensions.cs
index f4e0169..ddcf54b 100644
--- a/FFMpegCore/Extend/StringExtensions.cs
+++ b/FFMpegCore/Extend/StringExtensions.cs
@@ -2,6 +2,11 @@
 {
     internal static class StringExtensions
     {
+        /// <summary>
+        ///     Enclose string between quotes if contains an space character
+        /// </summary>
+        /// <param name="input">The input</param>
+        /// <returns>The enclosed string</returns>
         public static string EncloseIfContainsSpace(this string input)
         {
             return input.Contains(" ") ? $"'{input}'" : input;
diff --git a/FFMpegCore/FFMpeg/Arguments/SubtitleHardBurnArgument.cs b/FFMpegCore/FFMpeg/Arguments/SubtitleHardBurnArgument.cs
index 552a87b..1186ae2 100644
--- a/FFMpegCore/FFMpeg/Arguments/SubtitleHardBurnArgument.cs
+++ b/FFMpegCore/FFMpeg/Arguments/SubtitleHardBurnArgument.cs
@@ -25,6 +25,13 @@ public class SubtitleHardBurnOptions
 
         public readonly Dictionary<string, string> Parameters = new Dictionary<string, string>();
 
+        /// <summary>
+        ///     Create a new <see cref="SubtitleHardBurnOptions"/> using a provided subtitle file or a video file
+        ///     containing one.
+        /// </summary> 
+        /// <param name="subtitlePath"></param>
+        /// <returns></returns>
+        /// <remarks>Only support .srt and .ass files, and subrip and ssa subtitle streams</remarks>
         public static SubtitleHardBurnOptions Create(string subtitlePath)
         {
             return new SubtitleHardBurnOptions(subtitlePath);
@@ -35,26 +42,56 @@ private SubtitleHardBurnOptions(string subtitle)
             _subtitle = subtitle;
         }
 
+        /// <summary>
+        ///     Specify the size of the original video, the video for which the ASS file was composed.
+        /// </summary>
+        /// <param name="width"></param>
+        /// <param name="height"></param>
+        /// <returns></returns>
         public SubtitleHardBurnOptions SetOriginalSize(int width, int height)
         {
             return WithParameter("original_size", $"{width}x{height}");
         }
 
+        /// <summary>
+        ///     Specify the size of the original video, the video for which the ASS file was composed.
+        /// </summary>
+        /// <param name="size"></param>
+        /// <returns></returns>
         public SubtitleHardBurnOptions SetOriginalSize(Size size)
         {
             return SetOriginalSize(size.Width, size.Height);
         }
 
+        /// <summary>
+        ///     Set subtitles stream index.
+        /// </summary>
+        /// <param name="index"></param>
+        /// <returns></returns>
+        /// <remarks>
+        ///     Used when the provided subtitle is an stream of a video file (ex. .mkv) with multiple subtitles.
+        ///     Represent the index of the subtitle not the stream, them the first subtitle index is 0 and second is 1
+        /// </remarks>
         public SubtitleHardBurnOptions SetSubtitleIndex(int index)
         {
             return WithParameter("si", index.ToString());
         }
 
+        /// <summary>
+        ///     Set subtitles input character encoding. Only useful if not UTF-8
+        /// </summary>
+        /// <param name="encode">Charset encoding</param>
+        /// <returns></returns>
         public SubtitleHardBurnOptions SetCharacterEncoding(string encode)
         {
             return WithParameter("charenc", encode);
         }
 
+        /// <summary>
+        ///     Override default style or script info parameters of the subtitles
+        /// </summary>
+        /// <param name="styleOptions"></param>
+        /// <returns></returns>
         public SubtitleHardBurnOptions WithStyle(StyleOptions styleOptions)
         {
             return WithParameter("force_style", styleOptions.TextInternal);
@@ -78,6 +115,12 @@ public static StyleOptions Create()
             return new StyleOptions();
         }
 
+        /// <summary>
+        ///     Used to override default style or script info parameters of the subtitles. It accepts ASS style format
+        /// </summary>
+        /// <param name="key"></param>
+        /// <param name="value"></param>
+        /// <returns></returns>
         public StyleOptions WithParameter(string key, string value)
         {
             Parameters.Add(key, value);