[alsa-devel] [PATCH v5 0/2] amidi delay option
Takashi Sakamoto
o-takashi at sakamocchi.jp
Mon Aug 15 21:54:49 CEST 2016
Hi,
On Aug 16 2016 01:37, Felipe F. Tonello wrote:
> Small improvement to amidi, adding useful functionality.
>
> changes from v4:
> * simplified code
>
> changes from v3:
> * fixed a bug if memchr() returns NULL
>
> changes from v2:
> * only check of end of sysex (0xf7)
> * use memchr() instead of direct loop approach
> * fixed commit message
>
> changes from v1:
> * remove delay in between any MIDI message, only SysEx
> * added patch to use getopt_long() arguments
>
> Felipe F. Tonello (2):
> amidi: add delay option
> amidi: use GNU getopt_long() proper argument names
>
> amidi/amidi.1 | 14 ++++++++++++++
> amidi/amidi.c | 61 +++++++++++++++++++++++++++++++++++++++++++----------------
> 2 files changed, 59 insertions(+), 16 deletions(-)
Please notice the other developers' work. Clemens already posted his
patchset with similar improvements.
http://mailman.alsa-project.org/pipermail/alsa-devel/2016-August/111737.html
And I reviewed and tested it.
http://mailman.alsa-project.org/pipermail/alsa-devel/2016-August/111745.html
It's not merged yet, because subsystem maintainer has a summer vacation.
http://mailman.alsa-project.org/pipermail/alsa-devel/2016-August/111741.html
It's better for you to wait for merging or work based on Clemens'
patchset, I think.
Regards
Takashi Sakamoto
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0xD66FD341.asc
Type: application/pgp-keys
Size: 10205 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20160816/6d36ab8c/attachment-0001.bin>
More information about the Alsa-devel
mailing list