Fixed debug messages and removed trailing white space

[akturner]

Code cleanup and fixed messages in forcing module

Fixed debug messages with new logging system
Removed trailing whitespace

[mark-petersen]

I approve, based on visual inspection. I did not test, but this is so simple, I think if @akturner tested in MPAS-sea ice that is sufficient.

[matthewhoffman]

@akturner , this is clear to be merged. If you are comfortable doing so following the framework merge protocols, go ahead. If you're unsure, I can help you or do it.

[akturner]

@matthewhoffman: I will merge next week when I return if thats acceptable on the schedule

[matthewhoffman]

@akturner , yeah, that sounds fine. 👍

[mgduda]

I think the code changes look fine. I'm only requesting a merge message (i.e., the first comment on the PR) that describes the changes and that they are specifically for the forcing module.

[mark-petersen]

This carriage return here and on three following do's is very strange. Since you are cleaning up white space, could you change these to a single line?

[akturner]

Not sure what you mean. The do is a separate statement and here means loop to infinity. The infinite loop is broken by an exit statement

[mark-petersen]

Aha! I had no idea - somehow that slipped through the cracks of 15 years of Fortran programming. I get it now. Please disregard my comment.

[mark-petersen]

Tested by compiling develop with the ocean and atmosphere core. Merged into ocean/develop and ran nightly test suite with gnu, then intel/debug on grizzly. Passed all tests.

[mark-petersen]

There were several conflicts between this PR and #1417, mostly due to numerous white space corrections on both. I resolved the conflicts in the merge commit.