[Qgis-developer] Re: OSX build notes

[Marco Hugentobler]

I'm also in favour of turning pedantic off by default. Because people which 
are familiar with C++ compiler but not familiar with QGIS code don't know 
that pedantic flag exists. So they will have frequent problems if they don't 
use the same compiler versions as QGIS developers.
But I agree that it would be good for developers to turn it on (I promise to 
do so in future :-) ).

Marco

On Wednesday 06 June 2007 14:20:31 Martin Dobias wrote:
> On 6/5/07, Tim Sutton <tim at linfiniti.com> wrote:
> > Once school of thought says all warnings should be treated as a bug,
> > and that we should not allow code into SVN that has warnings. So
> > instead of doing svn checkout and turning pedantic on, you should do
> > checkout and turn pedantic *off* if you are disinclined to use them.
> > If you are doing dev work and you have unused vars etc then just turn
> > off pedantic while developing, then turn them on again for a final
> > build before committing,
> >
> > If warnings arise on different compilers etc we should treat these as
> > bugs too.  I thought at one stage pedantic was turned off in SVN -
> > maybe one of us (me turned it on and committed again without
> > realising. Ideally the warnings in WFS plugin can be resolved and we
> > can remove that warning. In summary, I prefer to keep pedantic on...
>
> My concern was that it's hard to ensure that some piece of code won't
> emit warnings on other compilers. It's less convenient for users when
> the warnings block their compilation... Of course I'm not insisting to
> turn in off, only thinking about possible problems :-)
>
> Cheers
> Martin
> _______________________________________________
> Qgis-developer mailing list
> Qgis-developer at lists.qgis.org
> http://lists.qgis.org/cgi-bin/mailman/listinfo/qgis-developer



-- 
Dr. Marco Hugentobler
Institute of Cartography
ETH Zurich
Technical Advisor QGIS Project Steering Committee
marco.hugentobler at karto.baug.ethz.ch