bnet: cleanup scope of functions and constructors

- BnetDump(const std::string&) is private because it should only by
  used by the static Create function
- deleted not used functions

core/src/lib/bnet_network_dump.cc

@@ -30,18 +30,16 @@ std::unique_ptr<BnetDump> BnetDump::Create(std::string own_qualified_name)
     std::unique_ptr<BnetDump> p;
     return p;
   } else {
-    std::unique_ptr<BnetDump> p(std::make_unique<BnetDump>(own_qualified_name));
+    // BnetDump constructor should be private
+    std::unique_ptr<BnetDump> p(new BnetDump(own_qualified_name));
     return p;
   }
 }
 
-BnetDump::BnetDump(const std::string& own_qualified_name) : BnetDump()
+BnetDump::BnetDump(const std::string& own_qualified_name)
+    : impl_(std::make_unique<BnetDumpPrivate>())
 {
   impl_->own_qualified_name_ = own_qualified_name;
-}
-
-BnetDump::BnetDump() : impl_(std::make_unique<BnetDumpPrivate>())
-{
   impl_->OpenFile();
 }
 
@@ -51,12 +49,6 @@ BnetDump::~BnetDump()
   impl_->CloseFile();
 }
 
-bool BnetDump::IsInitialized() const { return impl_->output_file_.is_open(); }
-
-void BnetDump::DisableLogging() { impl_->logging_disabled_ = true; }
-
-void BnetDump::EnableLogging() { impl_->logging_disabled_ = false; }
-
 void BnetDump::SetDestinationQualifiedName(
     std::string destination_qualified_name)
 {
@@ -83,7 +75,7 @@ bool BnetDump::EvaluateCommandLineArgs(const char* cmdline_optarg)
 
 void BnetDump::DumpMessageAndStacktraceToFile(const char* ptr, int nbytes) const
 {
-  if (!IsInitialized() || impl_->logging_disabled_) { return; }
+  if (!impl_->output_file_.is_open()) { return; }
 
   impl_->SaveAndSendMessageIfNoDestinationDefined(ptr, nbytes);
   impl_->DumpToFile(ptr, nbytes);

core/src/lib/bnet_network_dump.h

@@ -30,23 +30,21 @@ class BnetDump {
  public:
   static std::unique_ptr<BnetDump> Create(std::string own_qualified_name);
   static bool EvaluateCommandLineArgs(const char* cmdline_optarg);
-  void SetDestinationQualifiedName(std::string destination_qualified_name);
 
+  void SetDestinationQualifiedName(std::string destination_qualified_name);
   void DumpMessageAndStacktraceToFile(const char* ptr, int nbytes) const;
-  bool IsInitialized() const;
-  void DisableLogging();
-  void EnableLogging();
 
  public:
-  BnetDump(const std::string& own_qualified_name);
+  ~BnetDump();
+
   BnetDump(const BnetDump& other) = delete;
   BnetDump(const BnetDump&& other) = delete;
   BnetDump& operator=(const BnetDump& rhs) = delete;
   BnetDump& operator=(const BnetDump&& rhs) = delete;
-  ~BnetDump();
+  BnetDump() = delete;
 
  private:
-  BnetDump();
+  BnetDump(const std::string& own_qualified_name);
   std::unique_ptr<BnetDumpPrivate> impl_;
 };
 

core/src/lib/bnet_network_dump_private.h

@@ -52,7 +52,6 @@ class BnetDumpPrivate {
   std::string destination_qualified_name_;
 
   std::ofstream output_file_;
-  bool logging_disabled_ = false;
 
  private:
   void CreateAndWriteStacktraceToBuffer();