The call is correct! It is very likely that your merge failed or you deleted by accident the variable ncFRCid. This is the only ID variable that was retained. It is a very important one since it will allows to spread the forcing fields into different files. We cannot use the ncid in the derived-type FRC(:,:) structure because there is not association between the forcing field and the NetCDF ID of the file containing the needed field.
The SHARED variables in OpenMP parallel loops are correct in main2d/main3d. The logic in these routines changed because of the nesting structure. They are going to change even more in the future. It turns out that every compiler implements the OpenMP standard differently. I am using all the latest compilers. Sometimes, all the threads do not enter the parallel region if something is wrong in the SHARED token. I discovered this when investigation the odd behavior of the profiling clocks in shared-memory. I spent several days in the debugger looking at this.
