Merge fix

Author: Mikhail Arkhipov

src/LanguageServer/Impl/LanguageServer.cs

@@ -152,17 +152,7 @@ public async Task DidChangeConfiguration(JToken token, CancellationToken cancell
                 _idleTimeTracker?.Dispose();
                 _idleTimeTracker = new IdleTimeTracker(settings.diagnosticPublishDelay, PublishPendingDiagnostics);
 
-                _pathsWatcher?.Dispose();
-                var watchSearchPaths = GetSetting(analysis, "watchSearchPaths", true);
-                if (watchSearchPaths) {
-                    if (!_searchPaths.SetEquals(_initParams.initializationOptions.searchPaths)) {
-                        _pathsWatcher = new PathsWatcher(
-                            _initParams.initializationOptions.searchPaths,
-                            () => _server.ReloadModulesAsync(CancellationToken.None).DoNotWait(),
-                            _server
-                         );
-                    }
-                }
+                HandlePathWatchChange(token);
 
                 var errors = GetSetting(analysis, "errors", Array.Empty<string>());
                 var warnings = GetSetting(analysis, "warnings", Array.Empty<string>());

src/LanguageServer/Impl/PathsWatcher.cs

@@ -62,6 +62,7 @@ public PathsWatcher(string[] paths, Action onChanged, ILogger log) {
                     fsw.Deleted += OnChanged;
 
                     _disposableBag
+                        .Add(() => _throttleTimer?.Dispose())
                         .Add(() => fsw.Created -= OnChanged)
                         .Add(() => fsw.Deleted -= OnChanged)
                         .Add(() => fsw.EnableRaisingEvents = false)
@@ -83,7 +84,7 @@ private void OnChanged(object sender, FileSystemEventArgs e) {
 
         private void TimerProc(object o) {
             lock (_lock) {
-                if (_changedSinceLastTick) {
+                if (!_changedSinceLastTick) {
                     ThreadPool.QueueUserWorkItem(_ => _onChanged());
                     _throttleTimer?.Dispose();
                     _throttleTimer = null;