BitmapByteQRCode performance optimization#566
Conversation
Removed loops where possible and set fixed size objects
|
@Shane32 May I ask you for a review? (Please be critical with the benchmark project. I tried to exclude the QRCode data generation, as it is ramp-up stuff, from the benchmarks, but I'm not sure, if I did it right.) |
Shane32
left a comment
There was a problem hiding this comment.
Looks great! Benchmarks look great! I've just jotted down some notes for additional improvements that could be made, whether that be within this PR or another one.
Hi Shane, thanks for your review. I implemented and tested all of your suggestions. Thanks for the great input! :-) |
Summary
I tried to optimize the performance of BitmapByteQRCode by removing loops and pre-calculating data where possible. Also fixed object size/capacity to keep memory footprint low.
Performance details
Before this PR
After/with changes from this PR
After implementing changes suggested by @Shane32 via PR review
Test plan
To ensure that the optimizations work out and break anything I added test cases and benchmarks to the project.