On Sunday 11 November 2007 10:27:03 Frank Lichtenheld wrote:
> On Sat, Nov 10, 2007 at 05:27:43PM +0100, Michael Buesch wrote:
> > On Saturday 10 November 2007 16:25:33 John W. Linville wrote:
> > > On Sat, Nov 10, 2007 at 04:14:03PM +0100, Michael Buesch wrote:
> > > > From: Frank Lichtenheld <[EMAIL PROTECTED]>
> > > > 
> > > > inititalise ret to 0 to avoid the following bogus warning:
> > > >   CC [M]  drivers/net/wireless/b43/debugfs.o
> > > > drivers/net/wireless/b43/debugfs.c: In function ‘b43_debugfs_read’:
> > > > drivers/net/wireless/b43/debugfs.c:355: warning: ‘ret’ may be used 
> > > > uninitialized in this function
> > > > 
> > > > Signed-off-by: Frank Lichtenheld <[EMAIL PROTECTED]>
> > > > Signed-off-by: Michael Buesch <[EMAIL PROTECTED]>
> > > 
> > > Isn't this what uninitialized_var() is for?
> 
> Erh, I didn't actually knew that existed, sorry.
> OTOH in this case it is actually longer to write.
> 
> > I'd be OK with that, too.
> 
> Should I send a changed patch?

Yes.

-- 
Greetings Michael.
_______________________________________________
Bcm43xx-dev mailing list
Bcm43xx-dev@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev

Reply via email to