Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#8733 closed Feature Requests (fixed)

Testing for unprotected references to C99 math macros

Reported by: Mike Liang <mtliang@…> Owned by: John Maddock
Milestone: To Be Determined Component: math
Version: Boost Development Trunk Severity: Problem
Keywords: Cc:

Description

Attached is a way to test for unprotected references to C99 math macros on platforms where they are implemented as functions. Adding this to a released header would make it convenient to test new code for any omissions.

I used this to find the problems submitted in ticket #8732.

With, for example, fpclassify redefined, any unprotected references will expand out to a syntax error. 

#define fpclassify(x) (another_fpclassify(x))
bad_call = boost::math::fpclassify(arg);

becomes 

bad_call = boost::math::(another_flclassify(arg));

Attachments (1)

math_fwd.hpp.patch​ (1.7 KB ) - added by Mike Liang <mtliang@…> 9 years ago.
simulate C99 math macros

Download all attachments as: .zip

Change History (3)

by Mike Liang <mtliang@…>, 9 years ago

Attachment: math_fwd.hpp.patch​ added

simulate C99 math macros

comment:1 by John Maddock, 9 years ago

Resolution: → fixed
Status: new → closed

(In [84917]) Add test for things that are macros in C99 to ensure we call these correctly. Refs #8732. Fixes #8733.

comment:2 by John Maddock, 9 years ago

(In [85987]) Merge accumulated patches from Trunk. Refs #8384, Refs #8855, refs #9107, refs #9109, refs #8333, refs #8621, refs #8732, refs #8733, refs #8837, refs #8940, refs #9042, refs #9087, refs #9104, refs #9126.

Note: See TracTickets for help on using tickets.