Replace Util::run with Util::runUntil that takes a bool reference instead of a pointer
Created attachment 370260 [details] Patch
Created attachment 370261 [details] Patch
Comment on attachment 370261 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=370261&action=review > Tools/TestWebKitAPI/Utilities.h:33 > -void run(bool* done); > +void runUntil(const bool& done); I made this (or at least I think it was me) a pointer rather than a reference because I thought it looked better at call sites to use the &. I still kind of feel that way, but not too strongly.
(In reply to Sam Weinig from comment #3) > Comment on attachment 370261 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=370261&action=review > > > Tools/TestWebKitAPI/Utilities.h:33 > > -void run(bool* done); > > +void runUntil(const bool& done); > > I made this (or at least I think it was me) a pointer rather than a > reference because I thought it looked better at call sites to use the &. I > still kind of feel that way, but not too strongly. I think the new name makes it clear what is going on, and I think this is a significant improvement.
This has bit rotted, and apparently wasn't popular.