[Qgis-developer] Re: OSX build notes

[Martin Dobias]

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