[Webkit-unassigned] [Bug 130059] Refactor Vibration algorithm to use only 1 timer.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Mar 11 00:35:37 PDT 2014


https://bugs.webkit.org/show_bug.cgi?id=130059





--- Comment #3 from Darin Adler <darin at apple.com>  2014-03-11 00:32:33 PST ---
(From update of attachment 226363)
View in context: https://bugs.webkit.org/attachment.cgi?id=226363&action=review

> Source/WebCore/Modules/vibration/Vibration.cpp:72
> +    for (size_t i = 0; i < length; ++i) {
> +        if (sanitized[i] > MaxVibrationDuration)
> +            sanitized[i] = MaxVibrationDuration;
> +    }

I suggest writing it like this:

    for (auto& duration : sanitized)
        duration = std::min(duration, MaxVibrationDuration);

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list