[alsa-devel] [patch] ASoC: wm8994: missing break in wm8994_aif3_hw_params()

Mark Brown broonie at kernel.org
Tue Apr 30 20:07:10 CEST 2013


On Tue, Apr 30, 2013 at 09:49:34AM +0200, walter harms wrote:

> the patch seems ok, but i would question the wisdom of a nested switch here,
> obviously the readability suffers and since it is only one case it seems
> better to be replaced it with

I don't think that's a good idea, it does nothing for the readability
and isn't idiomatic.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20130430/a8f5b76f/attachment.sig>


More information about the Alsa-devel mailing list