[alsa-devel] [PATCH 02/11] ASoC: Intel: Skylake: Fix delay wrap condition

Mark Brown broonie at kernel.org
Thu Feb 4 12:24:36 CET 2016


On Wed, Feb 03, 2016 at 05:59:45PM +0530, Vinod Koul wrote:

> +	delay = (hstream->bufsize == delay) ? 0 : delay;

Please don't abuse the ternery operator, if you're not inserting it into
the middle of another expression there's no reason not to use a normal
if statement.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20160204/801352e0/attachment.sig>


More information about the Alsa-devel mailing list