Hey Josselin,
Thanks for finding this bug! I have one concern about your patches:
Josselin Poiret writes:
> The second patch removes renumber_file_descriptor, as it is no longer
> used.
One thing that ‘renumber_file_descriptor’ does that we seem to be losing
here is error checking. To my eye,
Hello Timothy,
Timothy Sample writes:
> One thing that ‘renumber_file_descriptor’ does that we seem to be losing
> here is error checking. To my eye, the old code will try and warn the
> user if they run out of file descriptors, but the new code will not.
I may have been too hasty on that fron
