Put back `-Wsign-conversion` for gcc in CompilerWarnings.cmake
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- Dưới một giờ
- Mức phù hợp với người mới
- 85/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- cmake, cpp
- Lĩnh vực
- build-system
Hướng nghiên cứu
Bắt đầu trong CompilerWarnings.cmake và kiểm tra cách các cờ cảnh báo được chọn cho gcc và C++. Tái hiện ví dụ C++23 được cung cấp bằng gcc với -Wconversion và -Wsign-conversion. Được xem là hoàn thành khi các bản build C++ bằng gcc hoạt động trở lại với -Wsign-conversion được bật, đồng thời vẫn giữ nguyên hành vi đã nêu cho C và Clang.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Issue #76 / PR #149 removed the -Wsign-conversion flag in CompilerWarnings.cmake. This may be OK for Clang and gcc using C, but it is not OK for gcc using C++.
The gcc documentation says:
In C, this option is enabled also by -Wconversion.
So NOT for C++.
See this demo for an example: https://godbolt.org/z/8YMbWr3Wx
Using gcc 15.2, compiling the code with -O3 -std=c++23 -Werror -Wconversion will work, but using -O3 -std=c++23 -Werror -Wconversion -Wsign-conversion will not.
- Ngôn ngữ chính
- CMake
- Star
- 1.8k
- Fork
- 205
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Không 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 cpp-best-practices/cmake_template
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
-
Enable sanitizers globallyĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
cpp-best-practices/cmake_template#140 · 2 bình luận ·
-
From-scratch clone/build fails on `src/ftxui_sample/CMakeFiles/intro.dir/main.cpp.obj`Có thể đã có người làm @leftibot đã nhận 195 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
cpp-best-practices/cmake_template#139 · 2 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
cpp-best-practices/cmake_template#91 · 2 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 38/100
cpp-best-practices/cmake_template#79 · 4 bình luận · 1 reaction ·
Tất cả issue của cpp-best-practices/cmake_template
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
immortalwrt/immortalwrt#2522 ·
-
Font inlining cache is not created or reusedCó thể đã có người làm @nagahara-naoki đã nhận hôm nay. Đang mởarea: @angular/build
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
angular/angular-cli#34719 ·
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 72/100
osaurus-ai/osaurus#3422 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ 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
foundation/foundation-emails#2127 ·