Replace weird qMin/qMax combinations with std::min, max and clamp - Githubissues

procxx / kepka

Unofficial Telegram Desktop messaging app

https://procxx.github.io

Other

114 stars 29 forks source link

Replace weird qMin/qMax combinations with std::min, max and clamp #36

Open berkus opened 6 years ago

berkus commented 6 years ago

Code in many places uses qMin+qMax or even qMin+qMin to clamp values, just use std::clamp.

Probably replace qXxx versions with their std counterparts - ?

See here for reference.

Mini task-list (added by @leha-bot):

[x] std::min - done in #89
[x] std::max - done in #89
[ ] std::clamp - requested in #89 but postponed to another PR.

leha-bot commented 6 years ago

The motivating links:

https://github.com/procxx/kepka/search?utf8=%E2%9C%93&q=qMin https://github.com/procxx/kepka/search?utf8=%E2%9C%93&q=qMax