kirk-baird opened a new pull request #18: URL: https://github.com/apache/incubator-milagro-crypto-rust/pull/18
# What has been changed - Copying has been replaced with the compiler optimised `Clone` trait. - Unnecessary variables have been removed where possible. - Constructors have been improved, to take ownership of variables rather than references and copies. # Additional Comments - The speedup is about 5-10% for `bls381` signing and verifying and will likely be similar for other curves and protocols. - The compiler optimises out unnecessary memory allocations in `clone()` resulting in an optimised copy for stack allocated items (which is nearly all structs used in `amcl`). Thus overall memory consumption should equivalent or at least very comparable. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@milagro.apache.org For additional commands, e-mail: issues-h...@milagro.apache.org
