Oh wow, I didn't realize it had the restart in there whether options changed or not, originally. That's definitely not good. The first PR (i.e., only restart on change) was an improvement, then, but this one is better still, I think. Restarting is potentially destructive enough that we should leave it to the users in most/all cases, IMO.
Oh wow, I didn't realize it had the restart in there whether options changed or not, originally. That's definitely not good. The first PR (i.e., only restart on change) was an improvement, then, but this one is better still, I think. Restarting is potentially destructive enough that we should leave it to the users in most/all cases, IMO.