ada-url / ada-url/ada

Missing test coverage for encode_json for values smaller than 0x1f

Đang mở
#516 3 bình luận 0 reaction 1 người được giao Được @Uzlopak nhận Xem trên GitHub
Ngôn ngữ chính
C++
Star
1.8k
Fork
142
Merge trung bình
19 giờ 11 phút
Pull request đã merge (30 ngày)
24

Mô tả

### What is the problem this feature will solve?

increases test coverage

### What is the feature you are proposing to solve the problem?

I was investigating ada. I saw that you do `2 * c` and was wondering if a bitshift makes a difference `c << 1` but then I realized that if replace with just returning '0', then `ctest --output-on-failure --test-dir build` will pass all the tests. So there is missing test coverage.

I guess that the compiler will create the same bytecode for 2 * c and c << 1. But it would make sense to have test coverage.

### What alternatives have you considered?

_No response_

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

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.