Skip to content
New issue

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

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

Already on GitHub? # to your account

Clarify the code intent #839

Merged
merged 1 commit into from
Feb 18, 2023
Merged

Conversation

bcoconni
Copy link
Member

Take into account @seanmcleod's comments that were made in the PR #836.

The method FGTable::GetMinValue() has been created in the process to extract the minimum temperature from StdAtmosTemperatureTable.

Co-authored-by: Sean McLeod <sean@seanmcleod.com>
@codecov
Copy link

codecov bot commented Feb 18, 2023

Codecov Report

Merging #839 (98bdcfe) into master (0c519f8) will increase coverage by 0.00%.
The diff coverage is 54.54%.

@@           Coverage Diff           @@
##           master     #839   +/-   ##
=======================================
  Coverage   22.43%   22.43%           
=======================================
  Files         167      167           
  Lines       19600    19610   +10     
=======================================
+ Hits         4397     4400    +3     
- Misses      15203    15210    +7     
Impacted Files Coverage Δ
src/math/FGTable.h 100.00% <ø> (ø)
src/models/atmosphere/FGStandardAtmosphere.cpp 53.61% <0.00%> (ø)
src/models/FGAtmosphere.cpp 49.70% <50.00%> (-4.10%) ⬇️
src/math/FGTable.cpp 95.23% <100.00%> (+0.11%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@seanmcleod
Copy link
Member

@bcoconni looks good!

@bcoconni bcoconni merged commit ebb770e into JSBSim-Team:master Feb 18, 2023
@bcoconni bcoconni deleted the bcoconni/issue815 branch February 18, 2023 17:44
@bcoconni
Copy link
Member Author

Great 👍 PR merged.

bcoconni added a commit to bcoconni/jsbsim that referenced this pull request Apr 8, 2023
Co-authored-by: Sean McLeod <sean@seanmcleod.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants