Paths

Table of Contentst

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

	F168152223: D27580.id80738.diff
	Wed, Aug 26, 3:06 PM

	F168126020: D27580.id80609.diff
	Wed, Aug 26, 12:15 PM

	F168124788: D27580.id.diff
	Wed, Aug 26, 12:08 PM

	F168107709: D27580.diff
	Wed, Aug 26, 10:29 AM

	Unknown Object (File)
	Tue, Aug 4, 6:45 AM

	Unknown Object (File)
	Tue, Aug 4, 3:00 AM

	Unknown Object (File)
	Mon, Aug 3, 6:13 AM

	Unknown Object (File)
	Sat, Aug 1, 5:03 PM

Subscribers

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

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

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

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

Path

Size

head/

sys/

dev/

mxge/

8 lines

Diff 80738

head/sys/dev/mxge/if_mxge.c

Loading...