Happy to help Michael, if somehow I did so.
I'm on the same opinion, that the reactor callbacks should be reduced (or atleast the ones that may fire-up others), hence one might be interested at my
Running Reactors Browser.
Your condition(s) look fine, aswell the event you're using.. so I don't see any room for improvement (BTW thanks for sharing).
Ofcourse there might be a small probability if the user invokes QSAVE at the same time where (zerop (getvar 'savetime)) = T.
Second look of the samples I posted, shows that the automatic save doesn't fire the beginSave event.
Also this could be included for your benchmark(s):
(defun dwg-reactor-save-complete-function ( calling-reactor command-info )
(if (wcmatch command-info "*\.sv$")
(AutoSaveTasksIfAny)
(UserInitiatedSaveTasks)
)
)
Although I'm not sure if the things I'm monitoring on my system can apply to the others.
Cheers.