[riot-notifications] [RIOT-OS/RIOT] pkg/libcose: update crypto support (#11996)

Koen Zandberg notifications at github.com
Fri Sep 13 09:44:55 CEST 2019


bergzand commented on this pull request.



>  /* COSE structs */
 static cose_sign_enc_t sign;
 static cose_sign_dec_t verify;
 static cose_signature_t signature1, signature2;
-static cose_key_t signer1, signer2, symm;
+static cose_key_t signer1, signer2;
+#if defined(MODULE_LIBCOSE_CRYPT_HACL) || defined(MODULE_LIBCOSE_CRYPT_MONOCYPHER)

Can I have a bit of whitespace before the `#if` and after `#endif`? Feel free to squash immediately

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11996#pullrequestreview-287856913
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190913/0b70aa8e/attachment.htm>


More information about the notifications mailing list