Subprogram bfb_initialization::first_call: Remove this module variable
Subprogram dyed_channel_initialization::ntr: This is a module variable. Move this variable into the control structure.
Subprogram dyed_obcs_initialization::ntr: This is a module variable. Move this variable into the control structure.
Subprogram mom_controlled_forcing::ctrl_forcing_cs::avg_time: Needs more complete documentation.
Module mom_lateral_mixing_coeffs: Check this reference to Bob on/off paper. The resolution function used in scaling diffusivities (Hallberg, 2010) is
Module mom_mixed_layer_restrat: Explain expression for momentum mixing time-scale.
Subprogram mom_neutral_diffusion::plm_diff (nk, h, S, c_method, b_method, diff): The use of c_method to choose a scheme is inefficient and should eventually be moved up the call tree.
Subprogram mom_regularize_layers::id_clock_pass: Should these be global?
Subprogram mom_set_diffusivity::double_diffusion (tv, h, T_f, S_f, j, G, GV, US, CS, Kd_T_dd, Kd_S_dd): Find reference for NCAR tech note above.
Subprogram mom_wave_interface::dataoverrideisinitialized: Module variable! Move into a control structure.
Subprogram mom_wave_interface::datasource: Module variable! Move into a control structure.
Subprogram mom_wave_interface::la_frachbl: Module variable! Move into a control structure.
Subprogram mom_wave_interface::la_misalignment: Module variable! Move into a control structure.
Subprogram mom_wave_interface::numbands: Module variable! Move into a control structure.
Subprogram mom_wave_interface::partitionmode: Module variable! Move into a control structure.
Subprogram mom_wave_interface::surfbandfilename: Module variable! Move into a control structure.
Subprogram mom_wave_interface::testprof: These module variables need to be documented as static/private variables or moved into a control structure.
Subprogram mom_wave_interface::wavemethod: Module variable! Move into a control structure.
Subprogram ocn_cap_methods::ocn_import (x2o, ind, grid, ice_ocean_boundary, ocean_public, logunit, Eclock, c1, c2, c3, c4): Why must this
Subprogram ocn_comp_mct::ocn_final_mct (EClock, cdata_o, x2o_o, o2x_o): This needs to be done here.
Subprogram ocn_comp_mct::ocn_init_mct (EClock, cdata_o, x2o_o, o2x_o, NLFilename): Why must this be intent(inout)?
Subprogram ocn_comp_mct::ocn_run_mct (EClock, cdata_o, x2o_o, o2x_o): Why must this be intent(inout)?
Subprogram ocn_comp_mct::ocn_setgsmap_mct (mpicom_ocn, MOM_MCT_ID, gsMap_ocn, gsMap3d_ocn): Find out if we should only provide indirect indexing for ocean points and not land.
Member SZI_ (G): Explain or remove C1_, C2_ and C3_
Subprogram user_initialization::first_call: Move this module variable into a control structure.