Modify

Ticket #2941 (closed Tasks: fixed)

Opened 5 years ago

Last modified 3 years ago

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

Reported by: philippe.vaucher Owned by: cnewbold
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

Change History

comment:1 Changed 4 years ago by steven_watanabe

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

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

comment:2 Changed 3 years ago by marshall

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

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.