AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO
Proposal to remove pystring dependency
- 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ả
Our current usage of pystring falls into 2 camps
String manipulation e.g. split, startswith, endswith, splitlines, lstrip etc
Filesystem interaction pystring::os::*
I would propose that this be carried out in 3 stages:
1) Replace string functionality with code based on our own StringUtils (add functionality where needed). We already have some tests for replacing the split() behaviour for example.
2) Replace filesystem interaction by calls to std::filesystem (C++17)
3) Remove all pystring from CMake
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu bằng cách xác định các chỗ sử dụng pystring hiện có và tách phần thao tác chuỗi khỏi phần tương tác với hệ thống tệp. Xem xét StringUtils và các kiểm thử split() của nó, sau đó đánh giá các thay đổi được đề xuất cho std::filesystem và CMake. Công việc được xem là hoàn tất khi ba giai đoạn thay thế đã hoàn thành và pystring đã được loại bỏ khỏi CMake.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- cmake, cpp
- Lĩnh vực
- build-system
- Loại issue
- Tái cấu trúc
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Ít trao đổi
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 35/100