Modify

Opened 9 years ago

Closed 7 years ago

Last modified 7 years ago

#2941 closed Tasks (fixed)

boost/pool/pool.hpp uses reinterpret_cast instead of static_cast

Reported by: philippe.vaucher Owned by: Chris Newbold
Milestone: Boost 1.39.0 Component: pool
Version: Boost 1.38.0 Severity: Optimization
Keywords: reinterpret_cast static_cast Cc:

Description

In pool.hpp there's a lot of reinterpret_cast usage like so:

static char * malloc(const size_type bytes)
{
  return reinterpret_cast<char *>(std::malloc(bytes));
}

This is a case where static_cast should be used instead. reinterpret_cast is known to yield implementation specific behavior.

Attachments (0)

Change History (2)

comment:1 Changed 7 years ago by Steven Watanabe

Resolution: fixed
Status: newclosed

(In [62705]) Use static_cast instead of reinterpret_cast. Fixes #2941

comment:2 Changed 7 years ago by Marshall Clow

(In [67958]) Merging fixes to release; Fixes #2696; Refs #4346; Refs #2941; Refs #386

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Chris Newbold.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.