[Enhancement] Allow attributes, like format_read, to have inlined code
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 35/100
Hướng nghiên cứu
Bắt đầu bằng cách theo dõi cách các thuộc tính format_read và transform được phân tích cú pháp và áp dụng, sử dụng các ví dụ DATE hiện tại và được đề xuất làm tham chiếu về hành vi. Xác định cách các biểu thức inline sẽ nhận được các trường struct và cách các hàm được đặt tên hiện có tiếp tục được hỗ trợ; được xem là hoàn tất khi cả hai dạng đều hoạt động mà không yêu cầu các hàm trợ giúp có tên duy nhất.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Currently, various attributes like format_read require a string containing a function name as the argument. A lot of the time, these functions seem to end up being a simple one-liner.
// == current ==
struct DATE
{
u16 year;
u8 month;
u8 day;
}
[[
format_read("read_DATE"),
transform("transform_DATE"),
...
]];
fn read_DATE(auto v)
{
return std::format("{}-{}-{}", v.year, v.month, v.day);
};
fn transform_DATE(auto v)
{
return (v.year * 10000) + (v.month * 100) + v.day;
};
This adds a decent amount of boilerplate overhead, making the underlying data format specification a little more difficult to read (due to tool integration 'noise'). Although the function could be moved to a different part of the pattern file, to somewhat separate data format from integration, the function would then no longer be co-located with the data it's operating on.
I feel things could be cleaner / clearer, if the contents of that function were optionally able to be inlined. This also avoids needing the pattern creator to make sure the function is uniquely named, as well as having to access the data through a (named) parameter.
As a first pass suggestion, inlining simple one-liners could look like:
// == proposed ==
struct DATE
{
u16 year;
u8 month;
u8 day;
}
[[
format_read(std::format("{}-{}-{}", year, month, day)),
transform((year * 10000) + (month * 100) + day),
...
]];
- Ngôn ngữ chính
- C++
- Star
- 275
- Fork
- 75
- Merge trung bình
- 1 ngày 22 giờ
- Pull request đã merge (30 ngày)
- 10
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của WerWolv/PatternLanguage
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
WerWolv/PatternLanguage#236 · 12 bình luận ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
WerWolv/PatternLanguage#226 · 3 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
WerWolv/PatternLanguage#221 · 4 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
WerWolv/PatternLanguage#219 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
WerWolv/PatternLanguage#218 ·
Tất cả issue của WerWolv/PatternLanguage
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
ros2/message_filters#338 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
subsurface/subsurface#4984 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
flutter-webrtc/flutter-webrtc#2206 ·
-
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
google-ai-edge/LiteRT-LM#3739 ·