Provide custom error messages for overflows in PyLong_AsLong/PyLong_AsInt()
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 77.2k
- Fork
- 35.9k
- Metriche di merge delle PR
- Metriche PR in attesa
Descrizione
Feature or enhancement
Proposal:
Both functions have old TODO comments:
https://github.com/python/cpython/blob/453d886f8592d2f4346d5621b1e4ff31c24338d5/Objects/longobject.c#L671-L676
https://github.com/python/cpython/blob/453d886f8592d2f4346d5621b1e4ff31c24338d5/Objects/longobject.c#L688-L694
Lets either remove these marks or provide different error messages, for example for longs:
if (overflow > 0) {
PyErr_Format(PyExc_OverflowError,
"Python int too large to convert to C long, value > %ld",
LONG_MAX);
return -1;
}
else if (overflow < 0) {
PyErr_Format(PyExc_OverflowError,
"Python int too large to convert to C long, value < %ld",
LONG_MIN);
return -1;
}
Has this already been discussed elsewhere?
This is a minor feature, which does not need previous discussion elsewhere
Links to previous discussion of this feature:
No response
Linked PRs
- gh-141585
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia in Objects/longobject.c, nei commenti TODO collegati nell’issue, e esamina la PR gh-141585 collegata per il lavoro già in corso. Il lavoro è completato quando questi TODO vengono rimossi oppure vengono forniti messaggi di overflow distinti per PyLong_AsLong() e PyLong_AsInt().
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- c, python
- Ambito
- api
- Tipo di issue
- Funzionalità
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 25/100