
20 Aug
2010
20 Aug
'10
4:39 p.m.
Jim Fehlig wrote:
Eric Blake wrote:
Hmm. These two new values are only valid for on_crash, but I don't see any code that rejects them for on_reboot or on_poweroff. Do we need a separate enum here, or do we just need to add better checking to the remaining clients to detect enum values they can't support?
Good point. I moved these new options into a new enum and adjusted the code accordingly.
V2 attached.
Any comments on V2 of this patch? Regards, Jim