[Webkit-unassigned] [Bug 192707] New: clang-tidy: Fix unnecessary object copy in CPUMonitor::setCPULimit()

[bugzilla-daemon at webkit.org]

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

            Bug ID: 192707
           Summary: clang-tidy: Fix unnecessary object copy in
                    CPUMonitor::setCPULimit()
           Product: WebKit
           Version: WebKit Local Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: ddkilzer at webkit.org
                CC: cdumez at apple.com

Running `clang-tidy -checks='-*,performance-*,-performance-noexcept-*' ...` on WebCore source files found this unnecessary object copy:

Source/WebCore/platform/CPUMonitor.cpp:38:52: warning: the parameter 'cpuLimit' is copied for each invocation but only used as a const reference; consider making it a const reference [performance-unnecessary-value-param]
void CPUMonitor::setCPULimit(std::optional<double> cpuLimit)
                                                   ^
                             const                &

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20181214/52bc035d/attachment.html>