Fix std::min conflict with min/max definitions from windows.h

[Chuvi-w]

No description provided.

[skrobinson]

What about std::min<size_t>? Does specifying a type stop the macro replacement on Windows?

[Chuvi-w]

Seems that specifying a type also works well.

Both methods are acceptable.
But with parentheses you sholdn't change type if you'll change type of args sometime.

[skrobinson]

I agree about a type change. But in this case the type is size_t, which is unlikely to change soon.

Another option woud be std::min<decltype(std::declval<T>().size())>. Then the type would be correct for any container T with a size function.

[Chuvi-w]

I've thought about decltype.
But it looks llike... a bit overprinted.

[Bedzior]

Hi, my two cents: I guess std::size_t is more appropriate in a C++ library.

[skrobinson]

@Bedzior

Yes, you're right. I copied size_t from the definition of repr_max_container_size, but std::size_t is more correct.

Looking through the rest of the file, there is an opportunity for someone to convert many size_t.

[Chuvi-w]

Ok, but replacing size_t to std::size_t is not purpose of this PR.
Can you take it like this, and then somehow add std :: to the types?

[Chuvi-w]

Thanks