fix: DebuggerKit: Documenting contract and making sure the destructor is

virtual.

Signed-off-by: Amlal El Mahrouss <amlal@nekernel.org>

1 files changed, 6 insertions, 3 deletions

diff --git a/dev/DebuggerKit/NeKernelContract.h b/dev/DebuggerKit/NeKernelContract.h
index 098244e..d5315ef 100644
--- a/dev/DebuggerKit/NeKernelContract.h
+++ b/dev/DebuggerKit/NeKernelContract.h
@@ -6,6 +6,9 @@
 #ifndef DK_NEKERNEL_CONTRACT_H
 #define DK_NEKERNEL_CONTRACT_H
 
+/// @brief NeKernel Debugging Protocol
+/// @author Amlal El Mahrouss
+
 #ifdef DK_NEKERNEL_DEBUGGER
 
 #include <DebuggerKit/DebuggerContract.h>
@@ -24,7 +27,7 @@ namespace Detail {
 class NeKernelContract : public DebuggerContract {
  public:
   NeKernelContract();
-  ~NeKernelContract() override;
+  virtual ~NeKernelContract() override;
 
  public:
   NeKernelContract& operator=(const NeKernelContract&) = default;
@@ -40,11 +43,11 @@ class NeKernelContract : public DebuggerContract {
   bool Detach() noexcept override;
 
  private:
-  std::string m_kernel_path;
+  std::string m_kernel_path{};
   int64_t     m_socket{0};
 };
 }  // namespace DebuggerKit::NeKernel
 
 #endif  // ifdef DK_NEKERNEL_DEBUGGER
 
-#endif  // DK_NEKERNEL_CONTRACT_H
\ No newline at end of file
+#endif  // DK_NEKERNEL_CONTRACT_H