JosiahGoeman

Results 6 comments of JosiahGoeman

Gotcha. I have changed the behavior for XMLExporter to save the position in the referenced PR, so I'll need to revise that.

Wait, why did GitHub include the BufferAllocator commit from my previous pr? That commit is already in the target branch.

Yes, the files changed are correct.

Alright, I think this should be ready for review.

> > I'd guess that BinaryExporter maybe has some sort of check for that sort of dependency issue. > > The test succeeds with `XMLExporter` in v3.7.0-stable, which means there...

Looks like it was indeed that simple. And here I was thinking my test suite was so thorough. -_- I added a test for the reference-loop case.