[RF][HS3] Remove dead code

- Remove the file-global 'verbose' flag in JSONFactories_HistFactory that
  was never set to true, along with its two dead diagnostic blocks.
- Remove the unused <mutex> include in JSONIO.cxx.
- Remove an unused local and two commented-out lines.

Author: guitargeek

roofit/hs3/src/JSONFactories_HistFactory.cxx

@@ -973,12 +973,6 @@ void addNormFactor(RooRealVar const *par, Sample &sample, RooWorkspace *ws)
       sample.normfactors.emplace_back(*par);
 }
 
-namespace {
-
-bool verbose = false;
-
-}
-
 struct Channel {
    std::string name;
    std::vector<Sample> samples;
@@ -1548,17 +1542,11 @@ bool tryExportHistFactory(RooJSONFactoryWSTool *tool, const std::string &pdfname
 {
    // some preliminary checks
    if (!sumpdf) {
-      if (verbose) {
-         std::cout << pdfname << " is not a sumpdf" << std::endl;
-      }
       return false;
    }
 
    for (RooAbsArg *sample : sumpdf->funcList()) {
       if (!dynamic_cast<RooProduct *>(sample) && !dynamic_cast<RooRealSumPdf *>(sample)) {
-         if (verbose)
-            std::cout << "sample " << sample->GetName() << " is no RooProduct or RooRealSumPdf in " << pdfname
-                      << std::endl;
          return false;
       }
    }

roofit/hs3/src/JSONFactories_RooFitCore.cxx

@@ -924,7 +924,6 @@ bool exportRealIntegral(RooJSONFactoryWSTool *, const RooAbsArg *func, JSONNode
    auto *integral = static_cast<const RooRealIntegral *>(func);
    elem["type"] << key;
    std::string integrand = integral->integrand().GetName();
-   // elem["integrand"] << RooJSONFactoryWSTool::sanitizeName(integrand);
    elem["integrand"] << integrand;
    if (integral->intRange()) {
       elem["domain"] << integral->intRange();

roofit/hs3/src/JSONIO.cxx

@@ -20,7 +20,6 @@
 
 #include <fstream>
 #include <iostream>
-#include <mutex>
 #include <sstream>
 
 // Include raw strings with initial export and import keys in JSON

roofit/hs3/src/RooJSONFactoryWSTool.cxx

@@ -1081,7 +1081,6 @@ std::string RooJSONFactoryWSTool::exportTransformed(const RooAbsReal *original,
  */
 void RooJSONFactoryWSTool::exportObject(RooAbsArg const &func, std::set<std::string> &exportedObjectNames)
 {
-   // const std::string name = sanitizeName(func.GetName());
    std::string name = func.GetName();
 
    // if this element was already exported, skip
@@ -1618,7 +1617,6 @@ void RooJSONFactoryWSTool::exportData(RooAbsData const &data)
    for (int i = 0; i < data.numEntries(); ++i) {
       data.get(i);
       coords.append_child().fill_seq(variables, [](auto x) { return static_cast<RooRealVar *>(x)->getVal(); });
-      std::string datasetName = data.GetName();
       if (data.isWeighted()) {
          weightVals.push_back(data.weight());
          if (data.weight() != 1.)