rapidcsv is not that rapid! about rapidcsv HOT 3 CLOSED

Thanks for this, it is great that you are willing to make improvements!

from rapidcsv.

Thank you, I'll try add some performance test(s) to this project and check if your suggestion applies across platforms, and change the code accordingly.

Generally speaking on performance there's a trade-off between the "simplicity" of the API vs. raw throughput, and rapidcsv (despite its name) focuses on ease-of-use (and enabling rapid development). So I'm pretty sure there are other CSV libraries which are faster out there (but leaving more work to the application code).

Anyway, you seem to have found a good low-level improvement (not affecting API usability) which I'm happy to include if I can confirm the results on my systems. I would have thought a decent compiler would produce similar code for those two lines, but that assumption seem to be wrong.

from rapidcsv.

Thanks @fordp2002 - I can confirm the suggested improvement applies for all three major platforms (Linux, Mac, Windows). Measurement data for ptest002.cpp (which essentially just loads a CSV file) before and after fix are provided below. Will proceed to merge the fix.

Linux: before 53.4 ms --> after 37.9 ms
Mac: before 72.4 ms --> after 30.2 ms
Windows: before 33.5 ms --> after 10.7 ms

from rapidcsv.