Code review comment for lp:~fluidity-core/fluidity/fix_coord_transformations

Revision history for this message
Simon Mouradian (mouradian) wrote :

In femtools/Coordinates.F90:L887
The intent for each subroutine argument should be specified.
Perhaps it would also be a good idea to have an assertion here that the two fields are on the same function space, before looping over all the nodes?

« Back to merge proposal