Opened 12 years ago

Closed 10 years ago

#4246 closed Bugs (fixed)

[value_init] Please replace Boost Test ("minimal.hpp") by lightweight_test

Reported by: niels_dekker Owned by: Fernando Cacciola
Milestone: Boost 1.43.0 Component: utility
Version: Boost 1.44.0 Severity: Problem
Keywords: Cc:

Description

At the moment, Boost Test has regression failures on various compilers, including IBM V11.1, Sun 5.10, MINGW-4.3.3, and Borland 6.2.1 (Codegear, Embarcadero). In utility/value_init_test.cpp, the following line triggers compile errors on those compilers:

#include "boost/test/minimal.hpp"

I think we should not continue resolving value_init issues like #3869 until "value_init_test" is colored entirely green again at http://www.boost.org/development/tests/trunk/developer/utility_.html

It seems to me that Borland is fundamentally broken in this case. In the past, Boost Test compiled well on Borland, but nowadays it has become too hard to swallow for Borland. For example, because it now depends on boost::exception. Boost Exception does not have enough support for Borland (ticket #4068).

Therefor I'm proposing to switch from Boost Test to the lightweight test library by Peter Dimov: http://svn.boost.org/svn/boost/trunk/boost/detail/lightweight_test.hpp

I would like to do so for http://svn.boost.org/svn/boost/trunk/libs/utility/value_init_test.cpp according to the attached patch.

Fernando, do you agree?

Attachments (1)

value_init_lightweight_test.patch (8.2 KB ) - added by niels_dekker 12 years ago.

Download all attachments as: .zip

Change History (3)

by niels_dekker, 12 years ago

comment:1 by niels_dekker, 12 years ago

(In [62158]) value_init_test now uses lightweight_test by Peter Dimov; see #4246. Fernando Cacciola mailed me he agreed as well.

comment:2 by viboes, 10 years ago

Resolution: fixed
Status: newclosed

Closed as for description it is already fixed.

Note: See TracTickets for help on using tickets.