[New-bugs-announce] [issue33101] Possible name inversion in heapq implementation

Yomguithereal report at bugs.python.org
Mon Mar 19 09:19:15 EDT 2018


New submission from Yomguithereal <guillaumeplique at gmail.com>:

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.

----------
components: Library (Lib)
messages: 314093
nosy: Yomguithereal
priority: normal
severity: normal
status: open
title: Possible name inversion in heapq implementation
type: enhancement
versions: Python 3.8

_______________________________________
Python tracker <report at bugs.python.org>
<https://bugs.python.org/issue33101>
_______________________________________


More information about the New-bugs-announce mailing list