Re: [ft-devel] [freetype2] master fc11af1: Various minor clang fixes.

[Werner LEMBERG]

>> The `default' case is redundant, since all possible enum values are
>> covered in the `switch' statement.
> 
> I personally prefer having code in front of me that I can prove
> correct, than having to go fish for all the places that value in the
> switch statement is set to convince myself that it can't possibly
> have any other value...

Well, I've commented this code out to fix a clang warning...  Today,
compilers tell you reliably whether a `default' label is redundant or
missing, so I consider your constraints as a non-issue.

> At any rate, having "#if 0" code like this is not a good idea as 5
> years from now no one will remember why that code is there and why
> it's disabled.

True.


    Werner