[alsa-devel] [PATCH 002b/003] snd-powermac: style awacs.s and awacs.h
Takashi Iwai
tiwai at suse.de
Wed Apr 16 13:03:43 CEST 2008
At Wed, 16 Apr 2008 13:53:31 +0300,
Risto Suominen wrote:
>
> 2008/4/16, Takashi Iwai <tiwai at suse.de>:
> >
> > Thanks, that works. But, I get checkpatch errors:
> >
> > ERROR: do not use assignment in if condition
> > #116: FILE: ppc/awacs.c:704:
> >
> > + if ((err = snd_ctl_add(chip->card,
> >
> >
> > ERROR: do not use assignment in if condition
> > #171: FILE: ppc/awacs.c:982:
> >
> > + if ((err = build_mixers(chip,
> >
> >
> > ERROR: do not use assignment in if condition
> > #191: FILE: ppc/awacs.c:999:
> >
> > + if ((err = build_mixers(chip,
> >
> >
> >
> > Please fix?
> >
>
> I can do that but it will make the code much clumsier. Actually, I
> think this is checkpatch misunderstanding something. There is a '< 0'
> at the end of those conditions. Is it really this kind of assingment
> they want to forbid? This is the style there has been all the time.
The style "if ((err = xxx) < 0) ..." has been disliked by many people,
and now it became to a style to be avoided. This should be simply
err = xxx;
if (err < 0)
...
Takashi
More information about the Alsa-devel
mailing list