Add (void) to some macros to satisfy compiler.
More modern versions of GCC (at least with aarch64) are warning about an
unused value in these locations. It's incorrect, but I guess that the
macro is confusing it.
Using a (void) tag is a little ugly but solves the problem.
Change-Id: If6ba5083ab6e501c81e7743ae1ed99a89565e57c
Reviewed-on: https://boringssl-review.googlesource.com/2810
Reviewed-by: Adam Langley <agl@google.com>
diff --git a/ssl/ssl_sess.c b/ssl/ssl_sess.c
index 117e53c..367b0f4 100644
--- a/ssl/ssl_sess.c
+++ b/ssl/ssl_sess.c
@@ -764,7 +764,7 @@
/* timeout */
/* The reason we don't call SSL_CTX_remove_session() is to
* save on locking overhead */
- lh_SSL_SESSION_delete(param->cache, sess);
+ (void) lh_SSL_SESSION_delete(param->cache, sess);
SSL_SESSION_list_remove(param->ctx, sess);
sess->not_resumable = 1;
if (param->ctx->remove_session_cb != NULL) {
