Re: [libvirt] [PATCH v2 16/42] util: add default: case to all switch statements
On Thu, Feb 15, 2018 at 04:43:21PM +0000, Daniel P. Berrangé wrote:
Even if the compiler has validated that all enum constants have case
statements in a switch, it is not safe to omit a default: case
statement. When assigning a value to a variable / struct field that is
defined with an enum type, nothing prevents an invalid value being
assigned. So defensive code must assume existance of invalid values and
s/existance/existence/
Jan
thus all switches should have a default: case.
Signed-off-by: Daniel P. Berrangé <berrange(a)redhat.com>
Attachments:
- signature.asc (application/pgp-signature — 833 bytes)