update for newer ztd version

+ Files now auto reload when changes are detected
+ Added --no-reload option to disable auto reloading
* Internal cleanups

.gitignore

@@ -0,0 +1,3 @@
+zmidimap
+obj
+Zmakefile

Makefile

@@ -11,6 +11,8 @@ CC=g++
 CXXFLAGS= -I$(IDIR) -Wall -pedantic -std=c++17
 ifeq ($(DEBUG),true)
 	CXXFLAGS += -g
+else
+  CXXFLAGS += -O2
 endif
 ifeq ($(STATIC),true)
 	LDFLAGS += -l:libztd.a

README.md

@@ -2,7 +2,7 @@
 
 Map midi signals coming from ALSA midi devices to shell commands  
 
-Dependencies: alsa-utils , [ztd](https://github.com/zawwz/ztd)
+Dependencies: alsa-utils
 
 ## Installing
 
@@ -14,11 +14,12 @@ Download the .deb package then run: `sudo dpkg -i zmidimap.deb ; sudo apt -f ins
 
 ### Standalone
 
-Download ``zmidimap-static.tar.gz`` for a standalone without library dependency  
-Then move the executable `zmidimap` to your PATH directory
+Download ``zmidimap.tar.gz`` for a standalone executable
 
 ### From source
 
+Depends on: [ztd](https://github.com/zawwz/ztd)
+
 ``make -j6`` for a shared build  
 ``STATIC=true make -j6`` for a static build  
 

example.mim

@@ -1,78 +1,43 @@
-[
+Device "Launch Control"
 
-// LAUNCH  CONTROL
-{
-  name=Launch Control
-  commands=[
-    // KNOBS HIGH
-    {
-      //KNOB HA
-      //displays value 0:127 for knobs 21-28
-      type=controller
+  //display value 0:127 for knobs 21-28
+  Command controller
     id=21:28
-      shell=echo "Knob #$id ch$channel:$value"
-    },
-    // KNOBS LOW
-    {
-      //KNOB L1
-      //displays value 0:127 for knob 41 from any channel
-      type=controller
+    shell='echo "Knob #$id ch$channel:$value"'
+
+  //display value -100:100 for knob 41 on channel 0
+  Command controller
     id=41
-      channel=*
-      shell=echo "Knob #$id ch$channel:$value"
-    },
-    {
-      //KNOB L2
-      //displays value -100:100 for knob 42 on channel 0
-      type=controller
-      id=42
     channel=0
     remap=-100:100
-      shell=echo "Knob #$id ch$channel:$value r:$rawvalue"
-    },
-    {
-      //KNOB L3 H1
-      //displays value 0:1:0 for knob 42 on channel 1 (first half)
-      type=controller
+    shell='echo "Knob #$id ch0:$value r:$rawvalue"'
+
+  //display value 0:1:0 for knob 41 on channel 1 (first half)
+  Command controller
     id=42
     channel=1
     range=0:63
     remap=0:1
     float=true
-      shell=echo "Knob #$id ch$channel:$value"
-    },
-    {
-      //KNOB L3 H2
-      //displays value 0:1:0 for knob 42 on channel 1 (second half)
-      type=controller
+    shell='echo "Knob #$id ch1:$value r:$rawvalue"'
+
+  //display value 0:1:0 for knob 41 on channel 1 (second half)
+  Command controller
     id=42
     channel=1
     range=64:127
     remap=1:0
     float=true
-      shell=echo "Knob #$id ch$channel:$value"
-    }
-  ]
-}
-,
-// LAUNCHPAD
-{
-  name=Launchpad S
-  commands=[
-    {
-      // ANY NOTE ON
-      type=note
-      id=*
-      shell=echo "Note $id on  velocity:$velocity"
-    },
-    {
-      // ANY NOTE OFF
-      type=note
-      id=*
-      trigger=0
-      shell=echo "Note $id off"
-    }
-  ]
-}
+    shell='echo "Knob #$id ch1:$value r:$rawvalue"'
 
-]
+
+Device "Launchpad S"
+
+  //display any note pressed and its velocity
+  Command note
+    shell=echo Note $id on  velocity:$velocity
+
+  //display any note off
+  Command note
+    trigger=0
+    shell=echo Note $id off

example.zfd

@@ -0,0 +1,67 @@
+[
+
+// LAUNCH  CONTROL
+{
+  name=Launch Control
+  commands=[
+    // KNOBS HIGH
+    {
+      //KNOB HA
+      //displays value 0:127 for knobs 21-28
+      type=controller
+      id=21:28
+      shell='echo "Knob #$id ch$channel:$value"'
+    },
+    {
+      //KNOB L2
+      //displays value -100:100 for knob 41 on channel 0
+      type=controller
+      id=42
+      channel=0
+      remap=-100:100
+      shell='echo "Knob #$id ch0:$value r:$rawvalue"'
+    },
+    {
+      //KNOB L3 H1
+      //displays value 0:1:0 for knob 41 on channel 1 (first half)
+      type=controller
+      id=42
+      channel=1
+      range=0:63
+      remap=0:1
+      float=true
+      shell='echo "Knob #$id ch1:$value r:$rawvalue"'
+    },
+    {
+      //KNOB L3 H2
+      //displays value 0:1:0 for knob 41 on channel 1 (second half)
+      type=controller
+      id=42
+      channel=1
+      range=64:127
+      remap=1:0
+      float=true
+      shell='echo "Knob #$id ch1:$value r:$rawvalue"'
+    }
+  ]
+}
+,
+// LAUNCHPAD
+{
+  name=Launchpad S
+  commands=[
+    {
+      // ANY NOTE ON
+      type=note
+      shell=echo Note $id on  velocity:$velocity
+    },
+    {
+      // ANY NOTE OFF
+      type=note
+      trigger=0
+      shell=echo Note $id off
+    }
+  ]
+}
+
+]

gen_help_format.sh

@@ -1,10 +1,38 @@
 FORMAT_FOLDER=help_format
 IDIR=include
 
-cp $FORMAT_FOLDER/help_template_head $IDIR/help.h
+SLASHSCRIPT='s|\\|\\\\|g;s|\"|\\\"|g'
+NEWLINESCRIPT=':a;N;$!ba;s/\n/\\n/g;'
 
-echo "#define FILE_FORMAT \"zmidimap$(sed -n -e 'H;${x;s/\n/\\n/g;s/^,//;p;}' $FORMAT_FOLDER/file-format)\"" >> $IDIR/help.h
-echo "#define SHELL_FORMAT \"zmidimap$(sed -n -e 'H;${x;s/\n/\\n/g;s/^,//;p;}' $FORMAT_FOLDER/shell-format)\"" >> $IDIR/help.h
-echo "#define COMMAND_TAGS \"zmidimap$(sed -n -e 'H;${x;s/\n/\\n/g;s/^,//;p;}' $FORMAT_FOLDER/command-tags)\"" >> $IDIR/help.h
+filetocstr()
+{
+  sed -e $SLASHSCRIPT "$1" | sed $NEWLINESCRIPT
+}
 
-cat $FORMAT_FOLDER/help_template_tail >> $IDIR/help.h
+gen_line()
+{
+  name="$(basename "$1")"
+  echo "#define $(echo "$name" | tr '[:lower:]' '[:upper:]') \"$(filetocstr "$1")\""
+}
+
+help_header()
+{
+  echo '#ifndef HELP_H
+#define HELP_
+'
+}
+
+help_footer()
+{
+  echo '
+#endif //HELP_H'
+}
+
+help_header > "$IDIR/help.h"
+
+for I in help_format/*
+do
+  gen_line "$I" >> "$IDIR/help.h"
+done
+
+help_footer >> "$IDIR/help.h"

help_format/command_tags

@@ -1,14 +1,5 @@
 -- [COMMAND TAGS] --
 
-[Global tags]
-  type=<type>
-  shell=<shell command>
---
-  *type: type of the signal: note/controller/pitch/system/connect/disconnect
-    > mandatory
-  *shell: shell command to be executed
-    > mandatory
-
 [Note tags]
   id=<interval>
   channel=<x/*>

help_format/file-format

@@ -1,23 +0,0 @@
--- [FILE FORMAT] --
-
-[<device>,<device>]
-
-<device> format
-  {
-    name=<name>
-    commands=[<command>,<command>]
-  }
---
-  *name: string referring to client name of the device
-
-<command> format
-  {
-    <tag>=<value>
-    <tag>=<value>
-    ...
-  }
---
-  value can be concatenated with \"\" or ''
-  see command-tags
-
-Full info on file format: https://github.com/zawwz/zFiledat

help_format/help_template_head

@@ -1,2 +0,0 @@
-#ifndef HELP_H
-#define HELP_H

help_format/help_template_tail

@@ -1 +0,0 @@
-#endif //HELP_H

help_format/mim_format

@@ -0,0 +1,26 @@
+-- [MIM FILE FORMAT] --
+
+Device <name>
+  Command <type>
+    shell=<shell command>
+    <tag>=<value>
+    <tag>=<value>
+    ...
+  Command <type>
+    ...
+
+Device <name>
+...
+
+--
+*name: string referring to client name of the device
+  > mandatory
+*type: type of the signal: note/controller/pitch/system/connect/disconnect
+  > mandatory
+*shell: shell command to be executed
+  > mandatory
+
+Shell commands can be concatenated with "" or ''
+Comments are written with // or # and end at end of line
+Multiple tags can be put on the same line by separating them with ;
+See --command-tags for optional command tags

help_format/zfd_format

@@ -0,0 +1,30 @@
+-- [ZFD FILE FORMAT] --
+
+ZFD format: http://zawz.net/doc/ztd/zfd.html
+
+[<device>,<device>]
+
+-- <device> format --
+  {
+    name=<name>
+    commands=[<command>,<command>]
+  }
+--
+*name: string referring to client name of the device
+
+<command> format
+  {
+    type=<type>
+    shell=<shell command>
+    <tag>=<value>
+    <tag>=<value>
+    ...
+  }
+--
+*type: type of the signal: note/controller/pitch/system/connect/disconnect
+  > mandatory
+*shell: shell command to be executed
+  > mandatory
+
+Shell commands can be concatenated with "" or ''
+see --command-tags for optional command tags

include/device.hpp

@@ -8,8 +8,6 @@
 #include "command.hpp"
 #include <ztd/filedat.hpp>
 
-void sh(std::string const& string);
-
 class Device
 {
 public:
@@ -19,7 +17,7 @@ public:
   bool start_loop();
   void run_signal(char* buff);
 
-  bool import_chunk(ztd::chunkdat const& ch);
+  bool import_chunk(const ztd::chunkdat& ch);
   ztd::chunkdat export_chunk();
 
   std::string name;
@@ -34,7 +32,6 @@ public:
   std::vector<ConnectCommand> connectCommands;
   std::vector<DisconnectCommand> disconnectCommands;
 
-  std::thread thread;
   pid_t thread_pid;
 private:
   static void loop(Device* dev);
@@ -42,4 +39,6 @@ private:
 
 extern std::vector<Device*> device_list;
 
+void clean_devices();
+
 #endif //DEVICE_HPP

include/format.hpp

@@ -0,0 +1,14 @@
+#ifndef FORMAT_HPP
+#define FORMAT_HPP
+
+#include <ztd/filedat.hpp>
+
+ztd::chunkdat mimtochk(std::string mim);
+
+std::string chktomim(ztd::chunkdat const& chk, std::string const& aligner);
+
+std::string file_strimport(const std::string& path);
+
+bool is_mim(const std::string& str);
+
+#endif //FORMAT_HPP

include/help.h

@@ -1,6 +1,9 @@
 #ifndef HELP_H
-#define HELP_H
-#define FILE_FORMAT "zmidimap\n-- [FILE FORMAT] --\n\n[<device>,<device>]\n\n<device> format\n  {\n    name=<name>\n    commands=[<command>,<command>]\n  }\n--\n  *name: string referring to client name of the device\n\n<command> format\n  {\n    <tag>=<value>\n    <tag>=<value>\n    ...\n  }\n--\n  value can be concatenated with \"\" or ''\n  see command-tags\n\nFull info on file format: https://github.com/zawwz/zFiledat"
-#define SHELL_FORMAT "zmidimap\n-- [SHELL FORMAT] --\n\nShell command follows regular shell format\n\n-- [Environment] --\n\n[Note]\n  $id: id of the note\n  $channel: channel of the note\n  $velocity: velocity of the note\n\n[Controller]\n  $value: value of the controller (remapped)\n  $id: id of the controller\n  $channel: channel of the controller\n  $rawvalue: original value of the controller\n\n[Pitch]\n  $value: value of the bend (remapped)\n  $rawvalue: original value of the bend\n\n[System]\n  $code: hexadecimal code of the system signal"
-#define COMMAND_TAGS "zmidimap\n-- [COMMAND TAGS] --\n\n[Global tags]\n  type=<type>\n  shell=<shell command>\n--\n  *type: type of the signal: note/controller/pitch/system/connect/disconnect\n    > mandatory\n  *shell: shell command to be executed\n    > mandatory\n\n[Note tags]\n  id=<interval>\n  channel=<x/*>\n  trigger=<interval>\n--\n  *id: note id from 0 to 127\n    > optional, default 0:127\n  *channel: value from 0 to 16 for channel, * for any channel\n    > optional, default *\n  *trigger: note velocity from 0 to 127 that triggers the command\n    > optional, default 1:127\n\n[Controller tags]\n  id=<interval>\n  channel=<x/*>\n  range=<interval>\n  remap=<interval>\n  float=<true/false>\n--\n  *id: controller id from 0 to 127\n    > optional, default 0:127\n  *channel: value from 0 to 16 for channel, * for any channel\n    > optional, default *\n  *range: controller value from 0 to 127 that triggers command\n    > optional, default 0:127\n  *remap: remaps the range to given interval\n    > optional, default same as range\n  *float: boolean value defining if output is a floating point value\n    > optional, default false\n\n[Pitch bend tags]\n  range=<interval>\n  remap=<interval>\n  float=<true/false>\n--\n  *range: controller value from -8192 to 8191 that triggers command\n    > optional, default -8192:8191\n  *remap: remaps the range to given interval\n    > optional, default same as range\n  *float: boolean value defining if output is a floating point value\n    > optional, default false\n\n[Interval Format]\n  x:y   range from x to y\n  x     single value x\n  *     all possible values"
+#define HELP_
+
+#define COMMAND_TAGS "-- [COMMAND TAGS] --\n\n[Note tags]\n  id=<interval>\n  channel=<x/*>\n  trigger=<interval>\n--\n  *id: note id from 0 to 127\n    > optional, default 0:127\n  *channel: value from 0 to 16 for channel, * for any channel\n    > optional, default *\n  *trigger: note velocity from 0 to 127 that triggers the command\n    > optional, default 1:127\n\n[Controller tags]\n  id=<interval>\n  channel=<x/*>\n  range=<interval>\n  remap=<interval>\n  float=<true/false>\n--\n  *id: controller id from 0 to 127\n    > optional, default 0:127\n  *channel: value from 0 to 16 for channel, * for any channel\n    > optional, default *\n  *range: controller value from 0 to 127 that triggers command\n    > optional, default 0:127\n  *remap: remaps the range to given interval\n    > optional, default same as range\n  *float: boolean value defining if output is a floating point value\n    > optional, default false\n\n[Pitch bend tags]\n  range=<interval>\n  remap=<interval>\n  float=<true/false>\n--\n  *range: controller value from -8192 to 8191 that triggers command\n    > optional, default -8192:8191\n  *remap: remaps the range to given interval\n    > optional, default same as range\n  *float: boolean value defining if output is a floating point value\n    > optional, default false\n\n[Interval Format]\n  x:y   range from x to y\n  x     single value x\n  *     all possible values"
+#define MIM_FORMAT "-- [MIM FILE FORMAT] --\n\nDevice <name>\n  Command <type>\n    shell=<shell command>\n    <tag>=<value>\n    <tag>=<value>\n    ...\n  Command <type>\n    ...\n\nDevice <name>\n...\n\n--\n*name: string referring to client name of the device\n  > mandatory\n*type: type of the signal: note/controller/pitch/system/connect/disconnect\n  > mandatory\n*shell: shell command to be executed\n  > mandatory\n\nShell commands can be concatenated with \"\" or ''\nComments are written with // or # and end at end of line\nMultiple tags can be put on the same line by separating them with ;\nSee --command-tags for optional command tags"
+#define SHELL_FORMAT "-- [SHELL FORMAT] --\n\nShell command follows regular shell format\n\n-- [Environment] --\n\n[Note]\n  $id: id of the note\n  $channel: channel of the note\n  $velocity: velocity of the note\n\n[Controller]\n  $value: value of the controller (remapped)\n  $id: id of the controller\n  $channel: channel of the controller\n  $rawvalue: original value of the controller\n\n[Pitch]\n  $value: value of the bend (remapped)\n  $rawvalue: original value of the bend\n\n[System]\n  $code: hexadecimal code of the system signal"
+#define ZFD_FORMAT "-- [ZFD FILE FORMAT] --\n\nZFD format: http://zawz.net/doc/ztd/zfd.html\n\n[<device>,<device>]\n\n-- <device> format --\n  {\n    name=<name>\n    commands=[<command>,<command>]\n  }\n--\n*name: string referring to client name of the device\n\n<command> format\n  {\n    type=<type>\n    shell=<shell command>\n    <tag>=<value>\n    <tag>=<value>\n    ...\n  }\n--\n*type: type of the signal: note/controller/pitch/system/connect/disconnect\n  > mandatory\n*shell: shell command to be executed\n  > mandatory\n\nShell commands can be concatenated with \"\" or ''\nsee --command-tags for optional command tags"
+
 #endif //HELP_H

include/system.hpp

@@ -5,10 +5,16 @@
 #define LIST_COMMAND "aseqdump -l | tail -n +2 | cut -c10-42 | tr -s ' '"
 #define LIST_EXTENDED_COMMAND "aseqdump -l | tail -n +2 | cut -c-42"
 
+#include <string>
+
 extern int announce_thread_pid;
 
+void kill_all();
+
 void device_check();
 
 void announce_loop();
 
+void filetime_loop(std::string const& filepath);
+
 #endif //SYSTEM_HPP

src/device.cpp

@@ -5,20 +5,40 @@
 #include <string.h>
 #include <signal.h>
 #include <iostream>
+
 #include <ztd/shell.hpp>
 
 #include "log.hpp"
 
 std::vector<Device*> device_list;
 
-static bool _isNum(char a)
+static std::string dequote(const std::string& in)
 {
-  return (a>='0' && a<='9');
+  std::string ret;
+  const char quote = in[0];
+  if(quote != '\'' && quote !='\"')
+    return in;
+  unsigned int i=1;
+  while(i<in.size())
+  {
+    if(in[i] == quote) {
+      break;
+    }
+    else if(i+1 < in.size() && in[i] == '\\' && in[i+1] == quote) {
+      i+=2;
+      ret += quote;
+    }
+    else{
+      ret += in[i];
+      i++;
+    }
+  }
+  return ret;
 }
 
-void sh(std::string const& string)
+static bool _isNum(const char& a)
 {
-  system(string.c_str());
+  return (a>='0' && a<='9');
 }
 
 Device::Device()
@@ -39,17 +59,16 @@ bool Device::start_loop()
   if(this->busy)
     return false;
   this->busy = true;
-  this->thread = std::thread(Device::loop, this);
-  this->thread.detach();
+  std::thread(Device::loop, this).detach();
   return true;
 }
 
-std::pair<int32_t,int32_t> importRange(ztd::chunkdat const& ch, std::string const& tag, int32_t low, int32_t high)
+std::pair<int32_t,int32_t> importRange(const ztd::chunkdat& ch, const std::string& tag, int32_t low, int32_t high)
 {
   ztd::chunkdat* pch=ch.subChunkPtr(tag);
   if(pch != nullptr)
   {
-    std::string str=pch->strval();
+    std::string str=*pch;
     auto tpos=str.find(':');
     if (str=="*") //whole range
     {
@@ -57,25 +76,25 @@ std::pair<int32_t,int32_t> importRange(ztd::chunkdat const& ch, std::string cons
     }
     else if(tpos == std::string::npos) //single value
     {
-      low=stoi(str);
+      low=std::stoi(str);
       high=low;
     }
     else //range
     {
-      low=stoi(str.substr(0,tpos));
+      low=std::stoi(str.substr(0,tpos));
       tpos++;
-      high=stoi(str.substr(tpos, str.size()-tpos));
+      high=std::stoi(str.substr(tpos, str.size()-tpos));
     }
   }
   return std::make_pair(low, high);
 }
 
-std::pair<float,float> importRangeFloat(ztd::chunkdat const& ch, std::string const& tag, float low, float high)
+std::pair<float,float> importRangeFloat(const ztd::chunkdat& ch, const std::string& tag, float low, float high)
 {
   ztd::chunkdat* pch=ch.subChunkPtr(tag);
   if(pch != nullptr)
   {
-    std::string str=pch->strval();
+    std::string str=*pch;
     auto tpos=str.find(':');
     if(tpos == std::string::npos)
     {
@@ -94,12 +113,12 @@ std::pair<float,float> importRangeFloat(ztd::chunkdat const& ch, std::string con
   return std::make_pair(low, high);
 }
 
-bool importBool(ztd::chunkdat const& ch, std::string const& tag, bool defbool)
+bool importBool(const ztd::chunkdat& ch, const std::string& tag, const bool& defbool)
 {
   ztd::chunkdat* pch=ch.subChunkPtr(tag);
   if(pch != nullptr)
   {
-    std::string str=pch->strval();
+    std::string str=*pch;
     if( str == "true" )
       return true;
     else if( str == "false" )
@@ -108,30 +127,30 @@ bool importBool(ztd::chunkdat const& ch, std::string const& tag, bool defbool)
   return defbool;
 }
 
-bool Device::import_chunk(ztd::chunkdat const& ch)
+bool Device::import_chunk(const ztd::chunkdat& ch)
 {
   ztd::chunkdat& cch = ch["commands"];
-  this->name=ch["name"].strval();
+  this->name=dequote(ch["name"]);
   for(int i=0 ; i<cch.listSize() ; i++)
   {
     ztd::chunkdat& tch=cch[i];
-    std::string tstr=tch["type"].strval();
+    std::string tstr=tch["type"];
     if(tstr == "system") //type system
     {
       std::string shell;
-      shell=tch["shell"].strval();
+      shell=tch["shell"];
       this->sysCommands.push_back(SystemCommand(shell));
     }
     else if (tstr == "connect")
     {
       std::string shell;
-      shell=tch["shell"].strval();
+      shell=tch["shell"];
       this->connectCommands.push_back(ConnectCommand(shell));
     }
     else if (tstr == "disconnect")
     {
       std::string shell;
-      shell=tch["shell"].strval();
+      shell=tch["shell"];
       this->disconnectCommands.push_back(DisconnectCommand(shell));
     }
     else
@@ -143,13 +162,13 @@ bool Device::import_chunk(ztd::chunkdat const& ch)
       std::pair<float,float> floatpair;
 
       //channel
-      if(tch.subChunkPtr("channel") == nullptr || tch["channel"].strval()=="*")
+      if(tch.subChunkPtr("channel") == nullptr || tch["channel"] == "*")
         channel=-1;
       else
-        channel=stoi(tch["channel"].strval());
+        channel=std::stoi(tch["channel"]);
 
       //shell
-      shell=tch["shell"].strval();
+      shell=tch["shell"];
 
       //type
       if(tstr == "note") //type note
@@ -158,13 +177,11 @@ bool Device::import_chunk(ztd::chunkdat const& ch)
         uint8_t id_high=127;
         uint8_t low=1;
         uint8_t high=127;
-        std::string tt;
 
         //id
         intpair=importRange(tch, "id", id_low, id_high);
         id_low=intpair.first;
         id_high=intpair.second;
-        // id=stoi(tch["id"].strval());
 
         //trigger
         intpair = importRange(tch, "trigger", low, high);
@@ -175,7 +192,6 @@ bool Device::import_chunk(ztd::chunkdat const& ch)
         {
           this->noteCommands[i].push_back(NoteCommand(channel,low,high,shell));
         }
-        this->nb_command++;
       }
       else if(tstr == "controller") //type controller
       {
@@ -209,7 +225,6 @@ bool Device::import_chunk(ztd::chunkdat const& ch)
         {
           this->ctrlCommands[i].push_back(ControllerCommand(channel,min,max,mapMin,mapMax,floating,shell));
         }
-        this->nb_command++;
       }
       else if(tstr == "pitch") //type pitch bend
       {
@@ -233,7 +248,6 @@ bool Device::import_chunk(ztd::chunkdat const& ch)
         floating = importBool(tch, "float", floating);
 
         this->pitchCommands.push_back(PitchCommand(channel,min,max,mapMin,mapMax,floating,shell));
-        this->nb_command++;
       }
       else
       {
@@ -241,6 +255,7 @@ bool Device::import_chunk(ztd::chunkdat const& ch)
         return false;
       }
     }
+    this->nb_command++;
   }
   return true;
 }
@@ -270,8 +285,8 @@ void Device::run_signal(char* buff)
       for( auto it : this->sysCommands )
       {
         std::string command="code=" + strval + ";";
-        command += it.shell;
-        std::thread(sh, command).detach();
+        command += dequote(it.shell);
+        std::thread(ztd::shr, command).detach();
       }
     }
     else
@@ -279,7 +294,7 @@ void Device::run_signal(char* buff)
       int t;
       char type;
       int8_t channel;
-      int ctid;
+      int8_t ctid=-1;
       int16_t value;
       char* pos=NULL;
       bool note_off=false;
@@ -298,7 +313,7 @@ void Device::run_signal(char* buff)
       else
       {
         // throw std::runtime_error("Unknown MIDI signal:\n" + std::string(buff));
-        printf("Unsupported signal, ignoring\n");
+        std::cerr << "Unsupported signal, ignoring\n" ;
         return;
       }
 
@@ -347,8 +362,8 @@ void Device::run_signal(char* buff)
             std::string command="id=" + std::to_string(ctid)
             + ";channel=" + std::to_string(channel)
             + ";velocity=" + std::to_string(value) + ";";
-            command += it.shell;
-            std::thread(sh, command).detach();
+            command += dequote(it.shell);
+            std::thread(ztd::shr, command).detach();
           }
         }
       }
@@ -374,8 +389,8 @@ void Device::run_signal(char* buff)
               command += std::to_string(result);
             else
               command += std::to_string((long int) result);
-            command += ";" + it.shell;
-            std::thread(sh, command).detach();
+            command += ";" + dequote(it.shell);
+            std::thread(ztd::shr, command).detach();
           }
         }
       }
@@ -400,8 +415,8 @@ void Device::run_signal(char* buff)
               command += std::to_string(result);
             else
               command += std::to_string((long int) result);
-            command += ";" + it.shell;
-            std::thread(sh, command).detach();
+            command += ";" + dequote(it.shell);
+            std::thread(ztd::shr, command).detach();
           }
         }
       } // if type
@@ -423,7 +438,7 @@ void Device::loop(Device* dev)
 
   for( auto it : dev->connectCommands )
   {
-    std::thread(sh, it.shell).detach();
+    std::thread(ztd::shr, dequote(it.shell)).detach();
   }
 
   while (getline(&buff, &buff_size, stream) > 0)
@@ -433,7 +448,7 @@ void Device::loop(Device* dev)
 
   for( auto it : dev->disconnectCommands )
   {
-    std::thread(sh, it.shell).detach();
+    std::thread(ztd::shr, dequote(it.shell)).detach();
   }
 
   log("Device '" + dev->name + "' disconnected\n");
@@ -442,3 +457,12 @@ void Device::loop(Device* dev)
   dev->thread_pid=-1;
   free(buff);
 }
+
+void clean_devices()
+{
+  for(auto it : device_list)
+  {
+    delete it;
+  }
+  device_list.clear();
+}

src/format.cpp

@@ -0,0 +1,129 @@
+#include "format.hpp"
+
+static void _skipline(const std::string& mim, unsigned int& i)
+{
+  while(i<mim.size() && mim[i] != '\n') {
+    i++;
+  }
+}
+
+static void _skipread(const std::string& mim, unsigned int& i)
+{
+  while(i<mim.size())
+  {
+    if(i+1<mim.size() && mim[i] == '/' && mim[i+1] == '/') {
+      _skipline(mim, i);
+    }
+    if(ztd::filedat::isRead(mim[i])) {
+      break;
+    }
+    i++;
+  }
+}
+
+static void _find_next(const std::string& mim, unsigned int& i, const std::string& str)
+{
+  while(i<mim.size())
+  {
+    _skipread(mim, i);
+    if(i+str.size() < mim.size() && mim.substr(i, str.size()) == str)
+    {
+      break;
+    }
+    else
+    {
+      _skipline(mim, i);
+    }
+  }
+}
+
+std::string file_strimport(const std::string& path)
+{
+  std::ifstream file(path);
+  std::string ret, line;
+  while(file)
+  {
+    getline(file, line);
+    ret += line + '\n';
+  }
+  return ret;
+}
+
+ztd::chunkdat mimtochk_commands(const std::string& mim)
+{
+  ztd::chunkdat chk;
+  unsigned int i=0,j=0;
+  _find_next(mim,i, "Command ");
+  while(i<mim.size())
+  {
+    i+=7;
+    j=i;
+    _skipline(mim,j);
+    std::string type=mim.substr(i,j-i);
+    j++;
+    i=j;
+    _find_next(mim,j, "Command ");
+    std::string flags=mim.substr(i,j-i);
+    i=j;
+
+    chk.add("{type=" + type + '\n' + flags + '}');
+  }
+  return chk;
+}
+
+ztd::chunkdat mimtochk(std::string mim)
+{
+  mim = ztd::filedat::removeComments(mim);
+  ztd::chunkdat chk;
+  unsigned int i=0,j=0;
+  _find_next(mim,i, "Device ");
+  while(i<mim.size())
+  {
+    ztd::chunkdat device;
+    i+=7;
+    j=i;
+    _skipline(mim,j);
+    std::string name=mim.substr(i,j-i);
+    j++;
+    i=j;
+    _find_next(mim,j, "Device ");
+    std::string commands=mim.substr(i,j-i);
+    i=j;
+
+    device.add("name", name);
+    device.add("commands", mimtochk_commands(commands));
+
+    chk.add(device);
+  }
+  return chk;
+}
+
+std::string chktomim(ztd::chunkdat const& chk, std::string const& aligner)
+{
+  std::string ret;
+  for(int i=0 ; i<chk.listSize() ; i++)
+  {
+    ztd::chunkdat& device=chk[i];
+    ret += "Device " + device.subChunkRef("name").str() + "\n\n";
+    for(int j=0 ; j < device["commands"].listSize() ; j++)
+    {
+      ztd::chunkdat command=device["commands"][j];
+      ret += aligner + "Command " + command["type"].str() + '\n';
+      command.erase("type");
+      for(auto it : command.getmap())
+      {
+        ret += aligner+aligner + it.first + '=' + it.second->str() + '\n';
+      }
+      ret += '\n';
+      // ret += command.str(2, aligner) + '\n';
+    }
+  }
+  return ret;
+}
+
+bool is_mim(const std::string& str)
+{
+  unsigned int i=0;
+  _skipread(str, i);
+  return str.substr(i,7) == "Device ";
+}

src/main.cpp

@@ -8,17 +8,27 @@
 #include "log.hpp"
 #include "help.h"
 
+#include "format.hpp"
+
 #include <ztd/filedat.hpp>
 #include <ztd/options.hpp>
 #include <ztd/shell.hpp>
 
+#define VERSION_STRING "v1.3.1a"
+
 ztd::option_set options;
 
 void help()
 {
-  printf("zmidimap [options] <midimap file>\n\nOptions:\n");
-  options.print_help(2, 25);
-  printf("\nSee --file-format --command-tags --shell-format options for details on map file format\n");
+  printf("zmidimap [options] <file>\n\nOptions:\n");
+  options.print_help(4, 25);
+  printf("\n");
+  printf("See --mim-format --zfd-format --command-tags --shell-format options for details on map file format\n");
+}
+
+void version()
+{
+  printf("zmidimap %s\n", VERSION_STRING);
 }
 
 void option_p(const std::string& port)
@@ -48,8 +58,7 @@ void cleanup()
 
 void stop(int ret)
 {
-  if(announce_thread_pid>0)
-    kill(announce_thread_pid, SIGINT);
+  kill_all();
   exit(ret);
 }
 
@@ -58,24 +67,104 @@ void inthandler(int dummy)
   stop(0);
 }
 
+void load_filedat(ztd::filedat& file, bool from_stdin, std::string const& path)
+{
+  if(from_stdin)
+  {
+    log("Loading map from stdin\n");
+    file.setFilePath(path);
+    std::string str=file_strimport(path);
+
+    if(options["zfd"])
+    {
+      file.data() = str;
+    }
+    else if(options["mim"])
+    {
+      file.data() = mimtochk(str);
+    }
+    else
+    {
+      if(is_mim(str))
+      {
+        file.data() = mimtochk(str);
+      }
+      else
+      {
+        file.data() = str;
+      }
+    }
+
+  }
+  else
+  {
+    file.setFilePath(path);
+    log("Loading map file '" + path + "'\n");
+    if(options["zfd"])
+    {
+      file.import_file();
+    }
+    else if(options["mim"])
+    {
+      file.data() = mimtochk(file_strimport(path));
+    }
+    else
+    {
+      std::string filestr=file_strimport(path);
+      if(is_mim(filestr))
+      {
+        file.data() = mimtochk(filestr);
+      }
+      else
+      {
+        file.import_file();
+      }
+    }
+  }
+}
+
+void load_commands(ztd::chunkdat const& data)
+{
+  clean_devices();
+  for(int i=0 ; i<data.listSize() ; i++)
+  {
+    Device *newDevice = new Device;
+    newDevice->import_chunk(data[i]);
+    device_list.push_back(newDevice);
+    log("Loaded "+std::to_string(newDevice->nb_command)+" commands for device '"+newDevice->name+"'\n");
+  }
+}
+
 int main(int argc, char* argv[])
 {
   signal(SIGINT, inthandler);
   signal(SIGCHLD, SIG_IGN); //not expecting returns from child processes
 
-  bool piped=false;
-  if (!isatty(fileno(stdin)))
-    piped = true;
+  bool autoreload=true;
+  bool from_stdin=false;
 
-  options.add(ztd::option('h',"help",        false, "Display this help message"));
-  options.add(ztd::option("file-format",     false, "Display file format help"));
-  options.add(ztd::option("command-tags",    false, "Display for command tag help"));
-  options.add(ztd::option("shell-format",    false, "Display for shell format help"));
-  options.add(ztd::option('l',"list",        false, "List detected devices"));
-  options.add(ztd::option('L',"full-list",   false, "Print whole device list details"));
-  options.add(ztd::option('p',"port",        true,  "Connect to device and output to console", "device"));
-  options.add(ztd::option("no-log",          false, "Disable console logging"));
-  // options.add(ztd::option('i',"interactive", false, "Start in interactive mode"));
+  options.add(
+    ztd::option("\r  [Help]"),
+    ztd::option('h',"help",         false, "Display this help message"),
+    ztd::option('v',"version",      false, "Display version"),
+    ztd::option("mim-format",       false, "Display mim file format help"),
+    ztd::option("zfd-format",       false, "Display zfd file format help"),
+    ztd::option("command-tags",     false, "Display for command tag help"),
+    ztd::option("shell-format",     false, "Display for shell format help"),
+    ztd::option("\r  [Format]"),
+    ztd::option("no-log",           false, "Disable console logging"),
+    ztd::option("\r  [Devices]"),
+    ztd::option('l',"list",         false, "List detected devices"),
+    ztd::option('L',"full-list",    false, "Print whole device list details"),
+    ztd::option('p',"port",         true,  "Connect to device and output to console", "device"),
+    ztd::option("\r  [Map file]"),
+    ztd::option('m',"mim",          false, "Read file in mim format"),
+    ztd::option('z',"zfd",          false, "Read file in zfd format"),
+    ztd::option('o',"output",       true,  "Output the resulting map to file", "file"),
+    ztd::option("out-zfd",          false, "Output in zfd format"),
+    ztd::option("aligner",          true,  "String to use for aligning output map format. Default \\t", "string"),
+    ztd::option("no-reload",        false, "Disable auto reloading when file changes are detected")
+  );
 
   std::vector<std::string> arg;
   try
@@ -84,112 +173,157 @@ int main(int argc, char* argv[])
   }
   catch(ztd::option_error& err)
   {
-    printf("Option error: %s\n", err.what());
+    printf("%s\n", err.what());
+    if(err.type() == ztd::option_error::unknown_option)
+      help();
     stop(1);
   }
 
   //exit options
-  ztd::option* op=nullptr;
-  op = options.find('h');
-  if( op->activated )
+  if( options['h'] )
   {
     help();
     stop(0);
   }
-  op = options.find("file-format");
-  if( op->activated )
+  if( options['v'] )
   {
-    printf("%s\n", FILE_FORMAT);
+    version();
     stop(0);
   }
-  op = options.find("command-tags");
-  if( op->activated )
+  if( options["mim-format"] )
+  {
+    printf("%s\n", MIM_FORMAT);
+    stop(0);
+  }
+  if( options["zfd-format"] )
+  {
+    printf("%s\n", ZFD_FORMAT);
+    stop(0);
+  }
+  if( options["command-tags"] )
   {
     printf("%s\n", COMMAND_TAGS);
     stop(0);
   }
-  op = options.find("shell-format");
-  if( op->activated )
+  if( options["shell-format"] )
   {
     printf("%s\n", SHELL_FORMAT);
     stop(0);
   }
-  op = options.find('h');
-  if( op->activated )
+  if( options['L'] )
   {
-    help();
+    ztd::shr("aseqdump -l");
     stop(0);
   }
-  op = options.find('L');
-  if( op->activated )
+  if( options['l'] )
   {
-    sh("aseqdump -l");
-    stop(0);
-  }
-  op = options.find('l');
-  if( op->activated )
-  {
-    sh(LIST_COMMAND);
-    stop(0);
-  }
-  op = options.find('p');
-  if( op->activated )
-  {
-    option_p(op->argument);
+    ztd::shr(LIST_COMMAND);
     stop(0);
   }
 
-  //behavioral options
-  op = options.find("no-log");
-  if( op->activated )
+  if( options['p'] )
+  {
+    option_p(options['p']);
+    stop(0);
+  }
+
+  if (options['o'])
   {
     log_on=false;
   }
 
+  //behavioral options
+  if( options["no-reload"] )
+  {
+    autoreload=false;
+  }
+  if( options["no-log"] )
+  {
+    log_on=false;
+  }
+  std::string aligner="\t";
+  if(options["aligner"])
+  {
+    aligner=options["aligner"].argument;
+  }
+
   //no argument: display help
   ztd::filedat file;
-  bool no_arg=false;
+  std::string filepath;
   if (arg.size() <= 0 || arg[0] == "")
-  {
-    no_arg=true;
-    if(!piped)
   {
     help();
     stop(0);
   }
-  }
   else
   {
-    file.setFilePath(arg[0]);
+    filepath=arg[0];
+  }
+  if(filepath == "-")
+  {
+    filepath = "/dev/stdin";
+    from_stdin = true;
   }
-
 
   //main processing
   try
   {
-    if(no_arg)
+    //load
+    load_filedat(file, from_stdin, filepath);
+    //output
+    if(options['o'])
     {
-      log("Loading map from stdin\n");
-      file.import_stdin();
-    }
+      std::string ret;
+      if(options["out-zfd"])
+        ret=file.data().str(0, aligner);
       else
-    {
-      log("Loading map file '" + arg[0] + "'\n");
-      file.import_file();
+        ret=chktomim(file.data(), aligner);
+      if(options['o'].argument == "-") {
+        std::cout << ret << std::endl;
+      }
+      else {
+        std::ofstream output(options['o']);
+        if(!output)
+        {
+          std::cerr << "Cannot write to file '" + options['o'].argument + "'\n";
+          return 1;
+        }
+        output << ret << std::endl ;
+      }
+      return 0;
     }
-
     //create commands
-    for(int i=0 ; i<file.data().listSize() ; i++)
-    {
-      Device *newDevice = new Device;
-      newDevice->import_chunk(file[i]);
-      device_list.push_back(newDevice);
-      log("Loaded "+std::to_string(newDevice->nb_command)+" commands for device '"+newDevice->name+"'\n");
-    }
+    load_commands(file.data());
 
     //main loop
     log("Starting scan for devices\n");
-    announce_loop();
+    if(autoreload)
+      std::thread(filetime_loop, filepath).detach(); // start the killer thread
+    announce_loop(); // loop until killed
+    ztd::chunkdat bak_data = file.data();
+    while(autoreload)
+    {
+      log("Reloading file\n");
+      try
+      {
+        load_filedat(file, from_stdin, filepath);
+        load_commands(file.data());
+        bak_data = file.data();
+      }
+      catch (ztd::format_error& e)
+      {
+        ztd::printFormatException(e);
+        log("Reloading old config\n");
+        load_commands(bak_data);
+      }
+      catch (std::exception& e)
+      {
+        std::cerr << "Exception: " << e.what() << std::endl;
+        log("Reloading old config\n");
+        load_commands(bak_data);
+      }
+      announce_loop(); // loop until killed
+    }
   }
   catch (ztd::format_error& e)
   {

src/system.cpp

@@ -3,6 +3,10 @@
 #include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
+#include <time.h>
+#include <unistd.h>
+#include <sys/stat.h>
+#include <signal.h>
 
 #include <string>
 #include <vector>
@@ -13,6 +17,17 @@
 
 int announce_thread_pid = 0;
 
+void kill_all()
+{
+  if(announce_thread_pid>0)
+    kill(announce_thread_pid, SIGINT);
+  for(auto it : device_list)
+  {
+    if(it->thread_pid>0)
+      kill(it->thread_pid, SIGINT);
+  }
+}
+
 void device_check()
 {
   char* buff = NULL;
@@ -86,3 +101,20 @@ void announce_loop()
   if(buff != NULL)
     free(buff);
 }
+
+void filetime_loop(std::string const& filepath)
+{
+  struct stat attrib;
+  stat(filepath.c_str(), &attrib);
+  time_t last_time=attrib.st_ctime;
+  while(true)
+  {
+    stat(filepath.c_str(), &attrib);
+    if(attrib.st_ctime > last_time)
+    {
+      kill_all();
+      last_time = attrib.st_ctime;
+    }
+    sleep(1);
+  }
+}