mc-imperial / dredd

Framework for evaluating C/C++ compiler testing tools
Apache License 2.0
11 stars 3 forks source link

Rework protobuf representation of mutation tree #281

Closed afd closed 1 month ago

afd commented 1 month ago

To avoid a restriction of the protobuf library that limits the recursion depth of a protobuf message, this change reworks the way the mutation tree is represented in protobuf format. The tree is now organised as a flat list of nodes, and the children of a node are identified via indices into this list.

Fixes #192.

afd commented 1 month ago

@JamesLee-Jones This reworking turned out to be reasonably straightforward, so - assuming that this solves the problems that you and @JonathanFoo0523 encountered - I think it's better to go this route rather than trying to upstream work-arounds to protobufs.