Fix issue with Session not being garbage collected

[rchl]

Sessions were not being garbage collected for two reasons:

• In connection with request_handler / notification_handler decorators, we've stored reference to methods on self. Since methods have hard reference to self that created a cycle. Store only mapping from handler name to m_* handler and use that for calling the right function when m_* method is called.
• The get_diagnostics_identifiers function uses lru_cache decorator which stores reference to Session. We need to clear the cache when session is destroyed. This is not a proper fix since it clears the whole cache instead of only caches related to ended session but I don't have time now to investigate it further. Addressed by switching to manual caching.

For testing one can add

    def __del__(self):
        print('DEL Session')

in Session to verify that session is destroyed on last file in the window being closed.

[rchl]

Switched to manual caching for diagnostic identifiers.

[jwortmann]

_reset is called when either the syntax or the URI of the view changes:

LSP/plugin/documents.py

Lines 1128 to 1129 in e74ce6b

	if something_changed:
	self._reset()

In that case the DocumentSelector might not match anymore for the diagnostic provider(s). That is the reason why we need to clear the cache here.

For the manual cache, you could also just pop the value for this particular view from the cache instead. That would even be more efficient than clearing the entire cache.

[netlify]

✅ Deploy Preview for sublime-lsp ready!

Name	Link
🔨 Latest commit	84d2b45
🔍 Latest deploy log	https://app.netlify.com/projects/sublime-lsp/deploys/69b072514ae3e90008e5ef7c
😎 Deploy Preview	https://deploy-preview-2801--sublime-lsp.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.