Use std::min replace min

[KangLin]

Use std::min replace min

[CendioOssman]

The PR looks mostly fine, but I'm unclear to the motivation. You mention "cross-platform", but how does this change improve that?

[CendioOssman]

I think the code looks good now. But I'm still not clear on the rationale. What was the reason you started working on this fix?

[KangLin]

- There are standard functions available, so why define them yourself? - I am will use to msvc.

[CendioOssman]

• There are standard functions available, so why define them yourself?

Generally, that's a good principle. But std::min() has some bad behaviours based on the fact that it is implemented as a template, which means it isn't always a good fit.

We also don't want to avoid creating unnecessary history, as that makes git blame annoying to use. In other words, there should be a decently sized cleanup.

That said, this didn't change too many lines. So I can accept this if it makes things easier for you.

• I am will use to msvc.

Ah. That's not one of our normally used compilers, but it's always a plus if we're more compatible.

[KangLin]

But std::min() has some bad behaviours based on the fact that it is implemented as a template, which means it isn't always a good fit.

Please describe any bad behaviours？