Fix elem2D_nodes pointer association after bugfix #789
Conversation
The recent bugfix in oce_mesh.F90 correctly reduced the elem2D_nodes allocation size from myDim_elem2D+eDim_elem2D+eXDim_elem2D to myDim_elem2D. However, the pointer association in associate_mesh_ass.h was not updated to match, causing 'Target of rank remapping is too small' errors. This commit updates the pointer association to match the corrected array allocation size. Fixes the regression introduced by the mesh reading bugfix.
AI, if it was seen in debugger mode we would have found it much earlier :) |
|
Problem is that we run of CI tests in release mode. This makes them a bit faster, but also less critical. Should we change that? |
Caution, AI at work, please check carefully ;-) @dsidoren
The recent bugfix in
oce_mesh.F90correctly reduced theelem2D_nodesallocation size frommyDim_elem2D+eDim_elem2D+eXDim_elem2DtomyDim_elem2D. However, the pointer association inassociate_mesh_ass.hwas not updated to match, causing 'Target of rank remapping is too small' errors. See:/work/bb1469/a270092/runtime/awiesm3-v3.4/test_tco95_07/run_19900101-19900101/log/test_tco95_07_awiesm3_compute_20093047.logThis commit updates the pointer association to match the corrected array allocation size.
The error occurred only in Debug compile mode. Which is why it went undetected.