AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

Add .none(false) for arguments of python exposed functions expecting strings

Aperta
#1,202 8 commenti 0 reazioni 0 assegnatari Vedi su GitHub
good first issue help wanted
Lingua principale
C++
Stelle
2.1k
Fork
503
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

Several API functions are taking const char * parameters. They are often exposed to Python with:
.def("function", &Class::function, "parameter"_a)
This is working fine, when used in python, the parameter type is checked, but when used with None a segfault occurs.
To handle that .none(false) can be added to the parameter:
.def("function", &Class::function, "parameter"_a.none(false))

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Non è indicato alcun file o test. Inizia individuando i binding di esposizione da C++ a Python che contengono chiamate .def per funzioni con parametri const char*, quindi riproduci il caso None per identificare le API interessate. Il lavoro è completato quando questi parametri stringa esposti rifiutano None senza un segfault e sono presenti test di copertura per i binding interessati.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp, python
Ambito
api
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
38/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.