toml v0.2.0 compatibility

[navaru]

I'm trying to fiddle online with peg.js to add support for Tables

I can't quite figure out how to properly do it, if it's not much, please have a look, maybe its something easy for you.

[BinaryMuse]

Tables are supported already (previously they were "hashes"), but not arrays of tables. I'll take a look.

[navaru]

I need it for a configuration setup, where i need to declare multiple resources, so array of tables is the feature I was aiming for.

[[resource]]
name = "file.ext"
type = "video"
path = "path/to/file"

[[resource]]
name = "file.ext"
type = "video"
path = "path/to/file"

Is it hard to add it to the current implementation?

[BinaryMuse]

Yeah, it's proving to be a bit tricky for me (note: I am no expert in parsers/compilers). Adding support to the grammar for matching the new syntax is easy enough, and compiling the basic case was easy, but the more advanced example from the TOML spec has proven more difficult.

If you check out the table-arrays branch, you'll find an implementation that works with your example above. There is a test that verifies that

[[resource]]
name = "file.ext1"
type = "video1"
path = "path/to/file1"

[[resource]]
name = "file.ext2"
type = "video2"
path = "path/to/file2"

parses to

{
  resource: [
    { name: 'file.ext1', type: 'video1', path: 'path/to/file1' },
    { name: 'file.ext2', type: 'video2', path: 'path/to/file2' }
  ]
}

I'll have to keep working on the more advanced case, but I don't think I'll finish this weekend. Perhaps this is enough to be usable to you for now; if so, you can point npm directly to commit 6b4dd00. You can see that all the other tests pass here: https://travis-ci.org/BinaryMuse/toml-node/jobs/19452878

[BinaryMuse]

I lied :) This works in v2.0.0, which has been published to npm. Please let me know if you have any problems.

[navaru]

Works great, I really appreciate your effort, thank you! :)