[C++] Remove an unnecessary null pointer check
Maintainer thường phản hồi trong vòng 1 ngà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
- 78/100
Hướng nghiên cứu
Thay đổi nằm ở destructor của ArrowLog trong cpp/src/arrow/util/logging.cc, khoảng dòng 258-262, nơi một kiểm tra null đang bao quanh delete. Hãy đọc destructor đó và xác nhận con trỏ chỉ được giải phóng ở đó; sau đó xóa kiểm tra thừa, vì delete trên con trỏ null không có tác dụng gì. Hoàn thành khi đã bỏ điều kiện bảo vệ, tệp vẫn biên dịch được, và bản build cùng các test C++ hiện có đều pass.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Describe the enhancement requested
An extra null pointer check is not needed in the destructor for the class “ArrowLog”.
Component(s)
C++
- Ngôn ngữ chính
- C++
- Star
- 17.2k
- Fork
- 4.4k
- Merge trung bình
- 3 ngày 9 giờ
- Pull request đã merge (30 ngày)
- 69
Chuẩn bị môi trường
- Có Dockerfile hoặc tệp Docker Compose
- Có mẫu pull request
- Đọc 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 apache/arrow
-
[R][Parquet] ParquetReaderProperties does not expose page_checksum_verification, so a corrupt page CRC reads silentlyCó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mởComponent: R good-first-issue Type: enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
apache/arrow#53138 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[C++][Python] IPC reader rejects a DictionaryEncoding without indexType, which the format allowsCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mởComponent: C++ Component: Python Type: bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[R] Silent loss of metadata integrity for date-time and numeric attributes in write_parquet()/read_parquet()Có thể làm lại được @james-finn-travers đã nhận 13 ngày trước và không có pull request nào đang mở. Đang mởComponent: R Type: bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
apache/arrow#51695 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[C++][Parquet] Plaintext-footer files written with AES_GCM_CTR_V1 record AES_GCM_V1 as the encryption algorithm and cannot be readCó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mởType: bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[R] Expose ignore_extra_columns and pad_short_rows CSV parse optionsCó thể đã có người làm Có pull request liên kết đang mở hoặc đã được merge. Đang mởComponent: R good-first-issue Type: enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
apache/arrow#51659 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Issue tương tự
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 66/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
NVIDIA-RTX/NRI#277 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
os:Linux
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 3 ngày