[PATCH] Tidyup. Remove a bunch of "unused variable _relative_save_offset" warnings.

Andreas Schneider asn at samba.org
Sun May 12 12:40:53 MDT 2013


On Friday 10 May 2013 08:56:33 Jeremy Allison wrote:
> On Fri, May 10, 2013 at 09:34:10AM +0200, Andreas Schneider wrote:
> > On Wednesday 08 May 2013 09:58:56 Jeremy Allison wrote:
> > > Ok, pushed. This does teach us not to ignore warnings
> > > however - there was an actual bug there (just not the
> > > one I fixed :-).
> > > 
> > > I'd like to have a code sprint sometime soon where
> > > we eliminate as many compiler warnings out of the
> > > master branch as possible. We used to be virtually
> > > warning-free, we've let this slip too much recently
> > > and we need to fix it up (IMHO).
> > 
> > I normally compile use -pedantic and -pedantic-error in my projects which
> > forces you to write cleaner code. I would vote to use them in Samba too :)
> 
> Yep ! But first we need to get there, and then we
> turn them on to stop us sliding back :-).
>
> Maybe you and I can work together on eliminating
> all warnings over the next few months ?

Sure, as soon as my bug count is down to a reasonable number again.


	-- andreas


-- 
Andreas Schneider                   GPG-ID: F33E3FC6
Samba Team                             asn at samba.org
www.samba.org



More information about the samba-technical mailing list