Opened 10 years ago

Closed 8 years ago

Last modified 8 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:


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.

Change History (2)

comment:1 Changed 8 years ago by Steven Watanabe

Resolution: fixed
Status: newclosed

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

comment:2 Changed 8 years ago by Marshall Clow

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

Note: See TracTickets for help on using tickets.