Handle object lists that start on the same line as the - #27

bluesmoon · 2011-08-02T05:32:50Z

The problem appears to be sections like this:

  - regex: '^(Opera)/(\d+)\.(\d+) \(Nintendo Wii'
    family_replacement: 'Wii'

This needs to translate into an object with keys regex and family_replacement, with the object inside a list.

The problem is that yaml.js sees a different indent for the two lines and gets confused.

This patch changes the input to look like this:

  -
    regex: '^(Opera)/(\d+)\.(\d+) \(Nintendo Wii'
    family_replacement: 'Wii'

Which yaml.js is much happier with.

We could probably do much more to make the parser more intelligent, but in this case just canonicalizing the input is simpler and may be faster.

…tand

bluesmoon · 2011-08-02T05:34:12Z

Note that this is also listed as Issue #19

tj · 2011-08-02T15:47:10Z

mind adding a test or two? thanks man!

bluesmoon added 2 commits August 1, 2011 22:23

work with object lists when object starts on same line as the -

8092566

do the replacement just once to turn the string into a form we unders…

48a87c0

…tand

Provide feedback