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 ?
- Ngôn ngữ chính
- C++
- Star
- 2.1k
- Fork
- 503
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
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' ?
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu với src/apps/ociocheck/main.cpp quanh dòng 90 và lần theo cách Interop IDs và namespaces được kiểm tra. Xác nhận hành vi dự kiến đối với một ID bắt đầu bằng ':' và mức độ cảnh báo dự kiến khi một ID có namespace khớp với một ID chính thức. Được coi là hoàn tất khi hành vi và cách diễn đạt đã được thống nhất và được các kiểm tra ociocheck liên quan bao phủ.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- cpp
- Lĩnh vực
- cli
- Loại issue
- Lỗi
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức độ hoạt động
- Ít trao đổi
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 45/100