Page MenuHomeFreeBSD
Differential D44757

sound: Update some LICENSE headers
ClosedPublic
Actions

Authored by christos on Fri, Apr 12, 6:11 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 29, 7:42 PM
Unknown Object (File)
Sat, Apr 27, 10:20 PM
Unknown Object (File)
Fri, Apr 26, 5:05 AM
Unknown Object (File)
Tue, Apr 16, 3:22 AM
Unknown Object (File)
Sun, Apr 14, 5:23 PM
Unknown Object (File)
Sun, Apr 14, 1:07 PM
Unknown Object (File)
Sun, Apr 14, 12:44 PM
Unknown Object (File)
Sun, Apr 14, 12:38 PM
View All 9 Files
Subscribers
imp

Details

Reviewers
markj
Commits
rG51abcd0c477a: sound: Update some LICENSE headers
rG4f854658c5d6: sound: Update some LICENSE headers
Summary

The following commits introduced substantial changes to pcm/dsp.c,
pcm/sndstat.c and pcm/sound.c.

9da3b645dbaaad724d524727d003fed7be05ff7c ("sound: Move
sndstat_prepare_pcm() to pcm/sndstat.c and remove
sndstat_entry->handler")
e8c0d15a64fadb4a330f2da7244becaac161bb70 ("sound: Get rid of snd_clone
and use DEVFS_CDEVPRIV(9)")

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.Fri, Apr 12, 6:11 PM
Herald added a subscriber: imp. · View Herald TranscriptFri, Apr 12, 6:11 PM
christos requested review of this revision.Fri, Apr 12, 6:11 PM
Harbormaster completed remote builds in B57056: Diff 136947.Fri, Apr 12, 6:11 PM
markj added a comment.Fri, Apr 12, 6:19 PM

The log message should say which commit(s) this is for.

sys/dev/sound/pcm/sndstat.c
7–11

The "All rights reserved" should come before the new lines.

christos edited the summary of this revision. (Show Details)Fri, Apr 12, 6:28 PM
christos updated this revision to Diff 136948.Fri, Apr 12, 6:29 PM
christos marked an inline comment as done.
christos edited the summary of this revision. (Show Details)

Address Mark's comments.

Harbormaster completed remote builds in B57057: Diff 136948.Fri, Apr 12, 6:29 PM
markj accepted this revision.Fri, Apr 12, 6:30 PM

Looks ok with one nit fixed.

sys/dev/sound/pcm/dsp.c
12

Don't include this blank line.

This revision is now accepted and ready to land.Fri, Apr 12, 6:30 PM
Closed by commit rG4f854658c5d6: sound: Update some LICENSE headers (authored by christos). · Explain WhyFri, Apr 12, 6:36 PM
This revision was automatically updated to reflect the committed changes.
christos added a commit: rG4f854658c5d6: sound: Update some LICENSE headers.
christos added a commit: rG51abcd0c477a: sound: Update some LICENSE headers.Thu, Apr 18, 12:42 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
sound/
pcm/
4 lines
4 lines
4 lines

Diff 136949

View Options

sys/dev/sound/pcm/dsp.c

Loading...
View Options

sys/dev/sound/pcm/sndstat.c

Loading...
View Options

sys/dev/sound/pcm/sound.c

Loading...