[webkit-changes] [WebKit/WebKit] cf7474: Include <chrono> for chrono_literals.

Stephan T. Lavavej noreply at github.com
Sun Dec 1 20:47:14 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: cf74749238ac857b78e76e2c01a2ed050b0a8fa3
      https://github.com/WebKit/WebKit/commit/cf74749238ac857b78e76e2c01a2ed050b0a8fa3
  Author: Stephan T. Lavavej <stl at microsoft.com>
  Date:   2024-12-01 (Sun, 01 Dec 2024)

  Changed paths:
    M Tools/TestWebKitAPI/Tests/WTF/WorkQueue.cpp

  Log Message:
  -----------
  Include <chrono> for chrono_literals.

Reviewed by Fujii Hironori.

The C++ Standard requires the <chrono> header to be included before
using the user-defined literals in the chrono_literals namespace.
WorkQueue.cpp was incorrectly assuming that including <thread>
would drag in these chrono UDLs, but this assumption will be broken
by an upcoming update of Microsoft Visual C++ (expected to ship in
VS 2022 17.13). Including <chrono> is portable and backwards-compatible.

No tests added - this is purely a compile-time fix, and if the code
builds, it's good.

* WebKit/Tools/TestWebKitAPI/Tests/WTF/WorkQueue.cpp: Include <chrono>.

Canonical link: https://commits.webkit.org/287203@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list