chore: dead code elimination

src/Docfx.App/RunBuild.cs

@@ -6,8 +6,6 @@
 
 namespace Docfx;
 
-#pragma warning disable CS0618 // Type or member is obsolete
-
 /// <summary>
 /// Helper class to build document.
 /// </summary>
@@ -20,7 +18,7 @@ public static string Exec(BuildJsonConfig config, BuildOptions options, string c
     {
         if (config.Template == null || config.Template.Count == 0)
         {
-            config.Template = new ListWithStringFallback { Constants.DefaultTemplateName };
+            config.Template = new ListWithStringFallback { "default" };
         }
 
         var baseDirectory = Path.GetFullPath(string.IsNullOrEmpty(configDirectory) ? Directory.GetCurrentDirectory() : configDirectory);

src/Docfx.Build/HostServiceCreator.cs

@@ -9,7 +9,7 @@
 
 namespace Docfx.Build.Engine;
 
-internal class HostServiceCreator : IHostServiceCreator
+class HostServiceCreator
 {
     private readonly DocumentBuildContext _context;
 

src/Docfx.Build/LinkPhaseHandler.cs

@@ -87,10 +87,10 @@ private InternalManifestItem HandleSaveResult(
         SaveResult result)
     {
         Context.SetFilePath(model.Key, ((RelativePath)model.File).GetPathFromWorkingFolder());
-        DocumentException.RunAll(
-            () => CheckFileLink(model, hostService, result),
-            () => HandleUids(result),
-            () => RegisterXRefSpec(result));
+
+        CheckFileLink(model, hostService, result);
+        HandleUids(result);
+        RegisterXRefSpec(result);
 
         return GetManifestItem(model, result);
     }

src/Docfx.Build/PostProcessors/ValidateBookmark.cs

@@ -16,15 +16,12 @@ public sealed class ValidateBookmark : HtmlDocumentHandler
     /// <summary>
     /// bookmarks mapping from output file -> bookmarks
     /// </summary>
-    private readonly OSPlatformSensitiveDictionary<HashSet<string>> _registeredBookmarks =
-        new();
+    private readonly Dictionary<string, HashSet<string>> _registeredBookmarks = new(FilePathComparer.OSPlatformSensitiveStringComparer);
     /// <summary>
     /// file mapping from output file -> src file
     /// </summary>
-    private readonly OSPlatformSensitiveDictionary<string> _fileMapping =
-        new();
-    private readonly OSPlatformSensitiveDictionary<List<LinkItem>> _linksWithBookmark =
-        new();
+    private readonly Dictionary<string, string> _fileMapping = new(FilePathComparer.OSPlatformSensitiveStringComparer);
+    private readonly Dictionary<string, List<LinkItem>> _linksWithBookmark = new(FilePathComparer.OSPlatformSensitiveStringComparer);
 
     #region IHtmlDocumentHandler members
 

src/Docfx.Build/SingleDocumentBuilder.cs

@@ -56,13 +56,12 @@ public Manifest Build(DocumentBuildParameters parameters, IMarkdownService markd
 
         // Start building document...
         List<HostService> hostServices = null;
-        IHostServiceCreator hostServiceCreator = null;
         try
         {
             using var templateProcessor = parameters.TemplateManager?.GetTemplateProcessor(context, parameters.MaxParallelism)
                                           ?? new TemplateProcessor(new EmptyResourceReader(), context, 16);
 
-            hostServiceCreator = new HostServiceCreator(context);
+            var hostServiceCreator = new HostServiceCreator(context);
             hostServices = GetInnerContexts(parameters, Processors, templateProcessor, hostServiceCreator, markdownService);
 
             templateProcessor.CopyTemplateResources(context.ApplyTemplateSettings);
@@ -100,7 +99,7 @@ private List<HostService> GetInnerContexts(
         DocumentBuildParameters parameters,
         IEnumerable<IDocumentProcessor> processors,
         TemplateProcessor templateProcessor,
-        IHostServiceCreator creator,
+        HostServiceCreator creator,
         IMarkdownService markdownService)
     {
         var files = (from file in parameters.Files.EnumerateFiles().AsParallel().WithDegreeOfParallelism(parameters.MaxParallelism)

src/Docfx.Common/Loggers/LoggerFileScope.cs

@@ -5,6 +5,7 @@ namespace Docfx.Common;
 
 public sealed class LoggerFileScope : IDisposable
 {
+    private static readonly AsyncLocal<string> t_fileName = new();
     private readonly string _originFileName;
 
     public LoggerFileScope(string fileName)
@@ -22,13 +23,7 @@ public void Dispose()
         SetFileName(_originFileName);
     }
 
-    internal static string GetFileName()
-    {
-        return LogicalCallContext.GetData(nameof(LoggerFileScope)) as string;
-    }
+    internal static string GetFileName() => t_fileName.Value;
 
-    private static void SetFileName(string fileName)
-    {
-        LogicalCallContext.SetData(nameof(LoggerFileScope), fileName);
-    }
+    private static void SetFileName(string fileName) => t_fileName.Value = fileName;
 }

src/Docfx.Common/StringExtension.cs

@@ -27,23 +27,6 @@ public static string ToDelimitedString(this IEnumerable<string> input, string de
         return string.Join(delimiter, input);
     }
 
-    public static string GetNormalizedFullPathKey(this IEnumerable<string> list)
-    {
-        if (list == null) return null;
-
-        // make sure the order consistent
-        var normalizedPaths = GetNormalizedFullPathList(list);
-        return normalizedPaths.ToDelimitedString();
-    }
-
-    public static IEnumerable<string> GetNormalizedFullPathList(this IEnumerable<string> paths)
-    {
-        if (paths == null) return null;
-        return (from p in paths
-                where !string.IsNullOrEmpty(p)
-                select ToNormalizedFullPath(p)).Distinct().OrderBy(s => s);
-    }
-
     /// <summary>
     /// Should not convert path to lower case as under Linux/Unix, path is case sensitive
     /// Also, Website URL should be case sensitive consider the server might be running under Linux/Unix