diff options
| author | John Keeping <john@metanate.com> | 2022-07-13 16:04:09 +0100 |
|---|---|---|
| committer | Jaroslav Kysela <perex@perex.cz> | 2022-11-04 22:01:19 +0100 |
| commit | 4daf22b278662721a59ccbb6ca33c8719f7f1565 (patch) | |
| tree | 926909c1a55b79edb7fde85c7dbfa0d5d7311cc8 | |
| parent | 0e657ca00b56bc06f8dd8f3396e79bc2329725aa (diff) | |
| download | alsa-utils-4daf22b278662721a59ccbb6ca33c8719f7f1565.tar.gz | |
amidi: restore space between bytes
Commit 9a8fcec ("amidi: add timestamp option for dump") removed the
space between bytes when dumping MIDI, changing the output from:
90 45 40
80 45 00
to:
904540
804500
It seems that this was an unintentional side effect of refactoring the
code to add the new timestamp output but the result is less readable
than it was before.
Restore the space between bytes in the same message.
Fixes: 9a8fcec ("amidi: add timestamp option for dump")
Fixes: https://github.com/alsa-project/alsa-utils/pull/158/
Signed-off-by: John Keeping <john@metanate.com>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
| -rw-r--r-- | amidi/amidi.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/amidi/amidi.c b/amidi/amidi.c index 3d7bd0f..b7eebdf 100644 --- a/amidi/amidi.c +++ b/amidi/amidi.c @@ -433,14 +433,12 @@ static void print_byte(unsigned char byte, struct timespec *ts) fputs("\n ", stdout); } - if (newline) { - printf("\n"); - + putchar(newline ? '\n' : ' '); + if (newline && do_print_timestamp) { /* Nanoseconds does not make a lot of sense for serial MIDI (the * 31250 bps one) but I'm not sure about MIDI over USB. */ - if (do_print_timestamp) - printf("%lld.%.9ld) ", (long long)ts->tv_sec, ts->tv_nsec); + printf("%lld.%.9ld) ", (long long)ts->tv_sec, ts->tv_nsec); } printf("%02X", byte); |