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

[philippe.vaucher]

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.

[Steven Watanabe]

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

[Marshall Clow]

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