cosmetic & ensure

hathitrust · Oct 19, 2023 · c655ef8 · c655ef8
1 parent a4a5861
commit c655ef8
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/lib/reports/shared_print_phase_count.rb b/lib/reports/shared_print_phase_count.rb
@@ -9,6 +9,8 @@ class SharedPrintPhaseCount
     attr_reader :phase, :finder, :output
     def initialize(phase: SharedPrint::Phases.list.max)
       @phase = phase.to_i
+      # Potentially use Reports::Dynamic instead of SharedPrint::Finder
+      # (but SP::Finder does the trick just fine for now)
       @finder = SharedPrint::Finder.new(phase: [@phase])
       @output = Utils::ReportOutput.new("sp_phase#{@phase}_count")
     end
@@ -29,6 +31,7 @@ def run
         handle.puts [org, phase, organization_tally[org]].join("\t")
       end
       puts "Finished writing to #{output.file}"
+    ensure
       handle.close
     end