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
Attachment:
0xD66FD341.asc
Description: application/pgp-keys
_______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
