From 0d6c86bae643d73cf1c7cb79a045d469656c2229 Mon Sep 17 00:00:00 2001 From: Jeffrey Walton Date: Sun, 14 Oct 2018 00:30:23 -0400 Subject: [PATCH] Update comments --- strciphr.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/strciphr.cpp b/strciphr.cpp index b84f2c60..194f6f1d 100644 --- a/strciphr.cpp +++ b/strciphr.cpp @@ -199,7 +199,7 @@ void CFB_CipherTemplate::ProcessData(byte *outString, const byte *inString // Using 'outString' for both input and output leads to incorrect results. // // Benchmarking on Cortex-A7 and Cortex-A9 indicates removing the block - // below does not have a material effect on performance. + // below costs about 9 cpb for CFB mode on ARM. // // Also see https://github.com/weidai11/cryptopp/issues/683. // @@ -235,8 +235,7 @@ void CFB_CipherTemplate::ProcessData(byte *outString, const byte *inString // One workaround is a distinct and aligned temporary buffer. It [mostly] works // as expected but requires an extra allocation (casts not shown): // - // std::string temp(length); - // std::memcpy(&temp[0], inString, length); + // std::string temp(inString, length); // policy.Iterate(outString, &temp[0], cipherDir, length / bytesPerIteration); // memcpy(outString, inString, length);