Sponsored by: The FreeBSD Foundation
MFC after: 2 days
Details
Diff Detail
- Repository
- rG FreeBSD src repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Commit message should include some more detail (e.g. explain that it's not really OSS) and will need an ObsoleteFiles.inc update for the now-removed ossmidi.h.
I will combine D46308 with this and update ObsoleteFiles.inc. Updating the diff in a few mins.
It might be better to keep the move & rewrite as separate commits so it's clear to git that it's the same file (rather than inferring this from delete + new file that are similar); if you do keep them separate I don't have a strong opinion on move-then-edit or edit-then-move.
Alright. Regarding the commit message I mention that this code is not OSS related in the other patch, which is the one that actually moves it out of oss/.
share/examples/sound/oss/midi.c | ||
---|---|---|
54 | Shouldn't this be read(fd, &raw, sizeof(raw)) < sizeof(raw) according to read(2)? See also similar lines below. |
share/examples/sound/oss/midi.c | ||
---|---|---|
54 | Can also be for more robustness. |