From 942b553c4408203d47d97bd17c9aacb3d154db91 Mon Sep 17 00:00:00 2001
From: Xavier Bestel <xav@bes.tel>
Date: Thu, 31 Mar 2022 11:34:33 +0200
Subject: [PATCH] error out in case of inst-by-position (instead of
 segfaulting)

---
 example/ot_parser.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/example/ot_parser.cpp b/example/ot_parser.cpp
index abe3e3d..0e2abdc 100644
--- a/example/ot_parser.cpp
+++ b/example/ot_parser.cpp
@@ -59,10 +59,12 @@ struct OpenTimerParser : public verilog::ParserVerilogInterface {
 
     std::string pin; 
     std::string net;
-    for(size_t i=0; i<inst.net_names.size(); i++) {
-      auto &net_name = inst.net_names[i][0];
+    if(inst.pin_names.size() != 0)
+      std::cerr << "Unsupported instanciation by position for " << inst.inst_name << std::endl;
+    else
+      for(size_t i=0; i<inst.net_names.size(); i++) {
+        auto &net_name = inst.net_names[i][0];
 
-      if(i < inst.net_names.size()) {
         auto &pin_name = inst.pin_names[i];
         switch(pin_name.index()) {
           case 0: pin = std::get<0>(pin_name); break;
@@ -80,7 +82,6 @@ struct OpenTimerParser : public verilog::ParserVerilogInterface {
         g.cellpin2net.insert({pin, net});
         g.net2cellpin.insert({std::move(net), std::move(pin)});
       }
-    }
   }
 
   Module module;