Page MenuHomeFreeBSD
Differential D37937

wdatwd: fix i386 build breakage.
ClosedPublic
Actions

Authored by takawata on Jan 3 2023, 11:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 8, 3:12 PM
Unknown Object (File)
Mon, Apr 8, 3:57 AM
Unknown Object (File)
Feb 29 2024, 6:40 AM
Unknown Object (File)
Dec 20 2023, 8:02 AM
Unknown Object (File)
Dec 3 2023, 10:58 PM
Unknown Object (File)
Aug 6 2023, 7:34 AM
Unknown Object (File)
Aug 2 2023, 12:05 PM
Unknown Object (File)
Aug 2 2023, 12:02 PM
View All 20 Files
Subscribers
imp

Details

Reviewers
hrs
Commits
rGeb03c1f81cbc: wdatwd: fix i386 build breakage.
Summary

Formatting string for uint64_t has portablity issue and
lead bulid breakage so fix it.

Test Plan

compile module both i386 and amd64

Diff Detail

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

Event Timeline

takawata created this revision.Jan 3 2023, 11:33 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 3 2023, 11:33 PM
takawata requested review of this revision.Jan 3 2023, 11:33 PM
Harbormaster completed remote builds in B48946: Diff 114738.Jan 3 2023, 11:33 PM
hrs accepted this revision.Jan 4 2023, 12:53 AM
hrs added inline comments.
sys/dev/wdatwd/wdatwd.c
366–371

This might be a matter of preference, but %ju and type casting by (uintmax_t) would be simpler.

681

Same here. %jx, in this case.

This revision is now accepted and ready to land.Jan 4 2023, 12:53 AM
Closed by commit rGeb03c1f81cbc: wdatwd: fix i386 build breakage. (authored by takawata). · Explain WhyJan 4 2023, 1:05 AM
This revision was automatically updated to reflect the committed changes.
takawata added a commit: rGeb03c1f81cbc: wdatwd: fix i386 build breakage..

Revision Contents
Changeset List

PathSize
sys/
dev/
wdatwd/
21 lines

Diff 114741

View Options

sys/dev/wdatwd/wdatwd.c

Loading...