sitsec
commented
7 years ago Hello Martyn,
Thank you very much for your changes. I somehow missed your second pull request in September and did not merged it until now. I had several merge conflicts to resolve, but everything seems to work.
Best regards, Martin
Added a lot more tests, and moved some functionality around.
I have a branch (mostly local at the moment) for spawning docker containers to batch process in a cloud environment, many of the added functions on this pull request allow the multi-processor features to work (for instance output files, splitting polynomial search space, non-verbose outputs). The bf_crc class should also work independently.
I am also building a more efficient CRC class to improve performance (there is a branch, but again it is mostly local until it at least passes the CRC test). By copying data into the CRC class and changing the class's properties there are less data operations and bytewise operations can be used to speed up calculations - important when calculating CRC's for very long data sets.
Also added in the ability to test known CRC's from the database at http://reveng.sourceforge.net. Speeds things up no end, but do need to find a way to inform the user when not in verbose mode that something was found right at the start.