Merge pull request #10852 from The-OpenROAD-Project-staging/coverity-make-uniq-hier-name

Author: osamahammad21

src/odb/src/db/dbInsertBuffer.cpp

@@ -539,9 +539,10 @@ std::string dbInsertBuffer::makeUniqueHierName(const dbModule* module,
                                                const char* suffix) const
 {
   // insertBuffer only punches scalar hierarchy ports, never bus ports.
-  std::string scalar_base_name = replaceBracketsWithUnderscores(base_name);
-  std::string name
-      = (suffix == nullptr) ? scalar_base_name : scalar_base_name + suffix;
+  std::string name = replaceBracketsWithUnderscores(base_name);
+  if (suffix != nullptr) {
+    name += suffix;
+  }
   std::string full = block_->makeNewNetName(
       module, name.c_str(), dbNameUniquifyType::IF_NEEDED_WITH_UNDERSCORE);
   return std::string(block_->getBaseName(full.c_str()));