[alsa-devel] [PATCH] ASoC: Fix warning from strict_strtoul()

[Takashi Iwai]

At Fri, 4 Jul 2008 20:57:54 +0100,
Mark Brown wrote:
> 
> On Fri, Jul 04, 2008 at 05:09:36PM +0200, Takashi Iwai wrote:
> > (BTW, at the next time, a bit more patch description would be helpful.
> >  For example, in this case, it's actually not about warning but the
> >  behavior of strict_strtoul() that resets the value, if I understand
> >  correctly.)
> 
> It is a warning fix - I was passing the wrong type of variable to
> strict_strtoul(), though on most architectures int and long are the same
> in memory so the behaviour before and after the change is identical.
> The fix is to pass a temporary variable of the correct type in and then
> assign the result to pop_time rather than passing pop_time in directly.

Ah, OK, that I totally missed.

Anyway this also fixes another strange behavior - strict_strtoul()
stores 0 when it gets a parse error.  So, in the older version, the
pop_wait itself is reset to 0 at error while the value is kept in the
newer version.


thanks,

Takashi