commit c8512110f3c41e79ae6aaca496fe28ffb288f1e7
author Ryan Prichard <rprichard@google.com> Thu Jul 20 20:52:45 2023 -0700
committer Ryan Prichard <rprichard@google.com> Tue Jul 25 21:35:18 2023 -0700
tree ce4d15491202cedd0fcb44a8e15ce001d7302664
parent 39a9427c02c1bb30434e532f0053d436aedf85b2

RESTRICT AUTOMERGE: Explicitly ignore the result of std::async

The newer libc++ marks std::async with [[nodiscard]] in C++20 mode.

Bug: b/175635923
Test: treehugger
Change-Id: Ia222778697ab4241f25e002073b3514c7e106007

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 8c46515..64348a5 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -6755,7 +6755,7 @@
         if (captureListener) {
             // TODO: The future returned by std::async blocks the main thread. Return a chain of
             // futures to the Binder thread instead.
-            std::async([=]() mutable {
+            (void)std::async([=]() mutable {
                 ATRACE_NAME("captureListener is nonnull!");
                 auto fenceResult = renderFuture.get();
                 // TODO(b/232535621): Change ScreenCaptureResults to store a FenceResult.