seafoam 0.2 → 0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 841e61f7601b4121b7367e9974b9920f8b8513fd6ca7a827c1be0e340bc2b3cf
-  data.tar.gz: 559e28e6b6a49964f31ac73bc1b1ed8c72fefae9d81381bb3e433bc39d57d0d7
+  metadata.gz: 85ce198d6b6117c1e691ea871829e0d93d480ab26709f6ba494aa8f73957fefb
+  data.tar.gz: 83275a9095d3827a6f5c7d8eab9d9ea49ed35b6fe3f527d6c6299f6719a65956
 SHA512:
-  metadata.gz: 75c306c0d6b9b8dffae51f62538a6b5e529150b59e5b6ea6ccbcf1b27ca9505fc8b7548086f2826d4f9bb8589648e2281e750a2caabd7b91f7da164065756883
-  data.tar.gz: c0c307acef6945d1841cc23d2cf5c00bbc52ed70cd74aaa95fc9365f83e796379b86009a16eef522b849e6a279b147e4d8350291b5fcdbebfa81d2ef4178bed5
+  metadata.gz: d5b74e656d6a80af67233f4fcae60bbdfc291bab4a6d4a92ce283450b5dd4e318da59440a4d78d539033910cba2560068c57f112213e4ce8b493e731e92ff799
+  data.tar.gz: a7acdd55bcb41bc5f78b0f06583cb678d7e1d2beaadab02c6e2e42945fac8d6d4c4dfccddabe64c9b47f223722e7808b5fb9dd1353d4912fdbe757ece60950c5

data/.rubocop.yml

@@ -32,3 +32,6 @@ Security/Eval:
 
 Naming/FileName:
   Enabled: false
+
+Metrics/LineLength:
+  Max: 300

data/README.md

@@ -10,6 +10,7 @@ GraalVM compiler graphs. Seafoam aims to solve several problems with IGV. Unlike
 IGV, Seafoam:
 
 * is open source and can be used according to the MIT license
+* supports gzip-compressed BGV files
 * is able to some extent seek BGV files to load specific graphs without loading the rest of the file
 * has a command-line interface
 * can be used as a library
@@ -37,34 +38,34 @@ Admittedly, Seafoam does not yet have:
 ### macOS
 
 ```
-$ brew install graphviz
-$ gem install seafoam
-$ seafoam --version
+% brew install graphviz
+% gem install seafoam
+% seafoam --version
 seafoam 0.1
 ```
 
 ### Ubuntu
 
 ```
-$ sudo apt-get install ruby graphviz
-$ gem install seafoam
-$ seafoam --version
+% sudo apt-get install ruby graphviz
+% gem install seafoam
+% seafoam --version
 seafoam 0.1
 ```
 
 #### RedHat
 
 ```
-$ sudo yum install ruby graphviz
-$ gem install seafoam
-$ seafoam --version
+% sudo yum install ruby graphviz
+% gem install seafoam
+% seafoam --version
 seafoam 0.1
 ```
 
 ## Quick-start demo
 
 ```
-$ seafoam examples/fib-java.bgv:0 render
+% seafoam examples/fib-java.bgv:0 render
 ```
 
 ## Getting compiler graphs
@@ -78,14 +79,20 @@ This is just a quick summary - see more information on
 ### GraalVM for Java
 
 ```
-$ javac Fib.java
-$ java -XX:CompileOnly=::fib -Dgraal.Dump=:2 Fib 14
+% javac Fib.java
+% java -XX:CompileOnly=::fib -Dgraal.Dump=:2 Fib 14
+```
+
+### GraalVM Native Image
+
+```
+% native-image -H:Dump=:2 -H:MethodFilter=fib Fib
 ```
 
 ### TruffleRuby and other Truffle languages
 
 ```
-$ ruby --experimental-options --engine.CompileOnly=fib --engine.Inlining=false --engine.OSR=false --vm.Dgraal.Dump=Truffle:2 fib.rb 14
+% ruby --experimental-options --engine.CompileOnly=fib --engine.Inlining=false --engine.OSR=false --vm.Dgraal.Dump=Truffle:2 fib.rb 14
 ```
 
 You will usually want to look at the *After TruffleTier* graph.
@@ -105,14 +112,14 @@ Note that a *graph ID* is an ID found in BGV files, but is not unique. A
 #### Print information about a file
 
 ```
-$ seafoam examples/fib-java.bgv info
+% seafoam examples/fib-java.bgv info
 BGV 6.1
 ```
 
 #### List graphs in a file
 
 ```
-$ seafoam examples/fib-java.bgv list
+% seafoam examples/fib-java.bgv list
 examples/fib-java.bgv:0  2:Fib.fib(int)/After phase org.graalvm.compiler.java.GraphBuilderPhase
 examples/fib-java.bgv:1  2:Fib.fib(int)/After phase org.graalvm.compiler.phases.PhaseSuite
 examples/fib-java.bgv:2  2:Fib.fib(int)/After phase org.graalvm.compiler.phases.common.DeadCodeEliminationPhase
@@ -125,7 +132,7 @@ examples/fib-java.bgv:5  2:Fib.fib(int)/After phase org.graalvm.compiler.phases.
 #### Search for strings in a graph, or node or edge within a graph
 
 ```
-$ seafoam examples/fib-java.bgv:0 search Start
+% seafoam examples/fib-java.bgv:0 search Start
 examples/fib-java.bgv:0:0  ...node_class":"org.graalvm.compiler.nodes.StartNode","name_template":"Start","inputs":[...
 examples/fib-java.bgv:0:0  ...piler.nodes.StartNode","name_template":"Start","inputs":[{"direct":true,"name":"state...
 ```
@@ -133,9 +140,9 @@ examples/fib-java.bgv:0:0  ...piler.nodes.StartNode","name_template":"Start","in
 #### Print edges of a graph, or node or edge within a graph
 
 ```
-$ seafoam examples/fib-java.bgv:0 edges
+% seafoam examples/fib-java.bgv:0 edges
 22 nodes, 30 edges
-$ seafoam examples/fib-java.bgv:0:13 edges
+% seafoam examples/fib-java.bgv:0:13 edges
 Input:
   13 (Call Fib.fib) <-() 6 (Begin)
   13 (Call Fib.fib) <-() 14 (@{:declaring_class=>"Fib", :method_name=>"fib", :signature=>{:args=>["I"], :ret=>"I"}, :modifiers=>9}:13)
@@ -145,14 +152,14 @@ Output:
   13 (Call Fib.fib) ->(values) 14 (@{:declaring_class=>"Fib", :method_name=>"fib", :signature=>{:args=>["I"], :ret=>"I"}, :modifiers=>9}:13)
   13 (Call Fib.fib) ->(values) 19 (@{:declaring_class=>"Fib", :method_name=>"fib", :signature=>{:args=>["I"], :ret=>"I"}, :modifiers=>9}:19)
   13 (Call Fib.fib) ->(x) 20 (+)
-$ seafoam examples/fib-java.bgv:0:13-20 edges
+% seafoam examples/fib-java.bgv:0:13-20 edges
 13 (Call Fib.fib) ->(x) 20 (+)
 ```
 
 #### Print properties of a file, graph, or node or edge within a graph
 
 ```
-$ seafoam examples/fib-java.bgv:0 props
+% seafoam examples/fib-java.bgv:0 props
 {
   "group": [
     {
@@ -160,7 +167,7 @@ $ seafoam examples/fib-java.bgv:0 props
       "short_name": "2:Fib.fib(int)",
       "method": null,
 ...
-$ seafoam examples/fib-java.bgv:0:13 props
+% seafoam examples/fib-java.bgv:0:13 props
 {
   "nodeSourcePosition": {
     "method": {
@@ -168,7 +175,7 @@ $ seafoam examples/fib-java.bgv:0:13 props
       "method_name": "fib",
       "signature": {
 ...
-$ seafoam examples/fib-java.bgv:0:13-20 props
+% seafoam examples/fib-java.bgv:0:13-20 props
 {
   "direct": true,
   "name": "x",
@@ -181,14 +188,14 @@ $ seafoam examples/fib-java.bgv:0:13-20 props
 Render a graph as a PDF image and have it opened automatically.
 
 ```
-$ seafoam examples/fib-java.bgv:0 render
+% seafoam examples/fib-java.bgv:0 render
 ```
 
 Render a graph showing just a few nodes and those surrounding them, similar to
 the IGV feature of gradually revealing nodes.
 
 ```
-$ seafoam examples/fib-java.bgv:0 render --spotlight 13,20
+% seafoam examples/fib-java.bgv:0 render --spotlight 13,20
 ```
 
 <p>
@@ -206,7 +213,7 @@ $ seafoam examples/fib-java.bgv:0 render --spotlight 13,20
 Convert a BGV file to the Isabelle graph format.
 
 ```
-$ bgv2isabelle examples/fib-java.bgv
+% bgv2isabelle examples/fib-java.bgv
 graph0 = # 2:Fib.fib(int)/After phase org.graalvm.compiler.java.GraphBuilderPhase
  (add_node 0 StartNode [2] [8]
  (add_node 1 (ParameterNode 0) [] [2, 5, 9, 11, 14, 16]

data/bin/bgv2isabelle

@@ -21,7 +21,7 @@ require 'seafoam'
 #     empty_graph))))))))))))))"
 
 ARGV.each do |file|
-  parser = Seafoam::BGV::BGVParser.new(File.new(file))
+  parser = Seafoam::BGV::BGVParser.new(file)
   parser.read_file_header
   parser.skip_document_props
 

data/bin/bgv2json

@@ -4,7 +4,7 @@ require 'seafoam'
 require 'json'
 
 ARGV.each do |file|
-  parser = Seafoam::BGV::BGVParser.new(File.new(file))
+  parser = Seafoam::BGV::BGVParser.new(file)
   parser.read_file_header
   parser.skip_document_props
 
@@ -23,20 +23,20 @@ ARGV.each do |file|
     edges = []
 
     graph.nodes.each_value do |node|
-      nodes.push({
+      nodes.push(
         id: node.id,
         props: node.props
-      })
+      )
 
       node.outputs.each do |edge|
-        edges.push({
+        edges.push(
           from: edge.from.id,
           to: edge.to.id,
           props: edge.props
-        })
+        )
       end
     end
 
-    puts JSON.generate({name: name, props: graph.props, nodes: nodes, edges: edges})
+    puts JSON.generate(name: name, props: graph.props, nodes: nodes, edges: edges)
   end
 end

data/bin/seafoam

@@ -14,7 +14,7 @@ begin
   commands.run(*ARGV)
 rescue StandardError => e
   if $DEBUG
-    # Re-raise the exception so the user sees it, it debugging is
+    # Re-raise the exception so the user sees it, if debugging is
     # enabled (ruby -d).
     raise e
   else

data/docs/bgv.md

@@ -22,11 +22,11 @@ BGV {
   char[4] = 'BIGV'
   sint8 major
   sint8 minor
-  GroupOrGraph*
+  GroupDocumentGraph*
 }
 
-GroupOrGraph {
-  BeginGroup GroupOrGraph* CloseGroup | Graph
+GroupDocumentGraph {
+  BeginGroup GroupDocumentGraph* CloseGroup | Document | Graph
 }
 
 BeginGroup {
@@ -37,6 +37,15 @@ BeginGroup {
   Props props
 }
 
+CloseGroup {
+  sint8 token = CLOSE_GROUP
+}
+
+Document {
+  sint8 token = BEGIN_DOCUMENT
+  Pops props
+}
+
 Graph {
   sint8 token = BEGIN_GRAPH
   String format
@@ -72,10 +81,6 @@ Blocks {
   sint32[followers_count] followers
 }
 
-CloseGroup {
-  sint8 token = CLOSE_GROUP
-}
-
 Props {
   sint16 props_count
   Prop[props_count] props
@@ -278,6 +283,10 @@ In order to 'seek' BGV files we have two code paths for reading the file, one
 with loads the data and one which loads as little as possible to know just how
 much further to advance in the file.
 
+## Compression
+
+Seafoam supports compressed BGV files in the gzip file format.
+
 ## Other information
 
 `seafoam file.bgv debug` checks that a file can be parsed and prints information

data/docs/getting-graphs.md

@@ -29,6 +29,14 @@ graph without significantly effecting too much how the logic in it is compiled.
   -Dgraal.VectorizeLoops=false
 ```
 
+## GraalVM's Native Image compiler
+
+When using `native-image` you will want to use the `-H:` format.
+
+```
+% native-image -H:Dump=:2 -H:MethodFilter=fib Fib
+```
+
 ## TruffleRuby and other Truffle languages
 
 Use the same options as for GraalVM for Java, except they're prefixed with

data/docs/json.md

@@ -0,0 +1,35 @@
+# JSON Output Format
+
+The `bgv2json` command outputs files in [JSON Lines](https://jsonlines.org/) format - one JSON object for each graph, one per line.
+
+Each graph is of the format:
+
+```js
+{
+  "name": ...,
+  "props": {...},
+  "nodes": [...],
+  "edges": [...]
+}
+```
+
+Each node is of the format:
+
+```js
+{
+  "id": ...,
+  "props": {...}
+}
+```
+
+Each edge is of the format:
+
+```js
+{
+  "from": ..., // node id
+  "to": ..., // node id
+  "props": {...}
+}
+```
+
+Note that `bgv2json` runs annotations, so for example all nodes have a `"label"` property with an easy-to-use name.

data/lib/seafoam.rb

@@ -1,9 +1,9 @@
 require 'seafoam/version'
 require 'seafoam/binary/io_binary_reader'
-require 'seafoam/binary/binary_reader'
 require 'seafoam/bgv/bgv_parser'
 require 'seafoam/colors'
 require 'seafoam/graph'
+require 'seafoam/graal'
 require 'seafoam/annotators'
 require 'seafoam/annotators/graal'
 require 'seafoam/annotators/fallback'

data/lib/seafoam/bgv/bgv_parser.rb

@@ -1,3 +1,6 @@
+require 'stringio'
+require 'zlib'
+
 module Seafoam
   module BGV
     # A parser for BGV files. It's a push-pull streaming interface that you need
@@ -5,8 +8,12 @@ module Seafoam
     # and some code is duplicated in order to support skipping over parts of the
     # file that you don't need.
     class BGVParser
-      def initialize(source)
-        @reader = Binary::BinaryReader.for(source)
+      def initialize(file)
+        data = File.read(file, encoding: Encoding::ASCII_8BIT)
+        if data[0..1].bytes == [0x1f, 0x8b]
+          data = Zlib.gunzip(data)
+        end
+        @reader = Binary::IOBinaryReader.new(StringIO.new(data))
         @group_stack = []
         @pool = {}
         @index = 0
@@ -164,6 +171,9 @@ module Seafoam
             break true
           when CLOSE_GROUP
             read_close_group
+          when BEGIN_DOCUMENT
+            # But what should we do with them?
+            skip_props
           else
             raise EncodingError, "unknown token 0x#{token.to_s(16)} beginning BGV object"
           end

data/lib/seafoam/commands.rb

@@ -51,7 +51,7 @@ module Seafoam
 
       raise ArgumentError, 'info does not take arguments' unless args.empty?
 
-      parser = BGV::BGVParser.new(File.new(file))
+      parser = BGV::BGVParser.new(file)
       major, minor = parser.read_file_header(version_check: false)
       @out.puts "BGV #{major}.#{minor}"
     end
@@ -63,7 +63,7 @@ module Seafoam
 
       raise ArgumentError, 'list does not take arguments' unless args.empty?
 
-      parser = BGV::BGVParser.new(File.new(file))
+      parser = BGV::BGVParser.new(file)
       parser.read_file_header
       parser.skip_document_props
       loop do
@@ -81,7 +81,7 @@ module Seafoam
       file, graph_index, node_id, = parse_name(name)
       raise ArgumentError, 'search only works with a file or graph' if node_id
 
-      parser = BGV::BGVParser.new(File.new(file))
+      parser = BGV::BGVParser.new(file)
       parser.read_file_header
       parser.skip_document_props
       loop do
@@ -211,7 +211,7 @@ module Seafoam
           end
         end
       else
-        parser = BGV::BGVParser.new(File.new(file))
+        parser = BGV::BGVParser.new(file)
         parser.read_file_header
         document_props = parser.read_document_props
         pretty_print document_props || {}
@@ -368,7 +368,7 @@ module Seafoam
     # Reads a file and yields just the graph requested by the index - skipping
     # the rest of the file as best as possible.
     def with_graph(file, graph_index)
-      parser = BGV::BGVParser.new(File.new(file))
+      parser = BGV::BGVParser.new(file)
       parser.read_file_header
       parser.skip_document_props
       graph_found = false
@@ -418,8 +418,12 @@ module Seafoam
 
     # Parse a name like file.bgv:g:n-e to [file.bgv, g, n, e].
     def parse_name(name)
-      *pre, file, graph, node = name.split(':')
-      file = [*pre, file].join(':')
+      *pre, post = name.split('.')
+
+      file_ext, graph, node, *rest = post.split(':')
+      raise ArgumentError, "too many parts to .ext:g:n-e in #{name}" unless rest.empty?
+
+      file = [*pre, file_ext].join('.')
 
       if node
         node, edge, *rest = node.split('-')

data/lib/seafoam/version.rb

@@ -1,5 +1,5 @@
 module Seafoam
   MAJOR_VERSION = 0
-  MINOR_VERSION = 2
+  MINOR_VERSION = 3
   VERSION = "#{MAJOR_VERSION}.#{MINOR_VERSION}"
 end

data/spec/seafoam/bgv/bgv_parser_spec.rb

@@ -11,7 +11,7 @@ describe Seafoam::BGV::BGVParser do
 
   it 'can read full files' do
     Seafoam::SpecHelpers::SAMPLE_BGV.each do |file|
-      parser = Seafoam::BGV::BGVParser.new(File.new(file))
+      parser = Seafoam::BGV::BGVParser.new(file)
       parser.read_file_header
       parser.skip_document_props
       loop do
@@ -24,9 +24,22 @@ describe Seafoam::BGV::BGVParser do
     end
   end
 
+  it 'can read full gzipped files' do
+    parser = Seafoam::BGV::BGVParser.new(File.expand_path('../../../examples/fib-java.bgv.gz', __dir__))
+    parser.read_file_header
+    parser.skip_document_props
+    loop do
+      index, = parser.read_graph_preheader
+      break unless index
+
+      parser.read_graph_header
+      parser.read_graph
+    end
+  end
+
   it 'can skip full files' do
     Seafoam::SpecHelpers::SAMPLE_BGV.each do |file|
-      parser = Seafoam::BGV::BGVParser.new(File.new(file))
+      parser = Seafoam::BGV::BGVParser.new(file)
       parser.read_file_header
       parser.skip_document_props
       loop do
@@ -41,7 +54,7 @@ describe Seafoam::BGV::BGVParser do
 
   it 'can alternate skipping and reading full files' do
     Seafoam::SpecHelpers::SAMPLE_BGV.each do |file|
-      parser = Seafoam::BGV::BGVParser.new(File.new(file))
+      parser = Seafoam::BGV::BGVParser.new(file)
       parser.read_file_header
       parser.skip_document_props
       skip = false
@@ -63,36 +76,36 @@ describe Seafoam::BGV::BGVParser do
 
   describe '#read_file_header' do
     it 'produces a version' do
-      parser = Seafoam::BGV::BGVParser.new(File.new(@fib_java_bgv))
+      parser = Seafoam::BGV::BGVParser.new(@fib_java_bgv)
       expect(parser.read_file_header).to eq [6, 1]
     end
 
     it 'raises an error for files which are not BGV' do
-      parser = Seafoam::BGV::BGVParser.new(File.new(File.expand_path('fixtures/not.bgv', __dir__)))
+      parser = Seafoam::BGV::BGVParser.new(File.expand_path('fixtures/not.bgv', __dir__))
       expect { parser.read_file_header }.to raise_error(EncodingError)
     end
 
     it 'raises an error for files which are an unsupported version of BGV' do
-      parser = Seafoam::BGV::BGVParser.new(File.new(File.expand_path('fixtures/unsupported.bgv', __dir__)))
+      parser = Seafoam::BGV::BGVParser.new(File.expand_path('fixtures/unsupported.bgv', __dir__))
       expect { parser.read_file_header }.to raise_error(NotImplementedError)
     end
 
     it 'does not raise an error for an unsupported version of BGV if version_check is disabled' do
-      parser = Seafoam::BGV::BGVParser.new(File.new(File.expand_path('fixtures/unsupported.bgv', __dir__)))
+      parser = Seafoam::BGV::BGVParser.new(File.expand_path('fixtures/unsupported.bgv', __dir__))
       expect(parser.read_file_header(version_check: false)).to eq [7, 2]
     end
   end
 
   describe '#read_graph_preheader' do
     it 'produces an index and id' do
-      parser = Seafoam::BGV::BGVParser.new(File.new(@fib_java_bgv))
+      parser = Seafoam::BGV::BGVParser.new(@fib_java_bgv)
       parser.read_file_header
       parser.skip_document_props
       expect(parser.read_graph_preheader).to eq [0, 0]
     end
 
     it 'returns nil for end of file' do
-      parser = Seafoam::BGV::BGVParser.new(File.new(@fib_java_bgv))
+      parser = Seafoam::BGV::BGVParser.new(@fib_java_bgv)
       parser.read_file_header
       parser.skip_document_props
       51.times do
@@ -104,7 +117,7 @@ describe Seafoam::BGV::BGVParser do
     end
 
     it 'returns unique indicies' do
-      parser = Seafoam::BGV::BGVParser.new(File.new(@fib_java_bgv))
+      parser = Seafoam::BGV::BGVParser.new(@fib_java_bgv)
       parser.read_file_header
       parser.skip_document_props
       indicies = []
@@ -120,7 +133,7 @@ describe Seafoam::BGV::BGVParser do
 
   describe '#read_graph_header' do
     it 'produces an expected header' do
-      parser = Seafoam::BGV::BGVParser.new(File.new(@fib_java_bgv))
+      parser = Seafoam::BGV::BGVParser.new(@fib_java_bgv)
       parser.read_file_header
       parser.skip_document_props
       parser.read_graph_preheader
@@ -131,7 +144,7 @@ describe Seafoam::BGV::BGVParser do
 
   describe '#read_graph' do
     it 'produces an expected graph' do
-      parser = Seafoam::BGV::BGVParser.new(File.new(@fib_java_bgv))
+      parser = Seafoam::BGV::BGVParser.new(@fib_java_bgv)
       parser.read_file_header
       parser.skip_document_props
       parser.read_graph_preheader

data/spec/seafoam/command_spec.rb

@@ -248,5 +248,21 @@ describe Seafoam::Commands do
       expect(node).to eq 12
       expect(edge).to eq 81
     end
+
+    it 'parses a BGV file name with periods and colons' do
+      file, graph, node, edge = @commands.send(:parse_name, 'TruffleHotSpotCompilation-13029[Truffle::ThreadOperations.detect_recursion_<split-3a5973bc>].bgv:4')
+      expect(file).to eq 'TruffleHotSpotCompilation-13029[Truffle::ThreadOperations.detect_recursion_<split-3a5973bc>].bgv'
+      expect(graph).to eq 4
+      expect(node).to be_nil
+      expect(edge).to be_nil
+    end
+
+    it 'parses a BGV.gz file name with periods and colons' do
+      file, graph, node, edge = @commands.send(:parse_name, 'TruffleHotSpotCompilation-13029[Truffle::ThreadOperations.detect_recursion_<split-3a5973bc>].bgv.gz:4')
+      expect(file).to eq 'TruffleHotSpotCompilation-13029[Truffle::ThreadOperations.detect_recursion_<split-3a5973bc>].bgv.gz'
+      expect(graph).to eq 4
+      expect(node).to be_nil
+      expect(edge).to be_nil
+    end
   end
 end

data/spec/seafoam/graphviz_writer_spec.rb

@@ -10,7 +10,7 @@ describe Seafoam::GraphvizWriter do
   describe '#write_graph' do
     before :all do
       file = File.expand_path('../../examples/fib-java.bgv', __dir__)
-      parser = Seafoam::BGV::BGVParser.new(File.new(file))
+      parser = Seafoam::BGV::BGVParser.new(file)
       parser.read_file_header
       parser.read_graph_preheader
       parser.read_graph_header
@@ -29,7 +29,7 @@ describe Seafoam::GraphvizWriter do
 
     it 'writes all graphs' do
       Seafoam::SpecHelpers::SAMPLE_BGV.each do |file|
-        parser = Seafoam::BGV::BGVParser.new(File.new(file))
+        parser = Seafoam::BGV::BGVParser.new(file)
         parser.read_file_header
         parser.skip_document_props
         parser.skip_document_props

data/spec/seafoam/spec_helpers.rb

@@ -10,7 +10,7 @@ module Seafoam
 
     def self.example_graph(file, graph_index)
       file = File.expand_path("../../examples/#{file}.bgv", __dir__)
-      parser = Seafoam::BGV::BGVParser.new(File.new(file))
+      parser = Seafoam::BGV::BGVParser.new(file)
       parser.read_file_header
       parser.skip_document_props
       loop do

data/spec/seafoam/spotlight_spec.rb

@@ -5,7 +5,7 @@ require 'rspec'
 describe Seafoam::Spotlight do
   before :each do
     file = File.expand_path('../../examples/fib-java.bgv', __dir__)
-    parser = Seafoam::BGV::BGVParser.new(File.new(file))
+    parser = Seafoam::BGV::BGVParser.new(file)
     parser.read_file_header
     parser.skip_document_props
     parser.read_graph_preheader

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: seafoam
 version: !ruby/object:Gem::Version
-  version: '0.2'
+  version: '0.3'
 platform: ruby
 authors:
 - Chris Seaton
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-11-03 00:00:00.000000000 Z
+date: 2020-12-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: benchmark-ips
@@ -83,9 +83,11 @@ files:
 - docs/images/seafoam.png
 - docs/images/spotlight-igv.png
 - docs/images/spotlight-seafoam.png
+- docs/json.md
 - examples/Fib.java
 - examples/MatMult.java
 - examples/fib-java.bgv
+- examples/fib-java.bgv.gz
 - examples/fib-js.bgv
 - examples/fib-ruby.bgv
 - examples/fib.js
@@ -188,7 +190,6 @@ files:
 - lib/seafoam/annotators/fallback.rb
 - lib/seafoam/annotators/graal.rb
 - lib/seafoam/bgv/bgv_parser.rb
-- lib/seafoam/binary/binary_reader.rb
 - lib/seafoam/binary/io_binary_reader.rb
 - lib/seafoam/colors.rb
 - lib/seafoam/commands.rb

data/lib/seafoam/binary/binary_reader.rb

@@ -1,21 +0,0 @@
-require 'stringio'
-
-module Seafoam
-  module Binary
-    # Factory for binary readers based on the input.
-    module BinaryReader
-      def self.for(source)
-        case source
-        when File
-          IOBinaryReader.new(StringIO.new(File.read(source)))
-        when IO
-          IOBinaryReader.new(source)
-        when String
-          IOBinaryReader.new(StringIO.new(source))
-        else
-          raise source.class
-        end
-      end
-    end
-  end
-end