byulparan / cl-collider

A SuperCollider client for CommonLisp
Other
218 stars 23 forks source link

LispWorks specific: the cleanup should be in "When quitting image" #124

Closed Yehouda closed 1 year ago

Yehouda commented 1 year ago

Currently, the quit cleanup is defined in the "Confirm when quitting image" action list: https://github.com/byulparan/cl-collider/blob/d601c1b6ace8fb7cf6d1ee0862a85fd80441d1ab/server.lisp#L457

It should be in the "When quitting image" action list instead.

The lists are documented (briefly) here: http://www.lispworks.com/documentation/lw80/lw/lw-action-ug-6.htm

Also:

1) The call to thread-wait inside server-quit should be changed to thread-wait-with-timeout to avoid hanging. https://github.com/byulparan/cl-collider/blob/d601c1b6ace8fb7cf6d1ee0862a85fd80441d1ab/server.lisp#L255 2) The call bt:join-thread inside cleanup-server should cbe changed to something that does not hang if domething goes wrong with the thread. https://github.com/byulparan/cl-collider/blob/d601c1b6ace8fb7cf6d1ee0862a85fd80441d1ab/server.lisp#L407

byulparan commented 1 year ago

I changed "When quitting image" for Lispworks, thanks. eefe90f945b4bcfa8a4bc0e0e4975c195eb0bc62

but improve about server-quit process needs time to think.