Fix ParseDump with an "empty" table declaration

[danwinship]

You can have an add table with no {} clause, but ParseDump didn't handle that.

(I also removed the use of noSpaceGroup here just to make the regexp more self-documenting.)

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: danwinship

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [danwinship]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[npinaeva]

/lgtm
Not sure though how removing the noSpaceGroup makes the regexp more self-documenting.

[danwinship]

Not sure though how removing the noSpaceGroup makes the regexp more self-documenting.

It makes it more obvious that there's a () there, to make it more obvious what the match indexes below refer to

[npinaeva]

It makes it more obvious that there's a () there, to make it more obvious what the match indexes below refer to

I tried to express this with Group in the variable name, but in this small rule it's fine either way.

[danwinship]

yeah, I figured out that naming convention after chasing down the variable definition 🙂