Closed martynp closed 8 years ago
Hello,
First of all, thank you very much for your effort. I going to review your changes. It may take some time - I am quite busy right now. And it will definitely take some time to come back to the review.
Best regards,
Martin
Thanks for the merge - I am on holiday at the moment so will be continuing on my fork for a while. I want to add in the rest of the CRC test cases and a few more brute force test cases.
Other bits I am looking into is putting in a set of tests for known common polynomials before starting the main search loop, saving the results to a comma separated file and looking into any optimisations or accelerations which can be made for the CRC algorithm itself.
I will put in a pull request when significant progress has been made!
I found this code quite high on Google search results and have spent some time making improvements on a fork. I have a lot more work to do until I consider it finished, but thought if you could pull now I might be able to work on the main repository as a contributor.