Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: intent(inout) for close_X_dist and close_X_ind #377

Merged
merged 3 commits into from
Jul 21, 2022

Conversation

hkershaw-brown
Copy link
Member

Description:

Added intent(inout) to get_close_X_dist and get_close_X_ind array for get_close_X_caching .

Currently giving correct answers, but intent(out) is undefined
on entry to the subroutine according to Fortran standard

Fixes issue

Addition to pull request #368
original isse #364

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 not work as expected)
  • Documentation update

Documentation changes needed?

  • My change requires a change to the documentation.
    • I have updated the documentation accordingly.

Tests

Please describe any tests you ran to verify your changes.
Bitwise wrf.

Checklist for merging

  • Updated changelog entry
  • Documentation updated
  • Version tag

Testing Datasets

  • Dataset needed for testing available upon request
  • Dataset download instructions included
  • No dataset needed

Currently giving correct answers, but intent(out) is undefined
on entry to the subrouting according to Fortran standard
@hkershaw-brown hkershaw-brown requested a review from mjs2369 July 21, 2022 15:51
Copy link
Contributor

@mjs2369 mjs2369 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hkershaw-brown looks good. I went ahead and compiled and ran with debugging flags on gfortran for the previous commit that doesn't incorporate these changes, and it didn't even produce a warning for the intent(out). It's odd, but it won't matter once these changes are implemented.

@hkershaw-brown
Copy link
Member Author

Thanks Marlee, I'll get this released with Moha's bug fix #375 shortly.

If you are curious it might be worth writing a 'hello world' test with intent(out) to see what warnings you can generate.

@hkershaw-brown hkershaw-brown merged commit cc1da67 into main Jul 21, 2022
@hkershaw-brown hkershaw-brown deleted the get_close_caching_inout branch July 21, 2022 20:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants