|
Boost : |
Subject: Re: [boost] [testing][regression] Heads up: compilation warnings are available for passing tests.
From: Andrzej Krzemienski (akrzemi1_at_[hidden])
Date: 2015-06-05 09:28:28
2015-06-05 0:14 GMT+02:00 Adam Wulkiewicz <adam.wulkiewicz_at_[hidden]>:
> 2015-06-03 17:27 GMT+02:00 Andrzej Krzemienski <akrzemi1_at_[hidden]>:
>
> > 2015-06-03 14:26 GMT+02:00 Adam Wulkiewicz <adam.wulkiewicz_at_[hidden]>:
> >
> > > Hi,
> > >
> > > For passing tests which are expected to pass, if warnings are generated
> > > during the compilation they can be seen after clicking on the "pass"
> > link.
> > > If the link isn't there it means that there were no warnings. I hope it
> > > will allow us to improve the quality of the libraries even more.
> > >
> >
> > This is really good.
> > How about changing the tint of the green color a bit for these cases?
> >
>
> Initially I thought that it might be distracting. But indeed it's more
> clear this way. So it's done.
>
> Regards,
> Adam
>
Adam,
Would it be possible to keep the passing tests with warnings green, and
only *slightly* change its tint? Currently they are white, and it sends a
strong signal that something is wrong with all these tests. I tried to
follow the links, but the majority of warnings are silly, buggy,
false-positive, or inadequate. For instance "warning: rvalue references are
a C++11 extension" I never intend to fix them, and at the same time, I do
want to send the message that the library is (or becomes) bug-free with the
help of green color.
Sorry for changing my mind like that.
Regards,
&rzej
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk