Page MenuHomeFreeBSD
Differential D27580

Use uintptr_t instead of unsigned long for integers holding pointers.
ClosedPublic
Actions

Authored by jhb on Dec 11 2020, 9:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Feb 23, 8:47 AM
Unknown Object (File)
Sat, Feb 22, 2:12 PM
Unknown Object (File)
Sat, Feb 22, 2:12 PM
Unknown Object (File)
Sat, Feb 22, 1:49 PM
Unknown Object (File)
Jan 31 2025, 10:36 PM
Unknown Object (File)
Jan 24 2025, 7:04 PM
Unknown Object (File)
Jan 24 2025, 5:17 PM
Unknown Object (File)
Jan 23 2025, 7:00 PM
View All 59 Files
Subscribers
imp

Details

Reviewers
imp
gallatin
Commits
rS368687: Use uintptr_t instead of unsigned long for integers holding pointers.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb created this revision.Dec 11 2020, 9:26 PM
Herald added a subscriber: imp. · View Herald TranscriptDec 11 2020, 9:26 PM
Harbormaster completed remote builds in B35360: Diff 80609.Dec 11 2020, 9:26 PM
jhb requested review of this revision.Dec 11 2020, 9:26 PM
imp accepted this revision.Dec 11 2020, 10:27 PM

Unions might be better for this, but these casts look good and are a more minimal change.

This revision is now accepted and ready to land.Dec 11 2020, 10:27 PM
jhb added a comment.Dec 12 2020, 4:52 PM

Well, the other fix is to use something like __aligned or __alignas, but I went for minimal changes, especially given that this driver is probably not as widely used.

jhb added a reviewer: gallatin.Dec 12 2020, 4:52 PM
imp added a comment.Dec 12 2020, 5:49 PM
In D27580#616464, @jhb wrote:

Well, the other fix is to use something like __aligned or __alignas, but I went for minimal changes, especially given that this driver is probably not as widely used.

Yea. Makes sense. Likely not that important for this driver.

gallatin accepted this revision.Dec 14 2020, 1:26 AM
Closed by commit rS368687: Use uintptr_t instead of unsigned long for integers holding pointers. (authored by jhb). · Explain WhyDec 16 2020, 12:18 AM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rS368687: Use uintptr_t instead of unsigned long for integers holding pointers..

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
mxge/
8 lines

Diff 80738

View Options

head/sys/dev/mxge/if_mxge.c

Loading...