improved output table formatting; removed "before" bytes

...because it's normal to have three leading 0 bytes in the first PmMessage that contains the initial sysex byte.
jimm · Aug 24, 2021 · 3bfa911 · 3bfa911
1 parent 8e7d3bb
commit 3bfa911
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 13 deletions.
diff --git a/src/kronos.cpp b/src/kronos.cpp
@@ -104,7 +104,6 @@ void Kronos::read_sysex() {
   SysexState state;
   time_t start = time(0);
   PmError err;
-  bool logged_bytes_before_sysex = false;
 
   state = waiting;
   clog << "waiting for sysex" << endl;
@@ -134,12 +133,6 @@ void Kronos::read_sysex() {
             state = received;
             clog << "sysex end seen" << endl;
           }
-
-          if (state == waiting && !logged_bytes_before_sysex) {
-            clog << "dropping data before sysex, first byte dropped "
-                 << setw(2) << setfill('0') << hex << (unsigned int)b << endl;
-            logged_bytes_before_sysex = true;
-          }
         }
       }
     }

diff --git a/src/set_list_file.cpp b/src/set_list_file.cpp
@@ -90,21 +90,21 @@ void SetListFile::table_headers(const char * const h1, const char * const h2) {
 
 void SetListFile::table_row(const char * const col1, const char * const col2) {
   _out << "| " << setw(COL1_DATA_WIDTH) << col1
-      << " | " << setw(COL2_DATA_WIDTH) << col2
-      << " |" << endl;
+       << " | " << setw(COL2_DATA_WIDTH) << left << col2
+       << " |" << endl;
 }
 
 void SetListFile::table_row(const char * const col1, const string &col2) {
   fprintf(stderr, "col1 %s, col2 %s\n", col1, col2.c_str()); // DEBUG
   _out << "| " << setw(COL1_DATA_WIDTH) << col1
-      << " | " << setw(COL2_DATA_WIDTH) << col2
-      << " |" << endl;
+       << " | " << setw(COL2_DATA_WIDTH) << left << col2
+       << " |" << endl;
 }
 
 void SetListFile::table_row(const char * const col1, int value) {
   _out << "| " << setw(COL1_DATA_WIDTH) << col1
-      << " | " << setw(COL2_DATA_WIDTH) << value
-      << " |" << endl;
+       << " | " << setw(COL2_DATA_WIDTH) << left << value
+       << " |" << endl;
 }
 
 void SetListFile::table_end() {