[MITgcm-support] Re: Reminder on TAMC-friendly programming
Jean-Michel Campin
jmc at gulf.mit.edu
Fri Jul 18 13:46:37 EDT 2003
Hi patrick,
I think generally we don't need to put comments like:
cswdblk -- add ---
LOGICAL useBulkforce
LOGICAL useThermSEAICE
cswdblk --- end add ---
COMMON /PARM_PACKAGES/
& useKPP, useGMRedi, useOBCS, useAIM, useECCO,
& useSHAP_FILT, useZONAL_FILT, useGrdchk, useFLT,
& usePTRACERS, useSBO, useSEAICE,
cswdblk -- add ---
& useThermSEAICE, useBulkforce
cswdblk --- end add ---
because
1) source files that are often modified can become rapidly full of
those comments, and not very helpfull at the end, nor very clear
when someone looks to the code.
2) concerning more independant piece of code that are added, they
often comes with a CPP_OPTION that is easy to grep through the code.
so, no real need for those comments.
3) one can easily see "who added what" with the cvs-webb option "anotate":
1.86 dimitri 637: LOGICAL useSBO
1.80 heimbach 638: LOGICAL useSEAICE
1.81 cheisey 639: cswdblk -- add ---
1.82 cheisey 640: LOGICAL useBulkforce
641: LOGICAL useThermSEAICE
1.81 cheisey 642: cswdblk --- end add ---
1.44 adcroft 643: COMMON /PARM_PACKAGES/
1.91 jmc 644: & useKPP, useGMRedi, useOBCS, useAIM, useLand, useECCO,
1.69 adcroft 645: & useSHAP_FILT, useZONAL_FILT, useGrdchk, useFLT,
1.86 dimitri 646: & usePTRACERS, useSBO, useSEAICE,
1.81 cheisey 647: cswdblk -- add ---
1.82 cheisey 648: & useThermSEAICE, useBulkforce
1.81 cheisey 649: cswdblk --- end add ---
What do people think about this ?
Cheers,
Jean-Michel
More information about the MITgcm-support
mailing list