commit 1859be8ad62e98c644db657faebb3742dd20c24e
author Avi Drissman <avi@chromium.org> Wed Jan 26 19:38:14 2022
committer Copybara-Service <copybara-worker@google.com> Wed Jan 26 19:49:18 2022
tree 9470dc23527957150a1151b2d99f773dabf64369
parent 842b1511e2b531114c53aeae8e2a57238916a2ae

Clean up uses of (void)

This migrates uses of (void) to [[maybe_unused]] or std::ignore as
appropriate.

This CL was uploaded by git cl split.

R=agl@chromium.org

Bug: 1289456
Change-Id: I9c1836f03907fedfbddf452eac2214cfb64fc63d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3415896
Auto-Submit: Avi Drissman <avi@chromium.org>
Reviewed-by: Adam Langley <agl@chromium.org>
Commit-Queue: Adam Langley <agl@chromium.org>
Cr-Commit-Position: refs/heads/main@{#963642}
NOKEYCHECK=True
GitOrigin-RevId: dd964ebc6dfe14f3ac297198ad0f0b267b75d47f

unexportable_key_win.cc

diff --git a/unexportable_key_win.cc b/unexportable_key_win.cc
index 6ca4dcb..c13f43d 100644
--- a/unexportable_key_win.cc
+++ b/unexportable_key_win.cc
@@ -7,6 +7,7 @@
 #include <ncrypt.h>
 
 #include <string>
+#include <tuple>
 #include <vector>
 
 #include "base/logging.h"
@@ -350,7 +351,7 @@
             MS_PLATFORM_CRYPTO_PROVIDER, /*flags=*/0))) {
       // If the operation failed then |provider| doesn't have a valid handle in
       // it and we shouldn't try to free it.
-      (void)provider.release();
+      std::ignore = provider.release();
       return absl::nullopt;
     }
 
@@ -369,7 +370,7 @@
             MS_PLATFORM_CRYPTO_PROVIDER, /*flags=*/0))) {
       // If the operation failed when |provider| doesn't have a valid handle in
       // it and we shouldn't try to free it.
-      (void)provider.release();
+      std::ignore = provider.release();
       return nullptr;
     }
 
@@ -387,7 +388,7 @@
             /*dwLegacyKeySpec=*/0, /*dwFlags=*/0))) {
       // If the operation failed then |key| doesn't have a valid handle in it
       // and we shouldn't try and free it.
-      (void)key.release();
+      std::ignore = key.release();
       return nullptr;
     }
 
@@ -435,7 +436,7 @@
             MS_PLATFORM_CRYPTO_PROVIDER, /*flags=*/0))) {
       // If the operation failed when |provider| doesn't have a valid handle in
       // it and we shouldn't try to free it.
-      (void)provider.release();
+      std::ignore = provider.release();
       return nullptr;
     }
 
@@ -447,7 +448,7 @@
             /*dwFlags=*/NCRYPT_SILENT_FLAG))) {
       // If the operation failed then |key| doesn't have a valid handle in it
       // and we shouldn't try and free it.
-      (void)key.release();
+      std::ignore = key.release();
       return nullptr;
     }