Skip to content

fix _operator._compare_digest — no-bind wrapper for class-attr use#2699

Open
fgallaire wants to merge 1 commit into
brython-dev:masterfrom
fgallaire:fix-operator-compare-digest
Open

fix _operator._compare_digest — no-bind wrapper for class-attr use#2699
fgallaire wants to merge 1 commit into
brython-dev:masterfrom
fgallaire:fix-operator-compare-digest

Conversation

@fgallaire

Copy link
Copy Markdown
Contributor
>>> import hmac
>>> class T:
...     compare_digest = hmac.compare_digest
>>> T().compare_digest(b'a', b'a')      # before — self auto-bound
TypeError: _compare_digest() takes 2 positional arguments but 3 were given
>>> T().compare_digest(b'a', b'a')      # after — no auto-bind, like CPython builtin
True

@PierreQuentel

Copy link
Copy Markdown
Contributor

Florent,

Dans le commit référencé ci-dessus j'ai plutôt changé le type de _operator._compare_digest en staticmethod. Ca a le même effet que ta proposition (pas de binding) mais avec un procédé standard plutôt qu'avec une classe spécifique. Qu'en penses-tu ?

@fgallaire

Copy link
Copy Markdown
Contributor Author

Pierre,

Ta solution est mieux, mais il faudrait un vrai builtin si on veut être type-perfect avec CPython.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants