Ubuntu Lucid's gcc 4.4.3 does not like it if not each possible value
of an enumeration is handled in the body of a switch statement.
Since gengetopt for some reason always creates an additional xxx_NULL
value in options of type enum, this more or less means we have to
introduce default: cases to avoid warnings of the form
fade.c:113: warning: enumeration value ‘mixer_channel__NULL’ not handled in switch
fade.c:346: warning: enumeration value ‘mode__NULL’ not handled in switch
Fortunately, these are the only two warnings, so add default: cases
to make gcc happy.
case mixer_channel_arg_reclev: val = SOUND_MIXER_RECLEV; break;
case mixer_channel_arg_igain: val = SOUND_MIXER_IGAIN; break;
case mixer_channel_arg_ogain: val = SOUND_MIXER_OGAIN; break;
+ default: break;
}
conf.mixer_channel_arg = val;
}
}
fixup_mixer_channel_arg();
switch (conf.mode_arg) {
- case mode_arg_sleep:
- ret = sweet_dreams();
- break;
case mode_arg_fade:
ret = fade(conf.fade_vol_arg, conf.fade_time_arg);
break;
case mode_arg_snooze:
ret = snooze();
break;
+ default: /* sleep mode */
+ ret = sweet_dreams();
+ break;
}
if (ret < 0)
PARA_EMERG_LOG("%s\n", para_strerror(-ret));