Clean up in forwarder install command

Author: nblumhardt

seqcli.sln.DotSettings

@@ -34,9 +34,11 @@
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=STORAGEPATH/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=subcommand/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=syslogdt/@EntryIndexedValue">True</s:Boolean>
+	<s:Boolean x:Key="/Default/UserDictionary/Words/=tcpip/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=Tokenizes/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=trailingindent/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=unawaited/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=Unclosable/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=urlacl/@EntryIndexedValue">True</s:Boolean>
+	<s:Boolean x:Key="/Default/UserDictionary/Words/=winmgmt/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/UserDictionary/Words/=xmpweb/@EntryIndexedValue">True</s:Boolean></wpf:ResourceDictionary>

src/SeqCli/Cli/Commands/Forwarder/InstallCommand.cs

@@ -101,7 +101,7 @@ void Install()
         if (netshResult != 0)
             Console.WriteLine($"Could not add URL reservation for {listenUri}: `netsh` returned {netshResult}; ignoring");
 
-        var exePath = Path.Combine(AppDomain.CurrentDomain.BaseDirectory!, Program.BinaryName);
+        var exePath = Path.Combine(AppDomain.CurrentDomain.BaseDirectory, Program.BinaryName);
         var forwarderRunCmdline = $"\"{exePath}\" forwarder run --pre --storage=\"{_storagePath.StorageRootPath}\"";
 
         var binPath = forwarderRunCmdline.Replace("\"", "\\\"");
@@ -117,7 +117,7 @@ void Install()
         var sc = Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.System), "sc.exe");
         if (0 != CaptiveProcess.Run(sc, scCmdline, Console.WriteLine, Console.WriteLine))
         {
-            throw new ArgumentException("Service setup failed");
+            throw new ArgumentException("Service setup failed.");
         }
 
         Console.WriteLine("Setting service restart policy...");
@@ -132,8 +132,6 @@ void Install()
 
     void GiveFullControl(string target)
     {
-        if (target == null) throw new ArgumentNullException(nameof(target));
-
         if (!Directory.Exists(target))
             Directory.CreateDirectory(target);
 
@@ -147,7 +145,7 @@ void GiveFullControl(string target)
     static string MakeListenUriReservationPattern(string uri)
     {
         var listenUri = uri.Replace("localhost", "+");
-        if (!listenUri.EndsWith("/"))
+        if (!listenUri.EndsWith('/'))
             listenUri += "/";
         return listenUri;
     }