Re: [PATCH 5/5] ASoC: fix uninitialized variable

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 04, 2010 at 08:59:49AM -0400, Bill Pemberton wrote:

> > No, just stomping over the variable to shut up the warning doesn't
> > improve anything at all.  You need to make sure that the compiler can
> > actually follow the control flow to see that the data is always
> > initialised rather than disabling the check and missing any actual
> > issues.

> I did, hence the comment in the commit that the variable is
> initialized by fll_factors().

You're completely missing the point here.  What I said was that you need
to make sure that the *compiler* can actually follow the control flow.
You've not done that, you've stomped over the warning to shut both this
and any valid warnings related to the structure up.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux