style: the exe/ utilities (missed in the style sweep)

Author: ronaldtse

exe/diagnose_parse_failures

@@ -23,7 +23,8 @@ fails.first(n).each do |path|
   src = File.read(path)
   lines = src.lines
   # Binary search for first failing prefix
-  lo = 1; hi = lines.size
+  lo = 1
+  hi = lines.size
   while lo < hi
     mid = (lo + hi) / 2
     snippet = lines[0...mid].join

exe/interscript

@@ -1,5 +1,5 @@
 #!/usr/bin/env ruby
-require 'interscript/command'
+require "interscript/command"
 
 if ARGV.any? && !Interscript::Command.all_tasks.key?(ARGV.first)
   ARGV.unshift :translit

exe/verify_isc_deep

@@ -3,7 +3,6 @@
 # at the rule level (from/to/constraints), not just test counts.
 
 require "interscript"
-require "set"
 require "interscript/isc"
 require "json"
 
@@ -39,7 +38,7 @@ class DeepVerifier
     end
 
     # Compare tests
-    imp_tests = Set.new(imp_data[:tests])
+    Set.new(imp_data[:tests])
     isc_tests = Set.new(isc_data[:tests])
     missing = imp_data[:tests].reject { |t| isc_tests.include?(t) }
     details << "missing #{missing.size} tests from ISC" if missing.any?
@@ -83,7 +82,7 @@ class DeepVerifier
       metadata: metadata,
       tests: tests,
       aliases: aliases,
-      rule_counts: rule_counts,
+      rule_counts: rule_counts
     }
   rescue => e
     warn "IMP fail #{path}: #{e.message[0..80]}"
@@ -108,7 +107,7 @@ class DeepVerifier
       metadata: metadata,
       tests: tests,
       aliases: aliases,
-      rule_counts: rule_counts,
+      rule_counts: rule_counts
     }
   rescue => e
     warn "ISC fail #{path}: #{e.message[0..80]}"
@@ -204,7 +203,7 @@ if $PROGRAM_NAME == __FILE__
     results.transform_values do |r|
       {
         status: r.status,
-        details: r.details,
+        details: r.details
       }
     end
   ))

exe/verify_isc_equivalence

@@ -3,7 +3,6 @@
 # Reports per-file equivalence status.
 
 require "interscript"
-require "set"
 require "interscript/isc"
 require "json"
 
@@ -31,7 +30,7 @@ class Verifier
     isc_set = Set.new(isc_data[:tests])
 
     missing_from_isc = imp_data[:tests].reject { |t| isc_set.include?(t) }
-    extra_in_isc = isc_data[:tests].reject { |t| imp_set.include?(t) }
+    isc_data[:tests].reject { |t| imp_set.include?(t) }
 
     if missing_from_isc.any?
       details << "missing #{missing_from_isc.size} tests from ISC"
@@ -55,7 +54,7 @@ class Verifier
     {
       tests_count: tests.size,
       tests: tests,
-      stages_count: dsl.stages&.size || 0,
+      stages_count: dsl.stages&.size || 0
     }
   rescue => e
     warn "IMP parse fail #{path}: #{e.message[0..80]}"
@@ -69,7 +68,7 @@ class Verifier
     {
       tests_count: doc[:tests].size,
       tests: doc[:tests].map { |t| [t[:input], t[:expected]] },
-      stages_count: doc[:stages].size,
+      stages_count: doc[:stages].size
     }
   rescue => e
     warn "ISC parse fail #{path}: #{e.message[0..80]}"
@@ -121,7 +120,7 @@ if $PROGRAM_NAME == __FILE__
         status: r.status,
         details: r.details,
         imp: r.imp_data,
-        isc: r.isc_data,
+        isc: r.isc_data
       }
     end
   ))