Dead code in _descend_and_update()

[cjdoucette]

The part of _descend_and_update starting from this line appears to be dead code:

https://github.com/pixos/poptrie/blob/master/poptrie.c#L929

Since this code returns immediately, the rest of this if statement is not executed.

I think this could be a bug that happens when you try to add prefixes of greater than 24 bits from this line:

https://github.com/pixos/poptrie/blob/master/poptrie.c#L880

[drpnd]

Thanks. I'll take a look at it and fix the issue.

[cjdoucette]

Great, thanks. I think the fix is just to delete that line of code. If you want me to submit a pull request for it, I'd be happy to.

[drpnd]

Thanks for your help. I don't think deleting the line fixes it because the test failed when I've removed the line. Let me review this version of my code and the test. Anyway thanks for your code review again. I'll update this ticket as soon as I find the cause of the bug.