odb: fix codegen check for _name to name_

maliberty · maliberty · commit 022b25d6d119 · 2025-12-19T07:06:58.000Z
Signed-off-by: Matt Liberty &lt;mliberty@precisioninno.com&gt;
diff --git a/src/odb/src/codeGenerator/gen.py b/src/odb/src/codeGenerator/gen.py
@@ -202,7 +202,7 @@ def add_field_attributes(field, klass, flags_struct, schema):
 
     # For fields that we need to free/destroy in the destructor
     if (
-        field["name"] == "_name"
+        field["name"] == "name_"
         and "no-destruct" not in field["flags"]
         or "table" in field
     ):
diff --git a/src/odb/src/db/dbChipConn.cpp b/src/odb/src/db/dbChipConn.cpp
@@ -84,6 +84,10 @@ void _dbChipConn::collectMemInfo(MemInfo& info)
   info.size += sizeof(*this);
 }
 
+_dbChipConn::~_dbChipConn()
+{
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbChipConn - Methods
diff --git a/src/odb/src/db/dbChipConn.h b/src/odb/src/db/dbChipConn.h
@@ -24,6 +24,8 @@ class _dbChipConn : public _dbObject
  public:
   _dbChipConn(_dbDatabase*);
 
+  ~_dbChipConn();
+
   bool operator==(const _dbChipConn& rhs) const;
   bool operator!=(const _dbChipConn& rhs) const { return !operator==(rhs); }
   bool operator<(const _dbChipConn& rhs) const;
diff --git a/src/odb/src/db/dbChipInst.cpp b/src/odb/src/db/dbChipInst.cpp
@@ -93,6 +93,10 @@ void _dbChipInst::collectMemInfo(MemInfo& info)
   info.size += sizeof(*this);
 }
 
+_dbChipInst::~_dbChipInst()
+{
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbChipInst - Methods
diff --git a/src/odb/src/db/dbChipInst.h b/src/odb/src/db/dbChipInst.h
@@ -28,6 +28,8 @@ class _dbChipInst : public _dbObject
  public:
   _dbChipInst(_dbDatabase*);
 
+  ~_dbChipInst();
+
   bool operator==(const _dbChipInst& rhs) const;
   bool operator!=(const _dbChipInst& rhs) const { return !operator==(rhs); }
   bool operator<(const _dbChipInst& rhs) const;
diff --git a/src/odb/src/db/dbChipNet.cpp b/src/odb/src/db/dbChipNet.cpp
@@ -67,6 +67,10 @@ void _dbChipNet::collectMemInfo(MemInfo& info)
   info.size += sizeof(*this);
 }
 
+_dbChipNet::~_dbChipNet()
+{
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbChipNet - Methods
diff --git a/src/odb/src/db/dbChipNet.h b/src/odb/src/db/dbChipNet.h
@@ -27,6 +27,8 @@ class _dbChipNet : public _dbObject
  public:
   _dbChipNet(_dbDatabase*);
 
+  ~_dbChipNet();
+
   bool operator==(const _dbChipNet& rhs) const;
   bool operator!=(const _dbChipNet& rhs) const { return !operator==(rhs); }
   bool operator<(const _dbChipNet& rhs) const;
diff --git a/src/odb/src/db/dbGroup.cpp b/src/odb/src/db/dbGroup.cpp
@@ -150,6 +150,13 @@ void _dbGroup::collectMemInfo(MemInfo& info)
   // User Code End collectMemInfo
 }
 
+_dbGroup::~_dbGroup()
+{
+  if (name_) {
+    free((void*) name_);
+  }
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbGroup - Methods
diff --git a/src/odb/src/db/dbGroup.h b/src/odb/src/db/dbGroup.h
@@ -32,6 +32,8 @@ class _dbGroup : public _dbObject
  public:
   _dbGroup(_dbDatabase*);
 
+  ~_dbGroup();
+
   bool operator==(const _dbGroup& rhs) const;
   bool operator!=(const _dbGroup& rhs) const { return !operator==(rhs); }
   bool operator<(const _dbGroup& rhs) const;
diff --git a/src/odb/src/db/dbIsolation.cpp b/src/odb/src/db/dbIsolation.cpp
@@ -105,6 +105,13 @@ void _dbIsolation::collectMemInfo(MemInfo& info)
   // User Code End collectMemInfo
 }
 
+_dbIsolation::~_dbIsolation()
+{
+  if (name_) {
+    free((void*) name_);
+  }
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbIsolation - Methods
diff --git a/src/odb/src/db/dbIsolation.h b/src/odb/src/db/dbIsolation.h
@@ -22,6 +22,8 @@ class _dbIsolation : public _dbObject
  public:
   _dbIsolation(_dbDatabase*);
 
+  ~_dbIsolation();
+
   bool operator==(const _dbIsolation& rhs) const;
   bool operator!=(const _dbIsolation& rhs) const { return !operator==(rhs); }
   bool operator<(const _dbIsolation& rhs) const;
diff --git a/src/odb/src/db/dbLevelShifter.cpp b/src/odb/src/db/dbLevelShifter.cpp
@@ -195,6 +195,13 @@ void _dbLevelShifter::collectMemInfo(MemInfo& info)
   // User Code End collectMemInfo
 }
 
+_dbLevelShifter::~_dbLevelShifter()
+{
+  if (name_) {
+    free((void*) name_);
+  }
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbLevelShifter - Methods
diff --git a/src/odb/src/db/dbLevelShifter.h b/src/odb/src/db/dbLevelShifter.h
@@ -22,6 +22,8 @@ class _dbLevelShifter : public _dbObject
  public:
   _dbLevelShifter(_dbDatabase*);
 
+  ~_dbLevelShifter();
+
   bool operator==(const _dbLevelShifter& rhs) const;
   bool operator!=(const _dbLevelShifter& rhs) const { return !operator==(rhs); }
   bool operator<(const _dbLevelShifter& rhs) const;
diff --git a/src/odb/src/db/dbLogicPort.cpp b/src/odb/src/db/dbLogicPort.cpp
@@ -74,6 +74,13 @@ void _dbLogicPort::collectMemInfo(MemInfo& info)
   // User Code End collectMemInfo
 }
 
+_dbLogicPort::~_dbLogicPort()
+{
+  if (name_) {
+    free((void*) name_);
+  }
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbLogicPort - Methods
diff --git a/src/odb/src/db/dbLogicPort.h b/src/odb/src/db/dbLogicPort.h
@@ -21,6 +21,8 @@ class _dbLogicPort : public _dbObject
  public:
   _dbLogicPort(_dbDatabase*);
 
+  ~_dbLogicPort();
+
   bool operator==(const _dbLogicPort& rhs) const;
   bool operator!=(const _dbLogicPort& rhs) const { return !operator==(rhs); }
   bool operator<(const _dbLogicPort& rhs) const;
diff --git a/src/odb/src/db/dbModInst.cpp b/src/odb/src/db/dbModInst.cpp
@@ -140,6 +140,13 @@ void _dbModInst::collectMemInfo(MemInfo& info)
   // User Code End collectMemInfo
 }
 
+_dbModInst::~_dbModInst()
+{
+  if (name_) {
+    free((void*) name_);
+  }
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbModInst - Methods
diff --git a/src/odb/src/db/dbModInst.h b/src/odb/src/db/dbModInst.h
@@ -26,6 +26,8 @@ class _dbModInst : public _dbObject
  public:
   _dbModInst(_dbDatabase*);
 
+  ~_dbModInst();
+
   bool operator==(const _dbModInst& rhs) const;
   bool operator!=(const _dbModInst& rhs) const { return !operator==(rhs); }
   bool operator<(const _dbModInst& rhs) const;
diff --git a/src/odb/src/db/dbModNet.cpp b/src/odb/src/db/dbModNet.cpp
@@ -140,6 +140,13 @@ void _dbModNet::collectMemInfo(MemInfo& info)
   // User Code End collectMemInfo
 }
 
+_dbModNet::~_dbModNet()
+{
+  if (name_) {
+    free((void*) name_);
+  }
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbModNet - Methods
diff --git a/src/odb/src/db/dbModNet.h b/src/odb/src/db/dbModNet.h
@@ -24,6 +24,8 @@ class _dbModNet : public _dbObject
  public:
   _dbModNet(_dbDatabase*);
 
+  ~_dbModNet();
+
   bool operator==(const _dbModNet& rhs) const;
   bool operator!=(const _dbModNet& rhs) const { return !operator==(rhs); }
   bool operator<(const _dbModNet& rhs) const;
diff --git a/src/odb/src/db/dbPowerDomain.cpp b/src/odb/src/db/dbPowerDomain.cpp
@@ -124,6 +124,13 @@ void _dbPowerDomain::collectMemInfo(MemInfo& info)
   // User Code End collectMemInfo
 }
 
+_dbPowerDomain::~_dbPowerDomain()
+{
+  if (name_) {
+    free((void*) name_);
+  }
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbPowerDomain - Methods
diff --git a/src/odb/src/db/dbPowerDomain.h b/src/odb/src/db/dbPowerDomain.h
@@ -26,6 +26,8 @@ class _dbPowerDomain : public _dbObject
  public:
   _dbPowerDomain(_dbDatabase*);
 
+  ~_dbPowerDomain();
+
   bool operator==(const _dbPowerDomain& rhs) const;
   bool operator!=(const _dbPowerDomain& rhs) const { return !operator==(rhs); }
   bool operator<(const _dbPowerDomain& rhs) const;
diff --git a/src/odb/src/db/dbPowerSwitch.cpp b/src/odb/src/db/dbPowerSwitch.cpp
@@ -190,6 +190,13 @@ void _dbPowerSwitch::collectMemInfo(MemInfo& info)
   // User Code End collectMemInfo
 }
 
+_dbPowerSwitch::~_dbPowerSwitch()
+{
+  if (name_) {
+    free((void*) name_);
+  }
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbPowerSwitch - Methods
diff --git a/src/odb/src/db/dbPowerSwitch.h b/src/odb/src/db/dbPowerSwitch.h
@@ -28,6 +28,8 @@ class _dbPowerSwitch : public _dbObject
  public:
   _dbPowerSwitch(_dbDatabase*);
 
+  ~_dbPowerSwitch();
+
   bool operator==(const _dbPowerSwitch& rhs) const;
   bool operator!=(const _dbPowerSwitch& rhs) const { return !operator==(rhs); }
   bool operator<(const _dbPowerSwitch& rhs) const;
diff --git a/src/odb/src/db/dbTechLayerCutClassRule.cpp b/src/odb/src/db/dbTechLayerCutClassRule.cpp
@@ -101,6 +101,13 @@ void _dbTechLayerCutClassRule::collectMemInfo(MemInfo& info)
   // User Code End collectMemInfo
 }
 
+_dbTechLayerCutClassRule::~_dbTechLayerCutClassRule()
+{
+  if (name_) {
+    free((void*) name_);
+  }
+}
+
 ////////////////////////////////////////////////////////////////////
 //
 // dbTechLayerCutClassRule - Methods
diff --git a/src/odb/src/db/dbTechLayerCutClassRule.h b/src/odb/src/db/dbTechLayerCutClassRule.h
@@ -25,6 +25,8 @@ class _dbTechLayerCutClassRule : public _dbObject
  public:
   _dbTechLayerCutClassRule(_dbDatabase*);
 
+  ~_dbTechLayerCutClassRule();
+
   bool operator==(const _dbTechLayerCutClassRule& rhs) const;
   bool operator!=(const _dbTechLayerCutClassRule& rhs) const
   {
