gv_fsm 0.0.2 → 0.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1d793e3f46c8098f7fd5a5e9cdac035fc19ab6fb744886775dbc1e7a7757398c
-  data.tar.gz: bc16702d154fddd02a2c774b0f651ada1ece6e1e76cc9845e64dfbbb3811aba9
+  metadata.gz: 2896e7f9617c0ed1a9917e78cd947d1da5ceef2d8554727a4db268a7f70e6229
+  data.tar.gz: 63d897002c102d5291ecbfd7b4c854f8c9fc029e01155ba573d13e2043621afb
 SHA512:
-  metadata.gz: d06e75af4f4d1c3d1ebf2d80293d60cceae9a75859ddeaef11e44b02bab4829b248071805174d15f5818767f3ee04164ac573174cb39a76db55d196cf649ba11
-  data.tar.gz: bea055285d64501305796e2fb0d49d9a5e915f52facc45224c7eace99d801ebd5f4cc7b4a39e88587b262f779c250dac651beb7235826d2fb445aaca03cf2b3a
+  metadata.gz: ae5be7ee072d06983c139320227c6b234d703b2556071df085cb3f6a3365d3d1c6a8cf38f9f6daac0b866ed5b77c4726933eee69eda7aa8d9ffe17b9f2c2ca0b
+  data.tar.gz: bd1129ccf730d930b42a8ba53f83eb2adeb6d66368ebd9aab6466c69e2b951d6b461d0e04579254153d2c753c390f16f023806040615e6ac52d53f3b2f60c71a

data/bin/gv_fsm

@@ -1,11 +1,60 @@
 #!/usr/bin/env ruby
 
 require "gv_fsm"
+require 'optparse'
 
-SM = GV_FSM::FSM.new(ARGV[0])
-SM.project_name = "test project"
-SM.description = "FSM designed to test the generator"
+sm = GV_FSM::FSM.new
+
+options = {header: true, source: true}
+OptionParser.new do |parser|
+  parser.banner =<<~EOB 
+  Graphviz to Finite State Machine generator
+  Version: #{GV_FSM::VERSION}
+  See also https://github.com/pbosetti/gv_fsm
+  
+  Usage: gv_fsm [options] scheme.dot
+  EOB
+
+  parser.on("-p", "--project PROJECT_NAME",
+            "Set the project name to PROJECT_NAME") do |pn|
+    sm.project_name = pn
+  end
+
+  parser.on("-d", "--description DESCRIPTION", "Use DESCRITION string in header") do |desc|
+    sm.description = desc
+  end
+
+  parser.on("-o", "--output_file NAME", "Use NAME for generated .c and .h files") do |f|
+    sm.cname = f
+  end
+
+  parser.on("-h", "--header-only", "Only generate header file") do
+    options[:source] = false
+  end
+
+  parser.on("-s", "--source-only", "Only generate source file") do
+    options[:header] = false
+  end
+
+  parser.on("-x", "--prefix PREFIX", "Prepend PREFIX to names of generated functions and objects") do |p|
+    sm.prefix = p
+  end
+
+end.parse!
+
+raise ArgumentError, "I need the path to a Graphviz file!" unless ARGV[0]
+raise ArgumentError, "#{ARGV[0]} does not look like a Graphviz file!" unless File.extname(ARGV[0]) == ".dot"
+
+sm.parse(ARGV[0])
+
+puts "Parsed #{sm.dotfile}.\nGenerating C stub for states: #{sm.states_list.join(", ")}."
+if options[:header] then
+  sm.generate_h 
+  puts "Generated header #{sm.cname}.h"
+end
+if options[:source] then
+  sm.generate_c 
+  puts "Generated source #{sm.cname}.c"
+end
 
-SM.generate_h
-SM.generate_c
 

data/lib/gv_fsm.rb

@@ -8,17 +8,22 @@ require File.expand_path("../version.rb", __FILE__)
 
 module GV_FSM
   class FSM
-    attr_reader :states, :transitions, :dotfile
+    attr_reader :states, :transitions, :dotfile, :prefix
     attr_accessor :project_name, :description, :cname
     include GV_FSM::Templates
 
-    def initialize(filename)
-      raise ArgumentError, "File must be in .dot format" unless File.extname(filename) == ".dot"
-      @cname = File.basename(filename, ".dot")
-      parse(filename)
+    def initialize(filename = nil)
+      @prefix = ""
+      parse(filename) if filename
+    end
+
+    def prefix=(v)
+      @prefix = v + '_'
     end
 
     def parse(filename)
+      raise ArgumentError, "File must be in .dot format" unless File.extname(filename) == ".dot"
+      @cname = File.basename(filename, ".dot") unless (@cname and ! @cname.empty?)
       @dotfile = filename
       @states = []
       @transitions = []
@@ -26,11 +31,12 @@ module GV_FSM
         g.each_node do |id|
           n = g.get_node(id)
           label = n[:label].source.empty? ? "do_#{id}" : n[:label].source
-          @states << {id: id, function: label}
+          @states << {id: id, function: @prefix+label}
         end
         g.each_edge do |e|
           from = e.node_one
           to = e.node_two
+          next unless e[:label]
           case e[:label].source
           when ""
             label = nil
@@ -39,7 +45,7 @@ module GV_FSM
           else
             label = e[:label].source
           end
-          @transitions << {from: from, to: to, function: label}
+          @transitions << {from: from, to: to, function: label ? @prefix+label : nil}
         end
       end
     end
@@ -82,6 +88,15 @@ module GV_FSM
       return dest
     end
 
+    def transitions_paths
+      path = {}
+      @transitions.each do |t|
+        path[t[:function]] = [] unless path[t[:function]]
+        path[t[:function]] << {from: t[:from], to: t[:to]}
+      end
+      return path
+    end
+
     def generate_c(filename = @cname)
       File.open("#{filename}.c", "w") do |f|
         f.puts ERB.new(HEADER, 0, "<>").result(binding)

data/lib/templates.rb

@@ -2,90 +2,124 @@
 module GV_FSM
   module Templates
     HEADER =<<~EOHEADER
-      // Finite State Machine
-      // Project: <%= self.project_name or self.dotfile %>
-      // Description: <%= self.description or "<none given>" %>
-      //
-      // Generated by gv_fsm ruby gem, see https://rubygems.org/gems/gv_fsm
-      // gv_fsm version <%= GV_FSM::VERSION %>
-      // Generation date: <%= Time.now %>
-      // Generated from: <%= self.dotfile %>
-      // The finite state machine has:
-      //   <%= self.states.count %> states
-      //   <%= self.transitions.count %> transitions
+      /******************************************************************************
+      Finite State Machine
+      Project: <%= @project_name or @dotfile %>
+      Description: <%= @description or "<none given>" %>
+      
+      Generated by gv_fsm ruby gem, see https://rubygems.org/gems/gv_fsm
+      gv_fsm version <%= GV_FSM::VERSION %>
+      Generation date: <%= Time.now %>
+      Generated from: <%= @dotfile %>
+      The finite state machine has:
+        <%= @states.count %> states
+        <%= @transitions.count %> transitions
+        <%= transition_functions_list.select {|e| e != 'NULL'}.count %> transition functions
+      <% if @prefix != '' %>
+      Functions and types have been generated with prefix "<%= @prefix %>"
+      <% end %>
+      ******************************************************************************/
 
     EOHEADER
 
     HH =<<~EOH
-      #ifndef <%= self.cname.upcase %>_H
-      #define <%= self.cname.upcase %>_H
+      #ifndef <%= @cname.upcase %>_H
+      #define <%= @cname.upcase %>_H
       #include <stdlib.h>
 
+      // State data object
+      // By default set to void; override this typedef or load the proper
+      // header if you need
+      typedef void <%= @prefix %>state_data_t;
+
+      // NOTHING SHALL BE CHANGED AFTER THIS LINE!
+
       // List of states
       typedef enum {
-      <% self.states.each_with_index do |s, i| %>
-        STATE_<%= s[:id].upcase %><%= i == 0 ? " = 0" : "" %>,  
+      <% @states.each_with_index do |s, i| %>
+        <%= @prefix.upcase %>STATE_<%= s[:id].upcase %><%= i == 0 ? " = 0" : "" %>,  
       <% end %>
-        NUM_STATES,
-        NO_CHANGE
-      } state_t;
+        <%= @prefix.upcase %>NUM_STATES,
+        <%= @prefix.upcase %>NO_CHANGE
+      } <%= @prefix %>state_t;
 
-      const char *state_names[] = {<%= self.states_list.map {|sn| '"'+sn+'"'}.join(", ") %>};
+      const char *state_names[] = {<%= states_list.map {|sn| '"'+sn+'"'}.join(", ") %>};
 
       // State function and state transition prototypes
-      typedef state_t state_func_t(void *data);
-      typedef void transition_func_t(void *data);
-
-      // state functions
-      <% self.states.each do |s| %>
-      state_t <%= s[:function] %>(void *data);
-      <% end %>
+      typedef <%= @prefix %>state_t state_func_t(<%= @prefix %>state_data_t *data);
+      typedef void transition_func_t(<%= @prefix %>state_data_t *data);
 
-      // transition functions
-      <% self.transition_functions_list.each do |t| %>
-      <% next if t == "NULL" %>
-      void <%= t %>(void *data);
+      // State functions
+      <% @states.each do |s| %>
+      <%= @prefix %>state_t <%= s[:function] %>(<%= @prefix %>state_data_t *data);
       <% end %>
 
       // List of state functions
-      state_func_t *const state_table[NUM_STATES] = {
-        <%= self.state_functions_list.join(",\n  ")%>
+      state_func_t *const <%= @prefix %>state_table[<%= @prefix.upcase %>NUM_STATES] = {
+      <% @states.each do |s| %>
+        <%= s[:function] %>, // in state <%= s[:id] %>
+      <% end %>
       };
+      
+      <% if transition_functions_list.count > 0 then %>
+      // Transition functions
+      <% transition_functions_list.each do |t| %>
+      <% next if t == "NULL" %>
+      void <%= t %>(<%= @prefix %>state_data_t *data);
+      <% end %>
 
       // Table of transition functions
-      transition_func_t *const transition_table[NUM_STATES][NUM_STATES] = {
-      <% sl = self.states_list %>
-      <% fw = self.transition_functions_list.max {|a, b| a.length <=> b.length}.length %>
-      <% sw = self.states_list.max {|a, b| a.length <=> b.length}.length %>
+      transition_func_t *const <%= @prefix %>transition_table[<%= @prefix.upcase %>NUM_STATES][<%= @prefix.upcase %>NUM_STATES] = {
+      <% sl = states_list %>
+      <% fw = transition_functions_list.max {|a, b| a.length <=> b.length}.length %>
+      <% sw = states_list.max {|a, b| a.length <=> b.length}.length %>
         /* <%= "states:".ljust(sw) %>     <%= sl.map {|e| e.ljust(fw) }.join(", ") %> */
-      <% self.transitions_map.each_with_index do |l, i| %>
+      <% transitions_map.each_with_index do |l, i| %>
         /* <%= sl[i].ljust(sw) %> */ {<%= l.map {|e| e.ljust(fw)}.join(", ") %>}, 
       <% end %>
       };
+      <% else %>
+      // No transition functions
+      <% end %>
 
       // state manager
-      state_t run_state(state_t cur_state, void *data);
+      <%= @prefix %>state_t <%= @prefix %>run_state(<%= @prefix %>state_t cur_state, void *data);
       
       #endif
     EOH
 
     CC =<<~EOC
       #include <syslog.h>
-      #include "<%= self.cname %>.h"
+      #include "<%= @cname %>.h"
       
-      // State functions
-      <% dest = self.destinations.dup %>
-      <% self.states.each do |s| %>
+      <% placeholder = "Your Code Here" %>
+      // SEARCH FOR <%= placeholder %> FOR CODE INSERTION POINTS!
+
+      //  ____  _        _       
+      // / ___|| |_ __ _| |_ ___ 
+      // \\___ \\| __/ _` | __/ _ \\
+      //  ___) | || (_| | ||  __/
+      // |____/ \\__\\__,_|\\__\\___|
+      //                         
+      //   __                  _   _                 
+      //  / _|_   _ _ __   ___| |_(_) ___  _ __  ___ 
+      // | |_| | | | '_ \\ / __| __| |/ _ \\| '_ \\/ __|
+      // |  _| |_| | | | | (__| |_| | (_) | | | \\__ \\
+      // |_|  \\__,_|_| |_|\\___|\\__|_|\\___/|_| |_|___/
+      //                                             
+      <% dest = destinations.dup %>
+      <% @states.each do |s| %>
       <% stable = true if dest[s[:id]].include? s[:id] %>
-      <% dest[s[:id]].map! {|n| "STATE_"+n.upcase} %>
+      <% dest[s[:id]].map! {|n| (@prefix+"STATE_"+n).upcase} %>
       <% if dest[s[:id]].empty? or stable then
-        dest[s[:id]].unshift "NO_CHANGE"
+        dest[s[:id]].unshift @prefix.upcase+"NO_CHANGE"
       end %>
-      state_t <%= s[:function] %>(void *data) {
-        state_t next_state = <%= dest[s[:id]].first %>;
+      // Function to be executed in state <%= s[:id] %>
+      <%= @prefix %>state_t <%= s[:function] %>(<%= @prefix %>state_data_t *data) {
+        <%= @prefix %>state_t next_state = <%= dest[s[:id]].first %>;
 
         syslog(LOG_INFO, "[FSM] In state <%= s[:id] %>");
-        /* Your code here */
+        /* <%= placeholder %> */
         
         // valid return states: <%= dest[s[:id]].join(", ") %>
         switch (next_state) {
@@ -95,43 +129,76 @@ module GV_FSM
             break;
           default:
             syslog(LOG_WARNING, "[FSM] Cannot pass from <%= s[:id] %> to %s, remaining in this state", state_names[next_state]);
-            next_state = NO_CHANGE;
+            next_state = <%= @prefix.upcase %>NO_CHANGE;
         }
         return next_state;
       }
 
       <% end %>
 
-      // Transition functions
-      <% self.transition_functions_list.each do |t| %>
+      <% if transition_functions_list.count > 0 then %>
+      //  _____                    _ _   _              
+      // |_   _| __ __ _ _ __  ___(_) |_(_) ___  _ __   
+      //   | || '__/ _` | '_ \\/ __| | __| |/ _ \\| '_ \\
+      //   | || | | (_| | | | \\__ \\ | |_| | (_) | | | | 
+      //   |_||_|  \\__,_|_| |_|___/_|\\__|_|\\___/|_| |_| 
+      //                                                
+      //   __                  _   _                 
+      //  / _|_   _ _ __   ___| |_(_) ___  _ __  ___ 
+      // | |_| | | | '_ \\ / __| __| |/ _ \\| '_ \\/ __|
+      // |  _| |_| | | | | (__| |_| | (_) | | | \\__ \\
+      // |_|  \\__,_|_| |_|\\___|\\__|_|\\___/|_| |_|___/
+      //    
+                                               
+      <% transition_functions_list.each do |t| %>
       <% next if t == "NULL" %>
-      void <%= t %>(void *data) {
+      <% tpaths = transitions_paths[t] %>
+      // This function is called in <%= tpaths.count %> transition<%= tpaths.count == 1 ? '' : 's' %>:
+      <% tpaths.each_with_index do |e, i| %>
+      // <%= i+1 %>. from <%= e[:from] %> to <%= e[:to] %>
+      <% end %>
+      void <%= t %>(<%= @prefix %>state_data_t *data) {
         syslog(LOG_INFO, "[FSM] State transition <%= t %>");
-        /* Your code here */
+        /* <%= placeholder %> */
       }
 
+      <% end %>
       <% end %>
 
-      // State manager
-      state_t run_state(state_t cur_state, void *data) {
-        state_t new_state = state_table[cur_state](data);
-        transition_func_t *transition = transition_table[cur_state][new_state];
+      //  ____  _        _        
+      // / ___|| |_ __ _| |_ ___  
+      // \\___ \\| __/ _` | __/ _ \\
+      //  ___) | || (_| | ||  __/ 
+      // |____/ \\__\\__,_|\\__\\___| 
+      //                          
+      //                                              
+      //  _ __ ___   __ _ _ __   __ _  __ _  ___ _ __ 
+      // | '_ ` _ \\ / _` | '_ \\ / _` |/ _` |/ _ \\ '__|
+      // | | | | | | (_| | | | | (_| | (_| |  __/ |   
+      // |_| |_| |_|\\__,_|_| |_|\\__,_|\\__, |\\___|_|   
+      //                              |___/           
+
+      <%= @prefix %>state_t <%= @prefix %>run_state(<%= @prefix %>state_t cur_state, <%= @prefix %>state_data_t *data) {
+        <%= @prefix %>state_t new_state = <%= @prefix %>state_table[cur_state](data);
+      <% if transition_functions_list.count > 0 then %>
+        transition_func_t *transition = <%= @prefix %>transition_table[cur_state][new_state];
         if (transition)
           transition(data);
-        return new_state == NO_CHANGE ? cur_state : new_state;
+      <% end %>
+        return new_state == <%= @prefix.upcase %>NO_CHANGE ? cur_state : new_state;
       };
 
 
       #ifdef TEST_MAIN
       #include <unistd.h>
       int main() {
-        state_t cur_state = STATE_INIT;
+        <%= @prefix %>state_t cur_state = <%= @prefix.upcase %>STATE_INIT;
         openlog("SM", LOG_PID | LOG_PERROR, LOG_USER);
         syslog(LOG_INFO, "Starting SM");
         do {
-          cur_state = run_state(cur_state, NULL);
+          cur_state = <%= @prefix %>run_state(cur_state, NULL);
           sleep(1);
-        } while (cur_state != STATE_STOP);
+        } while (cur_state != <%= @prefix.upcase %>STATE_STOP);
         return 0;
       }
       #endif

data/lib/version.rb

@@ -1,3 +1,3 @@
 module GV_FSM
-  VERSION = "0.0.2"
+  VERSION = "0.2.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gv_fsm
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.2.1
 platform: ruby
 authors:
 - Paolo Bosetti
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-08-28 00:00:00.000000000 Z
+date: 2020-08-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-graphviz