Subprogram awc__depth_integrated::awc_depth_integrated_read (frooting_depth): Need a LOT more data validation and guard code here to prevent execution in the case where only some data fields are found, or are only partially populated.
Subprogram crop_coefficients__fao56::crop_coefficients_fao56_initialize (): Implement thorough input error checking: are all soils in grid included in table values? is soil suffix vector continuous?
Subprogram et__jensen_haise::et_jh_calculate (idayofyear, inumdaysinyear, flatitude, ftmin, ftmax, fas, fbs, fsunpct): Check these equations against original paper.
Subprogram grid::grid_gridtopoint_int (pgrdfrom, pgrdto, icol, irow): check the logic here...intent is to ensure that the majority filter is searching an area that adequately covers corresponding gridcell areas
Subprogram grid::lu_temp: change these global (module) variables to local variables
Subprogram model_domain::model_initialize_soil_storage_max_internally_calculated (this): this should be in its own routine...
Subprogram netcdf4_support::netcdf_open_and_prepare_as_output (ncfile, svariablename, svariableunits, inx, iny, fx, fy, startdate, enddate, proj4_string, history_list, executable_name, dplat, dplon, fvalidmin, fvalidmax, write_time_bounds, filename_prefix, filename_modifier): implement more flexible method of assigning units
Subprogram netcdf4_support::nf_get_time_units (ncfile): this does not appear to have the fix that was applied to the master swb branch to deal with cases where no time values are given at all
Subprogram netcdf4_support::nf_get_time_vals (ncfile): allow time to be read in as float, short, or int as well
Module netcdf_c_api_interfaces: change "type (c_ptr), value" entries to "integer (c_size_t)" or similar, depending on the size of the integer declared in the C code
Subprogram solar_calculations::estimate_percent_of_possible_sunshine__psun (ftmax, ftmin): [Need to add reference here...]