Code review comment for lp://qastaging/~francesco-marella/redshift/bug-588086-appindicator

Revision history for this message
Francesco Marella (francesco-marella) wrote :

> > commit 103: I thought that wrapping only the main loop is enough to catch
> when
> > the user hits the interrupt key. Am I wrong?
>
> I think it is important not to leave a redshift process running in the
> background no matter when a user decides to hit CTRL-c. That's why I had the
> try block go back to right after the external process is started.

OK. Just reverted that commit.

« Back to merge proposal