minor refactoring in new symbol code

1 files changed, 23 insertions, 5 deletions

diff --git a/src/SMAPI/Framework/ModLoading/Symbols/SymbolWriterProvider.cs b/src/SMAPI/Framework/ModLoading/Symbols/SymbolWriterProvider.cs
index 33e91db0..8f7e05d1 100644
--- a/src/SMAPI/Framework/ModLoading/Symbols/SymbolWriterProvider.cs
+++ b/src/SMAPI/Framework/ModLoading/Symbols/SymbolWriterProvider.cs
@@ -4,19 +4,37 @@ using Mono.Cecil.Cil;
 
 namespace StardewModdingAPI.Framework.ModLoading.Symbols
 {
+    /// <summary>Provides assembly symbol writers for Mono.Cecil.</summary>
     internal class SymbolWriterProvider : ISymbolWriterProvider
     {
-        private readonly ISymbolWriterProvider BaseProvider = new DefaultSymbolWriterProvider();
+        /*********
+        ** Fields
+        *********/
+        /// <summary>The default symbol writer provider.</summary>
+        private readonly ISymbolWriterProvider DefaultProvider = new DefaultSymbolWriterProvider();
 
-        public ISymbolWriter GetSymbolWriter( ModuleDefinition module, string fileName )
+        /// <summary>The symbol writer provider for the portable PDB format.</summary>
+        private readonly ISymbolWriterProvider PortablePdbProvider = new PortablePdbWriterProvider();
+
+
+        /*********
+        ** Public methods
+        *********/
+        /// <summary>Get a symbol writer for a given module and assembly path.</summary>
+        /// <param name="module">The loaded assembly module.</param>
+        /// <param name="fileName">The assembly name.</param>
+        public ISymbolWriter GetSymbolWriter(ModuleDefinition module, string fileName)
         {
-            return this.BaseProvider.GetSymbolWriter( module, fileName );
+            return this.DefaultProvider.GetSymbolWriter(module, fileName);
         }
 
-        public ISymbolWriter GetSymbolWriter( ModuleDefinition module, Stream symbolStream )
+        /// <summary>Get a symbol writer for a given module and symbol stream.</summary>
+        /// <param name="module">The loaded assembly module.</param>
+        /// <param name="symbolStream">The loaded symbol file stream.</param>
+        public ISymbolWriter GetSymbolWriter(ModuleDefinition module, Stream symbolStream)
         {
             // Not implemented in default native pdb writer, so fallback to portable
-            return new PortablePdbWriterProvider().GetSymbolWriter( module, symbolStream );
+            return this.PortablePdbProvider.GetSymbolWriter(module, symbolStream);
         }
     }
 }