makefs: use `char ` not `void ` for buf b_data and drop casts from msdos
ClosedPublic
Actions

Authored by emaste on Aug 20 2019, 9:57 PM.

Tags

None

Referenced Files

	F151447260: D21348.id61067.diff
	Wed, Apr 8, 11:30 AM

	F151416894: D21348.id61067.diff
	Wed, Apr 8, 7:13 AM

	Unknown Object (File)
	Tue, Apr 7, 10:45 AM

	Unknown Object (File)
	Mon, Apr 6, 9:40 AM

	Unknown Object (File)
	Sun, Apr 5, 11:43 PM

	Unknown Object (File)
	Fri, Mar 27, 11:38 AM

	Unknown Object (File)
	Tue, Mar 17, 3:50 AM

	Unknown Object (File)
	Feb 8 2026, 11:19 PM

View All Files

Subscribers

imp

Details

Reviewers

cem

Commits

rS351319: makefs: use `char *` not `void *` for buf b_data, drop casts in msdos

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

emaste created this revision.Aug 20 2019, 9:57 PM

You've build-checked this, right? makefs/ffs has some casts of b_data to larger structure types, which are totally valid for void* but compilers may warn about casting from a lower-alignment type like char to a larger alignment type.

This revision is now accepted and ready to land.Aug 20 2019, 10:03 PM

You've build-checked this, right?

Yes on amd64.

Closed by commit rS351319: makefs: use `char *` not `void *` for buf b_data, drop casts in msdos (authored by emaste). · Explain WhyAug 21 2019, 1:46 AM

This revision was automatically updated to reflect the committed changes.

emaste added a commit: rS351319: makefs: use `char *` not `void *` for buf b_data, drop casts in msdos.

Herald added a subscriber: imp. · View Herald TranscriptAug 21 2019, 1:46 AM

Revision Contents
Changeset List

Path

Size

head/

usr.sbin/

makefs/

ffs/

buf.h

2 lines

msdos/

msdosfs_fat.c

44 lines

Diff 61070

View Options

head/usr.sbin/makefs/ffs/buf.h