The existing rule for matching class names in the allocation simplification pass didn't handle "$" inside names. I also updated the rule to match against non-ASCII Unicode characters, although I didn't need that for graphs I was looking at. Finally, I added more context to the error message because the message that was printed looked like:
and I couldn't immediately work out what that meant.
We may want to consider adding the ability to print debug data through an environment variable or a flag to Seafoam. Relying on ruby -d is a little awkward when the primary execution mechanism is the seafoam binary.
The existing rule for matching class names in the allocation simplification pass didn't handle "$" inside names. I also updated the rule to match against non-ASCII Unicode characters, although I didn't need that for graphs I was looking at. Finally, I added more context to the error message because the message that was printed looked like:
and I couldn't immediately work out what that meant.
We may want to consider adding the ability to print debug data through an environment variable or a flag to Seafoam. Relying on
ruby -dis a little awkward when the primary execution mechanism is theseafoambinary.Fixes #67.