From 03024f3cef5d0ffa92fa7735280a243f4cc5c496 Mon Sep 17 00:00:00 2001
From: Igor Zaworski <izaworski@internships.antmicro.com>
Date: Tue, 20 Jan 2026 12:38:58 +0100
Subject: [PATCH] Better comments

Signed-off-by: Igor Zaworski <izaworski@internships.antmicro.com>
---
 include/verilated_types.h | 30 ++++++++++--------------------
 1 file changed, 10 insertions(+), 20 deletions(-)

diff --git a/include/verilated_types.h b/include/verilated_types.h
index 7670e8d16..c046536bf 100644
--- a/include/verilated_types.h
+++ b/include/verilated_types.h
@@ -1931,36 +1931,26 @@ public:
     VlClassRef(VlNull) {};
     template <typename... T_Args>
     VlClassRef(VlDeleter& deleter, T_Args&&... args)
-        // () required here to avoid narrowing conversion warnings,
-        // when a new() has an e.g. CData type and passed a 1U.
         : m_objp{new T_Class} {
+        // Instantly init the object to presevrve RAII
         m_objp->init(std::forward<T_Args>(args)...);
-        // refCountInc was moved to the constructor of T_Class
-        // to fix self references in constructor.
         m_objp->m_deleterp = &deleter;
     }
     VlClassRef(VlDeleter& deleter, T_Class&& args)
-        // () required here to avoid narrowing conversion warnings,
-        // when a new() has an e.g. CData type and passed a 1U.
+        // Move constructor
         : m_objp{new T_Class{std::forward<T_Class>(args)}} {
-        // refCountInc was moved to the constructor of T_Class
-        // to fix self references in constructor.
         m_objp->m_deleterp = &deleter;
     }
-    VlClassRef(VlDeleter& deleter, const T_Class&& args)
-        // () required here to avoid narrowing conversion warnings,
-        // when a new() has an e.g. CData type and passed a 1U.
-        : m_objp{new T_Class{std::forward<T_Class>(args)}} {
-        // refCountInc was moved to the constructor of T_Class
-        // to fix self references in constructor.
+    VlClassRef(VlDeleter& deleter, const T_Class& args)
+        // Copy constructor
+        : m_objp{new T_Class{args}} {
         m_objp->m_deleterp = &deleter;
     }
-    VlClassRef(VlDeleter& deleter, T_Class args)
-        // () required here to avoid narrowing conversion warnings,
-        // when a new() has an e.g. CData type and passed a 1U.
-        : m_objp{new T_Class{std::move(args)}} {
-        // refCountInc was moved to the constructor of T_Class
-        // to fix self references in constructor.
+    VlClassRef(VlDeleter& deleter, T_Class& args)
+        // Copy constructor - this is required since if `T_Class&`
+        // will be provided a compilator will match it to the constructor
+        // with variading template inetead of `T_Class&&`
+        : m_objp{new T_Class{args}} {
         m_objp->m_deleterp = &deleter;
     }
     // Explicit to avoid implicit conversion from 0