Andy Porter
Andy Porter
Integration tests failed due to attempt to call `walk` on an `Extent` at line 293 of `sympy_writer.py`.
The latest NEMO code ends up with some pretty hairy array indexing and declarations due to the tiling. I've added an example to one of the tests. Integration tests are...
Integration tests were green second time around. I just need to address missing coverage.
Ready for review now. A fairly small change but ensuring that it worked when handling the declaration of members of a structure was quite difficult. One for @LonelyCat124, @hiker or...
There was an indirect coverage change, hence the new test. Ready for another look now.
This is ready for a first look now. One for @sergisiso, @hiker or @LonelyCat124. I've triggered the integration tests.
(I am wondering whether having the "force" option to InlineTrans mean "ignore CodeBlocks" is a good idea: perhaps it should be more specific such as "permit-codeblocks"?)
Note to self: there's inevitably going to be missed coverage and I need to move quite a lot of testing out of `symboltable_test` and into individual tests of the various...
The visitor approach works out very neatly but I take your point about decoupling this functionality from the Nodes themselves. There's also quite a lot to be said for keeping...
The `Node.update_symbols_from(table)` approach was almost as neat as the visitor so I've gone with it and renamed all the `relink` methods accordingly.