RESOLVED WONTFIX 214543
Use std::clamp for clarity at more call sites 
https://bugs.webkit.org/show_bug.cgi?id=214543
Summary Use std::clamp for clarity at more call sites
Darin Adler
Reported 2020-07-19 22:54:21 PDT
Use std::clamp for clarity at more call sites
Attachments
Patch (102.40 KB, patch)
2020-07-19 23:20 PDT, Darin Adler 		no flags
DetailsFormatted DiffDiff
Patch (102.41 KB, patch)
2020-07-20 10:36 PDT, Darin Adler 		no flags
DetailsFormatted DiffDiff
Patch (102.18 KB, patch)
2020-07-21 13:04 PDT, Darin Adler 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2020-07-19 23:20:15 PDT Comment hidden (obsolete)
Darin Adler
Comment 2 2020-07-20 10:36:45 PDT
Created attachment 404725 [details] Patch
Darin Adler
Comment 3 2020-07-21 13:04:45 PDT
Created attachment 404854 [details] Patch
Darin Adler
Comment 4 2020-07-21 15:35:57 PDT
Unfortunately I have learned if high is less than low, then std::clamp(x, low, high) returns high.
Radar WebKit Bug Importer
Comment 5 2020-07-26 22:55:17 PDT
<rdar://problem/66146541>
Darin Adler
Comment 6 2022-11-12 21:36:07 PST
Might do this some day, but not going to keep a bug open.
Note You need to log in before you can comment on or make changes to this bug.