|
|
|
|
|
by jemfinch
5332 days ago
|
|
Clearly you didn't actually read the source file. I graduated in CS. I know B+ trees. I also know that an 85-line, 7-argument method in a 1988-line file shouldn't depend on a global variable ("guessIncreasing") modified from several other, unrelated functions. I know that in bt_insert, which (apparently) assigns to "guessIncreasing" and then resets it to false just prior to exit, should be using an RAII class to do so instead of trying to catch every exit path, especially in a codebase that uses exceptions. This code is amateur hour. |
|