On Fri, Jul 05, 2013 at 06:13:53PM +0200, Peter Stuge wrote:
Please consider not having a default: case in the two switch () statements added by this patch.
No default: case makes the compiler warn if a value is added to the enum in the future, without corresponding cases being added to the switch ().
I agree with that. :)