Code consisitency/correctness around pointers

Đang mở

#96 mở ngày 27/6/2018 5 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó: 1/5
Thời gian dự kiến: Dưới một giờ
Mức phù hợp với người mới: 35/100

Loại issue: Tái cấu trúc
Độ rõ ràng: Đặc tả rõ ràng
Mức độ hoạt động: Đình trệ
Công nghệ: cpp
Lĩnh vực: tooling

Hướng nghiên cứu

Bắt đầu trong src/vtquery.cpp tại các dòng 26-29, nơi các khai báo con trỏ được tham chiếu. So sánh các khai báo đó với kiểu const-reference xung quanh và coi công việc là hoàn tất khi cú pháp con trỏ nhất quán với phần còn lại của tệp.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

In regards to https://github.com/mapbox/vtquery/blob/5ffe8d7dac2851031b873ba088a27f1ae5add78a/src/vtquery.cpp#L26-L29

@alliecrevier ask:

[nitpick] Since T const& instead of const T& is the style used throughout this code file, it might make sense to use this style for pointers as well. So instead of const char* and static const char* you would use char const* and static char const* respectively. It looks weird to me but it's consistent!

Ngôn ngữ chính: JavaScript
Star: 45
Fork: 14
Chỉ số merge pull request: Không có pull request nào được merge trong 30 ngày

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

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 mapbox/vtquery

Remove mason-js dependency Đang mở

Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100

mapbox/vtquery#127 · 1 reaction · Mở ngày 19/5/2022
More efficient response format Đang mở

Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100

mapbox/vtquery#102 · 1 bình luận · Mở ngày 24/10/2018
when to use std::int32_t and std::int64_t Đang mở

Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100

mapbox/vtquery#98 · 2 bình luận · Mở ngày 27/6/2018
union geometries Đang mở

Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100

mapbox/vtquery#66 · 1 bình luận · Mở ngày 1/3/2018
dedupe is too strong Đang mở

Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100

mapbox/vtquery#65 · 3 bình luận · 1 reaction · Mở ngày 1/3/2018

Tất cả issue của mapbox/vtquery

Issue tương tự

[Bug]: Linux launcher entry doesn't claim the window's app id — dock/taskbar shows a placeholder icon Đang mở

area/install-update comp/cli comp/desktop P3 sweeper:risk-compatibility type/bug

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100

NousResearch/hermes-agent#122386 · 1 bình luận · Mở ngày 25/9/2026
docs: manual client injection examples reference removed exports Đang mở

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100

vitejs/devtools#589 · 1 bình luận · Mở ngày 25/9/2026
🔒 [IBM OSPO Security Notification] — IBM/node-sdk-core Đang mở

security

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100

IBM/node-sdk-core#373 · Mở ngày 25/9/2026
docs(web): deprecate `moveToElement()` Đang mở

docs web/

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100

keymanapp/keyman#16660 · Mở ngày 25/9/2026
Missing hu metadata: javascript (exercise:legacy-grains:name) Đang mở

Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100

exercism/website#9894 · Mở ngày 25/9/2026

Thêm issue về JavaScript