First, good job on the merge, I don't think I saw a single refactor missed. Second, the program is very close to its final, testable, version. There is at least one issue left though.
The program works fine for a little bit, but some time into the hi-lo test it starts acting weird. One time it seemed to get stuck in an infinite loop somewhere in one of the range functions. When I re-ran it, it crashed due to using too much memory. So I checked as it was running and it is indeed using more and more memory; thus, we have a memory leak somewhere.
Memory leaks can cause very weird behavior so I bet once it is fixed everything else will work fine. I am investigating it right now with gdb and will update this if I find anything.
First, good job on the merge, I don't think I saw a single refactor missed. Second, the program is very close to its final, testable, version. There is at least one issue left though.
The program works fine for a little bit, but some time into the hi-lo test it starts acting weird. One time it seemed to get stuck in an infinite loop somewhere in one of the range functions. When I re-ran it, it crashed due to using too much memory. So I checked as it was running and it is indeed using more and more memory; thus, we have a memory leak somewhere.
Memory leaks can cause very weird behavior so I bet once it is fixed everything else will work fine. I am investigating it right now with gdb and will update this if I find anything.