Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

error: conversion from 'double' to 'float' may change value [-Werror=float-conversion]

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#963 5 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 12 ngày

@coveyjorjet đang làm issue này rồi.

Từ ngày 1/8/2026.

  • #1061 của @coveyjorjet — đang mở

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
62/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
c, cmake
Lĩnh vực
build-system

Hướng nghiên cứu

Tái hiện bản build với cảnh báo nghiêm ngặt bằng CMake FetchContent và kiểm tra cJSON.c quanh dòng 607 trong print_number, vị trí mà báo cáo xác định là nơi xảy ra lỗi. So sánh các chẩn đoán của trình biên dịch với các kiểm tra toán học xung quanh và chạy bản build cùng các bài kiểm thử của cJSON; được coi là hoàn tất khi thư viện build thành công với các cờ cảnh báo được báo cáo mà không vô hiệu hóa lỗi.

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

Mô tả

Description
I encountered a compilation error when integrating cJSON into my C++ project via CMake's FetchContent. The build fails at line 607 in cjson.c with the following error:

error: conversion from 'double' to 'float' may change value [-Werror=float-conversion]

Environment

  • cJSON Version: v1.7.19
  • Compiler: gcc version 15.1.0 (x86_64-win32-seh-rev0, Built by MinGW-Builds project)
  • Build System: cmake version 4.0.2 (CMake; JetBrains IDE bundle; build 1)
  • OS: Windows 11

Steps to Reproduce

  1. Configure CMake to include cJSON via FetchContent: 
    include(FetchContent)  
FetchContent_Declare(  
  cjson  
  # actualy I downloaded form release and use `SOURSE_DIR` to include it
  GIT_REPOSITORY https://github.com/DaveGamble/cJSON.git  
  GIT_TAG v1.7.19
)  
FetchContent_MakeAvailable(cjson)  
target_link_libraries(my_target cjson)
  2. Enable strict warnings (e.g., -Werror=float-conversion).
  3. Build the project.

Error Log

"D:\software\CLion 2024.3.5\bin\cmake\win\x64\bin\cmake.exe" --build D:\projects\prj-terminal\cmake-build-debug --target prj_terminal -j 22
[1/3] Building C object _deps/cjson-build/CMakeFiles/cjson.dir/cJSON.c.obj
FAILED: _deps/cjson-build/CMakeFiles/cjson.dir/cJSON.c.obj 
D:\software\mingw64\bin\cc.exe -DCJSON_API_VISIBILITY -DCJSON_EXPORT_SYMBOLS -DENABLE_LOCALES -Dcjson_EXPORTS  -std=c89 -pedantic -Wall -Wextra -Werror -Wstrict-prototypes -Wwrite-strings -Wshadow -Winit-self -Wcast-align -Wformat=2 -Wmissing-prototypes -Wstrict-overflow=2 -Wcast-qual -Wundef -Wswitch-default -Wconversion -Wc++-compat -fstack-protector-strong -Wdouble-promotion -Wparentheses -Wformat-overflow -Wunused-macros -Wmissing-variable-declarations -Wswitch-enum -fvisibility=hidden -g -std=gnu11 -fdiagnostics-color=always -MD -MT _deps/cjson-build/CMakeFiles/cjson.dir/cJSON.c.obj -MF _deps\cjson-build\CMakeFiles\cjson.dir\cJSON.c.obj.d -o _deps/cjson-build/CMakeFiles/cjson.dir/cJSON.c.obj -c D:/projects/prj-terminal/third_party/cJSON-1.7.19/cJSON.c
In file included from D:/projects/prj-terminal/third_party/cJSON-1.7.19/cJSON.c:42:
D:/projects/prj-terminal/third_party/cJSON-1.7.19/cJSON.c: In function 'print_number':
D:/projects/prj-terminal/third_party/cJSON-1.7.19/cJSON.c:607:15: error: conversion from 'double' to 'float' may change value [-Werror=float-conversion]
  607 |     if (isnan(d) || isinf(d))
      |               ^
D:/projects/prj-terminal/third_party/cJSON-1.7.19/cJSON.c:607:27: error: conversion from 'double' to 'float' may change value [-Werror=float-conversion]
  607 |     if (isnan(d) || isinf(d))
      |                           ^
cc1.exe: all warnings being treated as errors
ninja: build stopped: subcommand failed.

Proposed Solution
The error occurs due to an implicit double-to-float conversion.
I asked ai, and it told me that to resolve this, consider:

  1. Explicitly casting the result to float (if precision loss is acceptable).
  2. Using -Wno-float-conversion for cJSON (not ideal for strict compliance).
  3. Refactoring the calculation to avoid precision loss warnings.

Additional Context

  • This issue arises when compiling with strict warning flags (e.g., -Werror).
  • The problematic code is in the parse_hex4 function (line 607 in latest master).

I wander if there is a way to solve this without remove the flag Werror=float-conversion

I'm still learning Cpp and Cmake. So if you would be willing to help me, I would greatly appreciate it. Thank you for reading my issue. Let me know if you need more details!

Ngôn ngữ chính
C
Star
13k
Fork
3.5k
Không có pull request nào được merge trong 30 ngày

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của DaveGamble/cJSON

Tất cả issue của DaveGamble/cJSON

Issue tương tự

Thêm issue về C

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.