clang-tidy changes

Signed-off-by: AdityaPandeyCN <[email protected]>

Author: AdityaPandeyCN

benchmark/sam_to_ram_benchmark.cxx

@@ -10,72 +10,74 @@
 
 static void BM_SamToRamComparison(benchmark::State &state)
 {
-    int num_reads = state.range(0);
-    std::string sam_file = "test_data.sam";
-    std::string ttree_file = "output_ttree.root";
-    std::string rntuple_file = "output_rntuple.root";
-
-    GenerateSAMFile(sam_file, num_reads);
-
-    for (auto _ : state) {
-        FILE *original_stdout = stdout;
-        stdout = fopen("/dev/null", "w");
-        
-        auto ttree_start = std::chrono::high_resolution_clock::now();
-        samtoram(sam_file.c_str(), ttree_file.c_str(), true, true, true, 1, 0);
-        auto ttree_end = std::chrono::high_resolution_clock::now();
-        
-        auto rntuple_start = std::chrono::high_resolution_clock::now();
-        samtoramntuple(sam_file.c_str(), rntuple_file.c_str(), true, true, true, 505, 0);
-        auto rntuple_end = std::chrono::high_resolution_clock::now();
-        
-        fclose(stdout);
-        stdout = original_stdout;
-
-        auto ttree_time = std::chrono::duration_cast<std::chrono::milliseconds>(ttree_end - ttree_start).count();
-        auto rntuple_time = std::chrono::duration_cast<std::chrono::milliseconds>(rntuple_end - rntuple_start).count();
-        
-        double ttree_size_mb = 0, rntuple_size_mb = 0;
-        if (std::filesystem::exists(ttree_file)) {
-            auto ttree_size = std::filesystem::file_size(ttree_file);
-            ttree_size_mb = ttree_size / (1024.0 * 1024.0);
-        }
-
-        if (std::filesystem::exists(rntuple_file)) {
-            auto rntuple_size = std::filesystem::file_size(rntuple_file);
-            rntuple_size_mb = rntuple_size / (1024.0 * 1024.0);
-        }
-
-        double compression_ratio = (rntuple_size_mb > 0) ? ttree_size_mb / rntuple_size_mb : 0;
-
-        std::cout << "\n=== Results for " << num_reads << " reads ===" << std::endl;
-        std::cout << "TTree conversion time:    " << std::setw(8) << ttree_time << " ms" << std::endl;
-        std::cout << "RNTuple conversion time:  " << std::setw(8) << rntuple_time << " ms" << std::endl;
-        std::cout << "TTree file size:          " << std::setw(8) << std::fixed << std::setprecision(2) << ttree_size_mb << " MB" << std::endl;
-        std::cout << "RNTuple file size:        " << std::setw(8) << std::fixed << std::setprecision(2) << rntuple_size_mb << " MB" << std::endl;
-        std::cout << "Compression ratio:        " << std::setw(8) << std::fixed << std::setprecision(2) << compression_ratio << "x" << std::endl;
-        std::cout << "========================================" << std::endl;
-
-        std::remove(ttree_file.c_str());
-        std::remove(rntuple_file.c_str());
-    }
-
-    std::remove(sam_file.c_str());
+   int num_reads = state.range(0);
+   std::string sam_file = "test_data.sam";
+   std::string ttree_file = "output_ttree.root";
+   std::string rntuple_file = "output_rntuple.root";
+
+   GenerateSAMFile(sam_file, num_reads);
+
+   for (auto _ : state) {
+      FILE *original_stdout = stdout;
+      stdout = fopen("/dev/null", "w");
+
+      auto ttree_start = std::chrono::high_resolution_clock::now();
+      samtoram(sam_file.c_str(), ttree_file.c_str(), true, true, true, 1, 0);
+      auto ttree_end = std::chrono::high_resolution_clock::now();
+
+      auto rntuple_start = std::chrono::high_resolution_clock::now();
+      samtoramntuple(sam_file.c_str(), rntuple_file.c_str(), true, true, true, 505, 0);
+      auto rntuple_end = std::chrono::high_resolution_clock::now();
+
+      fclose(stdout);
+      stdout = original_stdout;
+
+      auto ttree_time = std::chrono::duration_cast<std::chrono::milliseconds>(ttree_end - ttree_start).count();
+      auto rntuple_time = std::chrono::duration_cast<std::chrono::milliseconds>(rntuple_end - rntuple_start).count();
+
+      double ttree_size_mb = 0, rntuple_size_mb = 0;
+      if (std::filesystem::exists(ttree_file)) {
+         auto ttree_size = std::filesystem::file_size(ttree_file);
+         ttree_size_mb = ttree_size / (1024.0 * 1024.0);
+      }
+
+      if (std::filesystem::exists(rntuple_file)) {
+         auto rntuple_size = std::filesystem::file_size(rntuple_file);
+         rntuple_size_mb = rntuple_size / (1024.0 * 1024.0);
+      }
+
+      double compression_ratio = (rntuple_size_mb > 0) ? ttree_size_mb / rntuple_size_mb : 0;
+
+      std::cout << "\n=== Results for " << num_reads << " reads ===" << std::endl;
+      std::cout << "TTree conversion time:    " << std::setw(8) << ttree_time << " ms" << std::endl;
+      std::cout << "RNTuple conversion time:  " << std::setw(8) << rntuple_time << " ms" << std::endl;
+      std::cout << "TTree file size:          " << std::setw(8) << std::fixed << std::setprecision(2) << ttree_size_mb
+                << " MB" << std::endl;
+      std::cout << "RNTuple file size:        " << std::setw(8) << std::fixed << std::setprecision(2) << rntuple_size_mb
+                << " MB" << std::endl;
+      std::cout << "Compression ratio:        " << std::setw(8) << std::fixed << std::setprecision(2)
+                << compression_ratio << "x" << std::endl;
+      std::cout << "========================================" << std::endl;
+
+      std::remove(ttree_file.c_str());
+      std::remove(rntuple_file.c_str());
+   }
+
+   std::remove(sam_file.c_str());
 }
 
 int main(int argc, char **argv)
 {
-    std::cout << "SAM to ROOT Conversion Benchmark" << std::endl;
-    std::cout << "=================================" << std::endl;
-
-    ::benchmark::RegisterBenchmark("SAM_Conversion", BM_SamToRamComparison)
-       ->Args({10000})
-       ->Args({100000})
-       ->Args({10000000})
-       ->Unit(benchmark::kMillisecond);
-
-    ::benchmark::Initialize(&argc, argv);
-    ::benchmark::RunSpecifiedBenchmarks();
-    return 0;
-}
+   std::cout << "SAM to ROOT Conversion Benchmark" << std::endl;
+   std::cout << "=================================" << std::endl;
+
+   ::benchmark::RegisterBenchmark("SAM_Conversion", BM_SamToRamComparison)
+      ->Args({10000})
+      ->Args({100000})
+      ->Args({10000000})
+      ->Unit(benchmark::kMillisecond);
 
+   ::benchmark::Initialize(&argc, argv);
+   ::benchmark::RunSpecifiedBenchmarks();
+   return 0;
+}