>>>>> "Pedro" == Pedro Alves <palves@redhat.com> writes: Pedro> It looks fine to me too. I'm a little surprised to see a parser Pedro> state object pointer being passed around instead of making the Pedro> functions be methods of the parser type, but I guess it's really Pedro> the same either way. I can fix that in a follow-up patch. Tom