diffutils icon indicating copy to clipboard operation
diffutils copied to clipboard

Fix compilation for 32-bit targets

Open LunarEclipse363 opened this issue 2 months ago • 1 comments

This change makes the code compile for 32-bit targets (in my case armv7-unknown-linux-musleabihf, targeting a 2012 kindle non-touch e-book reader).

The failure without it is that some of the string literals exceed the size limit of a u32.

LunarEclipse363 avatar Nov 24 '25 23:11 LunarEclipse363

Codecov Report

:x: Patch coverage is 88.88889% with 1 line in your changes missing coverage. Please review. :white_check_mark: Project coverage is 86.25%. Comparing base (83f6d2d) to head (ced30b0).

Files with missing lines Patch % Lines
src/cmp.rs 88.88% 1 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #146      +/-   ##
==========================================
- Coverage   86.30%   86.25%   -0.05%     
==========================================
  Files          13       13              
  Lines        6227     6206      -21     
  Branches      511      514       +3     
==========================================
- Hits         5374     5353      -21     
  Misses        852      852              
  Partials        1        1              
Flag Coverage Δ
macos_latest 86.20% <88.88%> (-0.05%) :arrow_down:
ubuntu_latest 86.32% <88.88%> (-0.05%) :arrow_down:
windows_latest 19.89% <0.00%> (-0.08%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov[bot] avatar Nov 25 '25 06:11 codecov[bot]