Differential D45095

sound: Remove unused "num" argument from chn_init() and related callers
ClosedPublic
Actions

Authored by christos on May 5 2024, 4:15 PM.

Tags

None

Referenced Files

	F150541492: D45095.diff
	Thu, Apr 2, 4:56 AM

	Unknown Object (File)
	Fri, Mar 27, 10:16 AM

	Unknown Object (File)
	Thu, Mar 26, 2:01 AM

	Unknown Object (File)
	Thu, Mar 19, 5:55 PM

	Unknown Object (File)
	Tue, Mar 17, 11:10 PM

	Unknown Object (File)
	Tue, Mar 17, 6:00 PM

	Unknown Object (File)
	Mon, Mar 16, 11:06 PM

	Unknown Object (File)
	Sat, Mar 14, 1:36 AM

View All 94 Files

Subscribers

Details

Reviewers

markj
emaste
dev_submerge.ch

Commits

rG17a50abe0e19: sound: Remove unused "num" argument from chn_init() and related callers
rG3af2beb8b24b: sound: Remove unused "num" argument from chn_init() and related callers

Summary

It is always -1 (i.e unused).

Sponsored by: The FreeBSD Foundation
MFC after: 1 week

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

christos created this revision.May 5 2024, 4:15 PM

Herald added a subscriber: imp. · View Herald TranscriptMay 5 2024, 4:15 PM

christos requested review of this revision.May 5 2024, 4:15 PM

Harbormaster completed remote builds in B57588: Diff 138150.May 5 2024, 4:15 PM

christos mentioned this in D44993: sound: Merge pcm_chn_create() and chn_init().May 5 2024, 4:15 PM

christos added parent revisions: D45015: sound: Move vchan-related code to pcm/vchan.*, D44993: sound: Merge pcm_chn_create() and chn_init().

Looks good to me, but I cannot compile and test it. Patch doesn't apply here locally because I'm missing previous commits. Time to get that stuff merged ;-)

This revision is now accepted and ready to land.May 6 2024, 4:42 PM

Closed by commit rG3af2beb8b24b: sound: Remove unused "num" argument from chn_init() and related callers (authored by christos). · Explain WhyMay 6 2024, 6:39 PM

This revision was automatically updated to reflect the committed changes.

christos added a commit: rG3af2beb8b24b: sound: Remove unused "num" argument from chn_init() and related callers.

christos added a commit: rG17a50abe0e19: sound: Remove unused "num" argument from chn_init() and related callers.May 17 2024, 7:34 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

sound/

pcm/

2 lines

11 lines

4 lines

4 lines

27 lines

Diff 138181

sys/dev/sound/pcm/channel.h

Loading...

sys/dev/sound/pcm/channel.c

Loading...

sys/dev/sound/pcm/sound.c

Loading...

sys/dev/sound/pcm/vchan.h

Loading...

sys/dev/sound/pcm/vchan.c

Loading...