The subroutines KPP_NonLocalTransport_temp and KPP_NonLocalTransport_saln appear to be unnecessary wrappers for KPP_NonLocalTransport. Three years ago, @mnlevy1981 did a major refactor of KPP_NonLocalTransport (commit link here) and moved the diagnostic that required C_P elsewhere. As a result, passing C_P to the call is no longer needed and may cause confusion, and the wrapper subroutines serve no additional purpose.
Proposed changes:
The subroutines
KPP_NonLocalTransport_tempandKPP_NonLocalTransport_salnappear to be unnecessary wrappers for KPP_NonLocalTransport. Three years ago, @mnlevy1981 did a major refactor ofKPP_NonLocalTransport(commit link here) and moved the diagnostic that requiredC_Pelsewhere. As a result, passingC_Pto the call is no longer needed and may cause confusion, and the wrapper subroutines serve no additional purpose.Proposed changes:
Remove KPP_NonLocalTransport_temp and KPP_NonLocalTransport_saln.
Remove C_P from the call to KPP_NonLocalTransport.