OBC + AGRIF in nemo_v3_2_1: requires fixes

[molines]

Hello !

As mentioned by Rachid in his mail presenting nemo_v3_2_1, mixing OBC and AGRIF is supported in this release. I should say 'almost' because there are still some fixes to do.

In this ticket I describe the 5 fixes that I found necessary:

1. obc_par.F90 : PARAMETER attribute should be restored for smooth compilation ...
2. obcdta.F90 : I suggest to use the directives AGRIF_DO_NOT_TREAT to skip the definition of zjcnes_obc, ztobc. It works and looks better than the actual workaround.
3. agrif_user.F90 : In this module lk_obc (and lk_bdy !) is set to false for the child grid. It is a very good idea that greatly simplify the code, but this should be done before the call to opa_init (hence obc_ini).
4. solmat.F90 : In the OBC test, I think that in the actual version, the case OBC + child grid is not treated.
5. traqsr.F90 : The use of agrif directives AGRIF_DO_NOT_TREAT is required to skip some arrays definition.

I have attached the 5 modified modules.

There are 2 other bugs in obc but not linked with agrif and described in next ticket !

Jean-Marc