Closes #1378. Convert many _ASSERT(.false.) to _FAIL()

[mathomp4]

Description

This is a cleanup PR. There were many instances of _ASSERT(.false., "msg") that should instead use _FAIL("msg").

Related Issue

Closes #1378

Motivation and Context

Makes the code cleaner.

How Has This Been Tested?

I'll need to test with a run of GEOSgcm. But if it compiles, it should work.

Tested and is zero-diff.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Trivial change (affects only documentation or cleanup)

Checklist:

• I have tested this change with a run of GEOSgcm (if non-trivial)
• I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
• I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

[mathomp4]

Huh. Well, something didn't like this:

Error: detected recursion whilst expanding macro "_FAIL"

But I know we use _FAIL in MAPL so, I might need @tclune level macro help.

[mathomp4]

Huh. Well, something didn't like this:

Error: detected recursion whilst expanding macro "_FAIL"

But I know we use _FAIL in MAPL so, I might need @tclune level macro help.

Wait. My fault. My sed was too aggressive!

[mathomp4]

Welp. The model builds and runs and is zero-diff. I guess this is safe.