Bug 214543

Summary: Use std::clamp for clarity at more call sites
Product: WebKit Reporter: Darin Adler <darin>
Component: WebKit Misc.Assignee: Darin Adler <darin>
Status: RESOLVED WONTFIX    
Severity: Normal CC: benjamin, calvaris, cdumez, changseok, cmarcelo, dbarton, dino, eric.carlson, esprehn+autocc, ews-watchlist, fmalita, fred.wang, glenn, graouts, gyuyoung.kim, hta, jer.noble, jfernandez, kangil.han, keith_miller, kondapallykalyan, macpherson, mark.lam, menard, mifenton, mmaxfield, msaboff, pdr, philipj, rego, saam, sabouhallawa, sam, schenney, sergio, simon.fraser, svillar, tommyw, tzagallo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Patch
none
Patch none

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.