Re: [Quantlib-dev] Interest in fixing MSVC Level 4 warnings

[Mike S. <ma...@gm...>]

Hi Luigi,

Thanks for your reply.

A quick upper bound on the amount of warnings that would need to be fixed
is 1800. This is high because VS reports the same warning multiple times if
the code in question is in headers, and also because boost has some level 4
warnings in uBLAS headers.

I'm more than willing to use my spare time to generate patches that would
fix all the warnings, especially if they appear in headers so people who
include quantlib headers can do so without needing to resort to workarounds
if they want to bump up the warning level. When I submit the patches, I'll
describe which warning the patch fixes and we can go from there.

Look for patches over the next week or two.

Mike


On Thu, Jan 10, 2013 at 9:31 AM, Luigi Ballabio <lui...@gm...>wrote:

> Hi Mike,
>     given that we won't be able to go entirely warning-free (due to
> QL_FAIL etc.) I wouldn't want you to invest too much time on this.
> But if you want to have a look at the warnings and check if there's
> anything that should be fixed, by all means go ahead.  A count might
> indeed be useful in order to assess if the thing is worth pursuing.
>
> Later,
>     Luigi
>
>
>
> On Sun, Jan 6, 2013 at 11:35 PM, Mike Sharpe <ma...@gm...> wrote:
> > Hi all,
> >
> > First, I'd like to start off by saying thanks for your hard work on
> > quantlib! This is my first post to a quantlib mailing list. I've recently
> > developed an interest in mathematical finance and discovered quantlib
> over
> > the holidays, which has only piqued my interest more.
> >
> > As I'm still new to the codebase and most of the techniques that are
> used in
> > the code, I'd like to give back in some other way and one of the ways I
> can
> > do that is by looking into fixing some of the level 4 warnings for the
> > Microsoft compilers. Because the warnings could appear anywhere and I
> don't
> > want to cause any difficulties for other work, I wanted to check here to
> see
> > if there was any interest in seeing these warnings fixed.
> >
> > I'm not necessarily proposing fixing all of the warnings that I've seen
> in
> > the code (QL_FAIL triggers C4127 "conditional expression is constant"
> > because of the do { } while(false) macro guard, for example), but I do
> think
> > it'd a good idea to look through the list and fix some of them. If a
> count
> > of warnings would help, I can follow up with that soon.
> >
> > Mike
> >
> >
> >
> ------------------------------------------------------------------------------
> > Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
> > MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
> > with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
> > MVPs and experts. ON SALE this month only -- learn more at:
> > http://p.sf.net/sfu/learnmore_123012
> > _______________________________________________
> > QuantLib-dev mailing list
> > Qua...@li...
> > https://lists.sourceforge.net/lists/listinfo/quantlib-dev
> >
>