AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
Add .none(false) for arguments of python exposed functions expecting strings
オープン
good first issue
help wanted
- 主要言語
- C++
- スター
- 2.1k
- フォーク
- 503
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
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))
コントリビューションガイド
調査の方向性
ファイルもテストも指定されていません。まず、const char* パラメータを持つ関数に対する .def 呼び出しを含む C++ の Python 公開用 bindings を探し、次に None のケースを再現して影響を受ける API を特定します。これらの公開された文字列パラメータが segfault なしに None を拒否し、影響を受ける bindings のカバレッジがある状態になれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- cpp, python
- 領域
- api
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 38/100