Fork gumbo-parser?

[stevecheckoway]

Google's gumbo-parser has a few bugs and it's not clear that it's actually maintained any longer. I worked around google/gumbo-parser#371 in #51 but google/gumbo-parser#375 seems potentially bad.

Two options spring to mind:

1. Incorporate an updated version of gumbo-parser directly in nokogumbo.
2. Fork gumbo-parser and point the git submodule at the fork.

In either case, fixes could be applied. I suspect 2 would be easier to do and easier to revert if gumbo starts being updated again.

[rubys]

I would accept a pull request for either. Looks like google/gumbo-parser#375 has a pull request waiting on a CLA being signed. (Additionally, it looks like there is a test failing, but I haven't looked into whether that was pre-existing or as a result of the change).

[stevecheckoway]

The failing test isn't related. The .travis.yml file contains the wrong url for gtest. I submitted a PR about that on the off chance it's actually being maintained.

[stevecheckoway]

Related bug #64.

[stevecheckoway]

Addressed by 4217876.