Modify

Ticket #8733 (closed Feature Requests: fixed)

Opened 10 months ago

Last modified 7 months ago

Testing for unprotected references to C99 math macros

Reported by: Mike Liang <mtliang@…> Owned by: johnmaddock
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

math_fwd.hpp.patch Download (1.7 KB) - added by Mike Liang <mtliang@…> 10 months ago.
simulate C99 math macros

Change History

Changed 10 months ago by Mike Liang <mtliang@…>

simulate C99 math macros

comment:1 Changed 10 months ago by johnmaddock

  • Status changed from new to closed
  • Resolution set to fixed

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

comment:2 Changed 7 months ago by johnmaddock

(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.

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.