Subject: Re: [boost] [test] [gcc-4.3.2] check_is_close and check_is_small issues
From: Juergen Hunold (juergen.hunold_at_[hidden])
Date: 2009-02-18 14:06:42
Hi Kim !
On Wednesday 18 February 2009, Kim Barrett wrote:
> At 1:01 PM +0800 2/18/09, Dean Michael Berris wrote:
> > I have recently upgraded to Boost 1.38 from Boost 1.35 and
> > suddenly I get the following warnings from GCC 4.3.2:
> > ../boost_1_38_0/boost/test/floating_point_comparison.hpp:224:
> > error: 'boost::test_tools::<unnamed>::check_is_close' defined but
> > not used
> > ../boost_1_38_0/boost/test/floating_point_comparison.hpp:246:
> > error: 'boost::test_tools::<unnamed>::check_is_small' defined but
> > not used
Sigh, yes those were introduced in 1.37 (maybe 1.36).
A search in the Trac reveals that this is #2390
> > I have warnings treated as errors in my build and I haven't
> > encountered this issue before.
> > Can this be considered a regression and if yes what can I do to
> > work around it?
Well, those issues are fixed in trunk, but were not merged to release in
time. I hope Gennadyi will merge them for 1.39.
> I ran into this same problem with the upgrade. For the moment my
> solution is to comment out the offending definitions for
> check_is_close and check_is_small, and change test_tools.hpp:
> - replace all (8) occurrences of check_is_close with
> check_is_close_t() - replace all (4) occurrences of check_is_small
> with check_is_small_t()
This is definitely ugly ;-))
The real patch is avaiable at
And you might want to apply the second part which is
This fixes all warnings for me.
-- * Dipl.-Math. Jürgen Hunold ! Ingenieurgesellschaft für * voice: ++49 511 262926 57 ! Verkehrs- und Eisenbahnwesen mbH * fax : ++49 511 262926 99 ! Lister Straße 15 * juergen.hunold_at_[hidden] ! www.ivembh.de * * Geschäftsführer: ! Sitz des Unternehmens: Hannover * Prof. Dr.-Ing. Thomas Siefer ! Amtsgericht Hannover, HRB 56965 * PD Dr.-Ing. Alfons Radtke !
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk