Page MenuHomeFreeBSD

bsddialog: Correct type for bsddialog_total_progview
ClosedPublic

Authored by emaste on May 9 2025, 12:41 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 9, 2:59 AM
Unknown Object (File)
Mon, Jul 7, 3:04 PM
Unknown Object (File)
Mon, Jul 7, 2:57 AM
Unknown Object (File)
Mon, Jul 7, 12:41 AM
Unknown Object (File)
Sun, Jul 6, 10:59 AM
Unknown Object (File)
Sat, Jul 5, 12:46 PM
Unknown Object (File)
Fri, Jul 4, 1:35 AM
Unknown Object (File)
Thu, Jul 3, 9:24 AM
Subscribers

Details

Summary
It was an int, but printed with %lli format.  Although it would be
reasonable to use a 32-bit int here (i.e., changing the printf format
instead) this matches what was done in bsddialog upstream (and now in
main).

PR:             286655

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste created this revision.
emaste added a reviewer: jhb.

OK, I remember the problem described in PR286655, I found it testing the installer on a 32bit (cheap) raspberry and watching a video on youtube (2 years ago). Of course "num extracted bytes" > "max int", I obseverd things like "-676330 files read" (negative num).
I fixed like in this review and now it is in main/CURRENT: I kept %lli because previously libdpv printed a %lli and I changed int to long long int.

This revision is now accepted and ready to land.May 9 2025, 4:52 PM

I fixed like in this review and now it is in main/CURRENT

Yeah; after 14.3 is out I think we should MFC the bsddialog updates. For now I just wanted a minimal change.