search

emancu / toml-rb

A parser for TOML using Citrus library.
MIT License
105 stars 39 forks source link

Fix NameError for Set #72

Closed tatzyr closed 9 years ago

tatzyr commented 9 years ago

This pull request fixes the following issue (#73):

$ ruby -r toml -e 'TOML.parse("")'
/Users/ot/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/toml-rb-0.3.7/lib/toml/parser.rb:7:in `initialize': uninitialized constant TOML::Parser::Set (NameError)
    from /Users/ot/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/toml-rb-0.3.7/lib/toml.rb:30:in `new'
    from /Users/ot/.rbenv/versions/2.2.2/lib/ruby/gems/2.2.0/gems/toml-rb-0.3.7/lib/toml.rb:30:in `parse'
    from -e:1:in `<main>'
emancu commented 9 years ago

Thank you for your report and contribution.

Your solution is great, but for this specific case, I prefer to use arrays instead of Sets. So I'm closing this PR in favor of #74. Hope you it doesn't bother you.