Parser should store from section:name:value when saving not section:name: value

Only question is, are there any unintended uses or other corrections of this feature.

Thanks. That does indeed solve it.

On the top of my head the only place I would worry about reading things twice in the current code is upon restart.

added 1 commit

3897416c - Extend parser tests for case when we return an optional string twice, these should match

I have tried restarting a run and saw no issues. Other than that I think we currently never read the same fields a second time.

Good, I've added a check to the test so that we keep this behaviour, so all looks good to go.

added 1 commit

Excellent. Thanks for solving this.

merged

mentioned in commit aabbaa0a

Activity