[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kernel-janitors
Subject:    Re: [alsa-devel] [patch] ASoC: wm8994: missing break in wm8994_aif3_hw_params()
From:       Mark Brown <broonie () kernel ! org>
Date:       2013-04-30 18:07:10
Message-ID: 20130430180710.GB26160 () sirena ! org ! uk
[Download RAW message or body]

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.

["signature.asc" (application/pgp-signature)]
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[prev in list] [next in list] [prev in thread] [next in thread]