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

added common READ_BDF_LINE subroutine #71

Merged
merged 2 commits into from
May 6, 2024
Merged

added common READ_BDF_LINE subroutine #71

merged 2 commits into from
May 6, 2024

Conversation

SteveDoyle2
Copy link
Collaborator

added common READ_BDF_LINE subroutine; fixes #69

  • inside function, it uppercases the line so mystran is case insensitive
  • in the one place that the subroutine isn't called to_upper_line is used

fixed stricter NEXTC.f90 check (see #57) that had issues with

  • LOAD, ...
  • FORCE, ...
    icontinue was incorrect. Simplified code added specific check to backspace to the previous line. This seems to be fixed now, but we should run all the decks. The big fix to this is seen in the "different card type" block from nextc.f90.

image

 - inside function, it uppercases the line so mystran is case insensitive
 - in the one place that the subroutine isn't called to_upper_line is used

fixed stricter NEXTC.f90 check that had issues with
 - LOAD, ...
 - FORCE, ...
icontinue was incorrect.  Simplified code added specific check to backspace to the previous line
@SteveDoyle2 SteveDoyle2 added the bug Something isn't working label Apr 12, 2024
@zchlrnr zchlrnr merged commit 8aeaf3c into main May 6, 2024
1 check passed
@SteveDoyle2 SteveDoyle2 deleted the case_insensitive branch May 6, 2024 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cards are case sensitive
3 participants