From 58aba49ca2e8a6067e9bbed2c84dbef52d525575 Mon Sep 17 00:00:00 2001
From: Thom Wiggers <thom@thomwiggers.nl>
Date: Wed, 27 Feb 2019 17:08:59 +0100
Subject: [PATCH] take out assignment in conditional because windows is not
 happy about it

---
 test/crypto_sign/functest.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/test/crypto_sign/functest.c b/test/crypto_sign/functest.c
index 014b0542..41ab091a 100644
--- a/test/crypto_sign/functest.c
+++ b/test/crypto_sign/functest.c
@@ -114,9 +114,9 @@ static int test_wrong_pk(void) {
 
         // By relying on m == sm we prevent having to allocate CRYPTO_BYTES
         // twice
-        if (!(returncode = crypto_sign_open(sm, &mlen, sm, smlen, pk2))) {
-            printf("ERROR Signature did verify correctly under wrong public "
-                   "key!\n");
+        returncode = crypto_sign_open(sm, &mlen, sm, smlen, pk2);
+        if (!returncode) {
+            printf("ERROR Signature did verify correctly under wrong public key!\n");
             if (returncode > 0) {
                 puts("ERROR return code should be < 0");
             }