Skip to content

Minor error handling and code cleanup#1293

Merged
ebranlard merged 3 commits intoOpenFAST:devfrom
bjonkman:f/UpdateFVW
Oct 20, 2022
Merged

Minor error handling and code cleanup#1293
ebranlard merged 3 commits intoOpenFAST:devfrom
bjonkman:f/UpdateFVW

Conversation

@bjonkman
Copy link
Contributor

Feature or improvement description
This PR makes a few minor changes to the source code:

  • all hard-coded CHAR(10) and CHAR(13) line ending character strings are replaced with the NWTC Library parameter NewLine.
  • FVW error handling has been modified so that blank values are not considered valid numbers
  • unused subroutine parameters for FVW's ui_seg routine have been removed

Impacted areas of the software

  • Some OLAF or SubDyn output files
  • OLAF error messages if a line doesn't contain enough columns

Test results, if applicable
This does not change any test results.

@ebranlard ebranlard self-requested a review October 20, 2022 21:00
@ebranlard ebranlard self-assigned this Oct 20, 2022
@ebranlard ebranlard merged commit a449a52 into OpenFAST:dev Oct 20, 2022
@bjonkman bjonkman deleted the f/UpdateFVW branch October 24, 2022 15:19
@rafmudaf rafmudaf mentioned this pull request Oct 27, 2022
10 tasks
@andrew-platt andrew-platt mentioned this pull request Dec 19, 2022
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments