Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmdserver is started and killed every operation during normal use #109

Open
ncalexan opened this issue Jan 24, 2021 · 1 comment
Open

cmdserver is started and killed every operation during normal use #109

ncalexan opened this issue Jan 24, 2021 · 1 comment

Comments

@ncalexan
Copy link
Contributor

ncalexan commented Jan 24, 2021

I've used monky for ages and just noticed that during "normal use" the cmdserver process will be started and killed every operation. At least, that's what I witness and what the use of unwind-protect around here suggests:

monky/monky.el

Lines 603 to 607 in e046322

(unwind-protect
(progn ,@body)
(when (and monky-cmd-process outer (eq monky-process-type 'cmdserver))
(delete-process monky-cmd-process)
(setq monky-cmd-process nil)))))

The user can work around this with a manual (setq monky-cmd-process (monky-cmdserver-start)) but that's both awkward and not documented.

Perhaps this was supposed to be condition-case so that the delete-process only happens in error cases? Or perhaps there's supposed to be some other flow in normal use that I don't know about?

@Wilfred
Copy link
Collaborator

Wilfred commented Jan 25, 2021

Yeah, I agree that looks broken. I've gradually been trying to reduce the amount of macros in monky, because I find the code hard to reason about.

ncalexan added a commit to ncalexan/monky that referenced this issue Jan 28, 2021
ncalexan added a commit to ncalexan/monky that referenced this issue Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants