Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upMemory profiling needs automated testing #7255
Open
Labels
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The memory profiler currently gets no automated testing, AFAIK. This leads to cases like #7252 where it panicked in very common cases.
It's hard to test the memory profiler in any great detail because the measurements are so non-deterministic. But some basic smoke testing -- i.e. run it on a few sites and check it doesn't crash or panic -- would go a long way. I don't know how this could be made to fit into Servo's existing test infrastructure, but I'd love to hear suggestions :)