alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
* Redesign the rawmidi midi read with timestamping API
       [not found] <1632309281569562912-webhooks-bot@alsa-project.org>
@ 2021-09-22 11:14 ` GitHub pull_request - opened
  0 siblings, 0 replies; only message in thread
From: GitHub pull_request - opened @ 2021-09-22 11:14 UTC (permalink / raw)
  To: alsa-devel

alsa-project/alsa-lib pull request #179 was opened from perexg:

It's follow-up for https://github.com/alsa-project/alsa-lib/pull/173 and https://github.com/alsa-project/alsa-lib/issues/172 . 

I am confident, that the direct framing mechanism for the transfer of the midi stream with the timestamping should be hidden and abstracted in the alsa-lib's API. This version uses a new read function (`snd_rawmidi_tread()`) which decodes and merges the data from the kernel frames. The framing mechanism is very specific to the kernel API and the cost of added data copy is negligible in this case (the kernel code basically does similar thing).

I kept the `snd_rawmidi_read()` functionality to read raw frames), but the structure for the parsing must be defined on it's own. This mechanism is unsupported.

This version seems more abstract and easy to use for apps than the separate decoder function for `snd_rawmidi_read()`.

Request URL   : https://github.com/alsa-project/alsa-lib/pull/179
Patch URL     : https://github.com/alsa-project/alsa-lib/pull/179.patch
Repository URL: https://github.com/alsa-project/alsa-lib

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2021-09-22 11:15 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <1632309281569562912-webhooks-bot@alsa-project.org>
2021-09-22 11:14 ` Redesign the rawmidi midi read with timestamping API GitHub pull_request - opened

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).