Andy Porter

I'm getting to the point where #2413 is almost ready for review. However, it is big and one of the changes it contains is to modify ModuleManager to use a...

Actually, my simple change means that every source file is read twice in two different locations - once for the regex to check what modules it contains (in `ModuleManager`) and...

I've just refactored what I have in #2413 to remove this duplication. If you have a look at my latest changes you'll see I've altered ModuleInfo to take an optional...

If this is something that has been coming up regularly then I think it is a priority. Especially as we try to broaden the user base of PSyclone. What you...

Looking in `fparser.two.utils` it seems that we only populate an object's `item` if we still have a reader object. I think this is because, if the match recurses, we only...

I think you're confusing `items` with `item` there Rupert. `item` seems to be a separate thing.

I think the `validate` method should reject this as a character sub-string does not imply a loop?

It looks as though the extension to VariablesAccessInfo could be done as a separate PR, is that possible? I'm quite happy for PRs to add code that is, initially, only...

Integration tests were green.

I'm thinking about @hiker's point 3. above - "Comparable debugging". This would also be useful when porting a code to e.g. GPU as one often needs to work out at...