With at least some compilers, "enum QUALITY_TYPE" is unsigned, and the

compiler warns about checking whether it's >= 0, as it always is.

Don't do that check - we don't use "type" until the switch statement,
and GCC should issue a warning if not all valid "enum QUALITY_TYPE"
values are handled in the switch statement.

Sort the items in the switch statement in the order in which they appear
in the enum definition, so it's clearer that it covers all values.

svn path=/trunk/; revision=21890

0 files changed, 0 insertions, 0 deletions