Page MenuHomeFreeBSD
Differential D45776

sound: Make DSP_FIXUP_ERROR() regular code
ClosedPublic
Actions

Authored by christos on Jun 29 2024, 3:38 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 30, 8:20 PM
Unknown Object (File)
Wed, Nov 26, 10:48 AM
Unknown Object (File)
Tue, Nov 25, 11:49 PM
Unknown Object (File)
Sun, Nov 23, 10:56 PM
Unknown Object (File)
Mon, Nov 17, 12:10 PM
Unknown Object (File)
Nov 4 2025, 3:29 AM
Unknown Object (File)
Nov 2 2025, 8:20 AM
Unknown Object (File)
Nov 1 2025, 9:55 PM
View All 59 Files
Subscribers
imp

Details

Reviewers
markj
emaste
dev_submerge.ch
Commits
rGd4b65f3f53ee: sound: Make DSP_FIXUP_ERROR() regular code
rGadc1713fb13f: sound: Make DSP_FIXUP_ERROR() regular code
Summary

No reason to have this as a macro. No functional change intended.

Sponsored by: The FreeBSD Foundation
MFC after: 2 days

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 58394
Build 55282: arc lint + arc unit

Event Timeline

christos created this revision.Jun 29 2024, 3:38 PM
Herald added a subscriber: imp. · View Herald TranscriptJun 29 2024, 3:38 PM
christos requested review of this revision.Jun 29 2024, 3:38 PM
Harbormaster completed remote builds in B58394: Diff 140359.Jun 29 2024, 3:38 PM
christos added a parent revision: D45775: sound: Simplify getchns().Jun 29 2024, 3:38 PM
markj accepted this revision.Jun 30 2024, 3:30 PM
This revision is now accepted and ready to land.Jun 30 2024, 3:30 PM
emaste accepted this revision.Jul 1 2024, 1:28 PM

Expanding this inline LGTM. I wonder if it should be else if though (as a subsequent change)

christos added a comment.Jul 1 2024, 6:22 PM
In D45776#1044783, @emaste wrote:

Expanding this inline LGTM. I wonder if it should be else if though (as a subsequent change)

I agree with making it an else if. If the open flags are invalid then we don't need to check the second if at all. I think this change can be included in this patch though.

dev_submerge.ch accepted this revision.Jul 3 2024, 5:05 PM
Closed by commit rGadc1713fb13f: sound: Make DSP_FIXUP_ERROR() regular code (authored by christos). · Explain WhyJul 6 2024, 6:24 PM
This revision was automatically updated to reflect the committed changes.
christos added a commit: rGadc1713fb13f: sound: Make DSP_FIXUP_ERROR() regular code.
christos added a commit: rGd4b65f3f53ee: sound: Make DSP_FIXUP_ERROR() regular code.Jul 10 2024, 4:49 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
pcm/
16 lines

Diff 140359

View Options

sys/dev/sound/pcm/dsp.c

Loading...