Issue 33101: Possible name inversion in heapq implementation (original) (raw)

Hello Python team,

I might be hallucinating but I am under the impression that the heapq module uses reverse naming.

What I mean is that it seems to me that the _siftup method should actually be named _siftdown and, the other way around, _siftdown should be named _siftup.

This has absolutely no practical consequence since the module works as it should but I am a bit confused since I don't know if the module got naming wrong or if it followed another canonical naming I don't know about.

I am willing to open a PR to fix this if the named reverasl was to be confirmed.

Good day to you.