search

tadly / hideIt.sh

Automagically hide/show a window by its name when the cursor is within a defined region or you mouse over it.
GNU General Public License v3.0
248 stars 11 forks source link

Clean up child processes on exit #8

Closed manvithn closed 4 years ago

manvithn commented 4 years ago

When using --hover, the xev and while loop child processes are not cleaned up on exit. Similarly, when using --signal, the sleep child process is not cleaned up on exit. This commit enables job control and adds cleanup logic to the exit signal handler.

tadly commented 4 years ago

Thank you very much for your contribution.

While testing I noticed that sending a SIGUSR1 now "kills" the process which is not what should happen.

Mind looking at this again?

manvithn commented 4 years ago

Fixed! I'll squash the commits if everything looks okay.

tadly commented 4 years ago

Awesome. Works just fine now!

Again, thank you very much for the contribution :)