Merge pull request #20 from tintoy/master

Skip dynamic registration of server capabilities if client does not support it

Author: david-driscoll

src/JsonRpc/JsonRpc.csproj

@@ -2,6 +2,7 @@
     <PropertyGroup>
         <TargetFrameworks>net46;netstandard1.6</TargetFrameworks>
         <PlatformTarget>AnyCPU</PlatformTarget>
+        <LangVersion>Latest</LangVersion>
         <!-- Needed for Microsoft.Composition -->
         <PackageTargetFallback Condition=" '$(TargetFramework)' == 'netstandard1.6' ">$(PackageTargetFallback);dotnet5.4;portable-net45+win8</PackageTargetFallback>
         <PackageVersion Condition="'$(GitVersion_NuGetVersion)' != ''">$(GitVersion_NuGetVersion)</PackageVersion>

src/Lsp/LanguageServer.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Concurrent;
 using System.Collections.Generic;
 using System.IO;
@@ -213,6 +213,9 @@ private async Task DynamicallyRegisterHandlers()
                 registrations.Add(handler.Registration);
             }
 
+            if (registrations.Count == 0)
+                return; // No dynamic registrations supported by client.
+
             var @params = new RegistrationParams() { Registrations = registrations };
 
             await this.RegisterCapability(@params);
@@ -260,4 +263,4 @@ public void Dispose()
 
         public IDictionary<string, JToken> Experimental { get; } = new Dictionary<string, JToken>();
     }
-}
+}