Page MenuHomeFreeBSD

sound examples: Simplify MIDI example
ClosedPublic

Authored by christos on Aug 15 2024, 3:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 15, 5:14 PM
Unknown Object (File)
Sat, Nov 9, 10:02 AM
Unknown Object (File)
Fri, Nov 8, 12:13 AM
Unknown Object (File)
Thu, Oct 24, 7:45 AM
Unknown Object (File)
Tue, Oct 22, 12:43 PM
Unknown Object (File)
Sun, Oct 20, 10:54 PM
Unknown Object (File)
Sun, Oct 20, 10:53 PM
Unknown Object (File)
Sun, Oct 20, 10:53 PM
Subscribers

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.

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.

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/.

Add ossmidi.h remove to ObsoleteFiles.inc.

This revision is now accepted and ready to land.Aug 17 2024, 4:24 PM
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.

christos marked an inline comment as done.

Make reads more robust.

This revision now requires review to proceed.Aug 20 2024, 9:18 PM
share/examples/sound/oss/midi.c
54

Can also be for more robustness.

This revision is now accepted and ready to land.Aug 23 2024, 8:51 PM
This revision was automatically updated to reflect the committed changes.