assignment vs comparison in unit test

[Ulrich Eckhardt <ulrich.eckhardt@…>]

In file date_time\test\gregorian\testdate_input_facet.cpp is the following line:

    check("No extra characters consumed", m = greg_month(Mar) && *sitr == '.');

Compiling with VC8 gives a warning C4800: 'unsigned short' : forcing value to bool 'true' or 'false' (performance warning). I believe the problem is that m is being assigned to, so that the actual expression being given to check() or operator&& is m, which is converted to a boolean. I would also split the test into two lines, just for clarity:

    check("Correct input", m == greg_month(Mar));
    check("No extra characters consumed", *sitr == '.');

I found this bug in 1.48, but it is still present in the trunk, even though in the trunk the tests were converted from the ad-hoc test framework to Boost's general test framework. It might apply to other versions, too.

[Marshall Clow]

(In [80709]) Fix typo in test case; Refs #6513

[Marshall Clow]

(In [80797]) Merge bug fixes to release; Fixes #5550 Fixes #6136 Fixes #6513 Fixes #7111 Fixes #7112 Fixes #7113 Fixes #7342 Fixes #7426