domsson / succade

Run, feed and style your Lemonbar with ease
Creative Commons Zero v1.0 Universal
55 stars 3 forks source link

No cleanup on SIGPIPE #16

Closed domsson closed 5 years ago

domsson commented 5 years ago

Sometimes (but not always), when ending succade via CTRL+C, we run into a SIGPIPE. This leads to some (theoretical) memory leaks because we don't get to the clean-up code. The OS takes care of it, so nothing too bad, but still not exactly nice.

We should either take the time to figure out what circumstances exactly lead to the SIGPIPE and how to prevent this, or simply catch the SIGPIPE with the existing signal handler, so that we can be sure clean-up will happen either way.

domsson commented 5 years ago

This was fixed the quick-and-dirty way (meaning we catch SIGPIPE) in 9d5d7da5ddd66a4e35ac9293723002f1bcf229b5