AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

NIT PICKS ociocheck: Empty namespace for Interop ID? Warning message too strong when a valid Interop ID is used with a namespace ?

オープン
#2,301 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
C++
スター
2.1k
フォーク
503
PR マージ指標
30日以内にマージされた PR はありません

説明

Whilst investigating the unused variable [here](https://github.com/AcademySoftwareFoundation/OpenColorIO/blob/main/src/apps/ociocheck/main.cpp#L90) I had some questions:

1. I wondered what it should do if the namespace is empty? (If the string begins with a ':')

2. I note that ociocheck will warn if a namespaced ID has the same spelling as an official ID - all good but do we feel the word 'must' is a little strong here? I know we are not a "Standard" body, but I'm imagining a world where a pre-official ID is being migrated to an official one and a user of ociocheck now thinks the file is in error. We'd obviously want users to migrate to an official one so maybe switch to 'should' ?

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

src/apps/ociocheck/main.cpp の 90 行目付近から始め、Interop IDs と namespaces がどのようにチェックされるかを追跡します。':' で始まる ID に対する期待される動作と、namespace 付きの ID が公式の ID と一致した場合に警告に持たせる意図した強度を確認します。動作と文言が確定し、関連する ociocheck のチェックでカバーされれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
cpp
領域
cli
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
静か
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。