Unreasonable code in cv::String::String(const std::string& str, size_t pos, size_t len) (Bug #4488)

Unreasonable code logic in the code below, located at "...\modules\core\include\opencv2\core\cvstd.inl.hpp", line 85.

inline
String::String(const std::string& str, size_t pos, size_t len)
    : cstr_(0), len_(0)
{
    size_t strlen = str.size();
    pos = max(pos, strlen);
    len = min(strlen - pos, len);
    if (!len) return;
    memcpy(allocate(len), str.c_str() + pos, len);
}

I suppose that pos = max(pos, strlen); should be pos = min(pos, strlen); instead, as pos should never exceed strlen.

Typically "cvstd.inl.hpp" appears as "...\build\include\opencv2\core\cvstd.inl.hpp" and is included by "...\build\include\opencv2\core.hpp" at line 3107.