There are schemes, like SABER (#192) that have more than one principal
submitter. Consistency warrants that we turn it into a list for all
schemes and don't do something with allowing either a str or a list:
that would just be very annoying to parse.

Closes #194

* fixes dynamic memory allocation test. previously a function called freeze() would trigger it

* this adds DilithiumII. Preprocessor conditionals still need to be removed

* fix ms Makefile

* fix MS compiler warnings

* clean-up

* remove preprocessor conditionals

* add dilithium3

* add dilithium4

* add duplicate consistency checks

* SHA2 state constants in common

* clean up symmetric.h

* Port SPHINCS+-SHA256 to sha256ctx struct

* Implement ctx struct for fips202

* Port Kyber{512,768,1024} to fips202 ctx struct

* Port NewHope to fips202 structs

* Port SPHINCS+-SHAKE256 to fips202 ctx structs

* Use opaque fips202 structs in MQDSS

* port dilithium to use fips202 ctx structs

* include -Wredundant-decls

* remove comment; format NTT constants

* reduce casts in power2round

This reverts commit 00639575b0.

Windows already complains about this in CI, but this will
let us catch these issues on Linux as well.

Windows already complains about this in CI, but this will
let us catch these issues on Linux as well.

Thanks to Stefan Kölbl

* fix prototypes for sphincs and static functions in aes.c

* fix missing prototypes in all frodo variants

* fix missing prototypes in kyber

* remove const from non-pointer arguments in Frodo

* add missing prototypes to requirements in README

* Minor changes

* Gracefully degrade if clang-tidy segfaults

* Typo on returncode

This does not implement the detached signature API, and will
change significantly for NIST's 2nd round anyway.

This is actually different from signature functests,
which only tests mlen=32. Can be fixed later as that does
not impact the testvectors.

Does this slightly uglily, but it should work.