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)
Sat, Sep 14, 8:23 AM
Unknown Object (File)
Wed, Sep 11, 5:52 AM
Unknown Object (File)
Sun, Sep 8, 5:54 PM
Unknown Object (File)
Sun, Sep 8, 1:06 AM
Unknown Object (File)
Sat, Sep 7, 7:39 PM
Unknown Object (File)
Sat, Sep 7, 10:18 AM
Unknown Object (File)
Thu, Sep 5, 1:19 PM
Unknown Object (File)
Wed, Sep 4, 12:51 PM
Subscribers

Diff Detail

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

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.Tue, Aug 20, 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.Fri, Aug 23, 8:51 PM
This revision was automatically updated to reflect the committed changes.