[alsa-devel] [PATCH v2] ASoC: da7210: Add support for PLL and SRM

Mark Brown broonie at opensource.wolfsonmicro.com
Tue Apr 17 16:11:28 CEST 2012


On Tue, Apr 17, 2012 at 07:51:00PM +0530, Ashish Chavan wrote:
> On Tue, 2012-04-17 at 14:43 +0100, Mark Brown wrote:

> > I've applied but this would be a lot clearer if it were writen as a
> > single if statement with &&s.

> Actually I implemented it like that first. But then realized that
> because of the 80 chars limit and long conditions, readability was
> hampered. Let me submit a patch to convert it to single if, if that is
> preferred way of doing it.

Hrm?  The major problem with multiple if statements is that it introduces
massive levels of indentation which are hard to parse.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20120417/dbaa0358/attachment.sig 


More information about the Alsa-devel mailing list