Async jsonrpc - Githubissues

svaante / dape

Debug Adapter Protocol for Emacs

GNU General Public License v3.0

477 stars 31 forks source link

Async jsonrpc #40

Closed svaante closed 9 months ago

svaante commented 10 months ago

I am deeply sorry João

I know you think this an inferior solution... and I am butchering the transforms.

There is some more stuff that I need to do before I am happy.

Push all connection related var into to class properties including parent connection
Update doc strings
Move stuff over from dape-debug into events buffer
Move away from dape--live-connection
Remove dape--setup as it does not really have a purpose
jsonrpc bootstrap?

joaotavora commented 10 months ago

No need to be sorry :-) I'm fixing the bug you reported in jsonrpc.el right now. My advice would be to wait for that, but it's your project :-)

joaotavora commented 9 months ago

Hi @svaante I've left some very generic comments, which probably are not unexpected. My opinion is that in the longer run, async call-back passing jungles become hard to manage, but maybe you have some good reason to prefer this over #37 , which is how I think jsonrpc.el should be integrated. Anyway, if this passes tests it is a step in the right direction.