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

Better #include for expat.h #838

Merged
merged 1 commit into from
Feb 16, 2023
Merged

Better #include for expat.h #838

merged 1 commit into from
Feb 16, 2023

Conversation

heshpdx
Copy link
Contributor

@heshpdx heshpdx commented Feb 15, 2023

Since expat.h is in the current directory, changed angle brackets to quotes to match the best practice.
In the case when we don't have a -Isrc/simgear/xml directive, this change also removes a build failure for some compilers.

Changed angle brackets to quotes, since expat.h is in the current directory.
@codecov
Copy link

codecov bot commented Feb 15, 2023

Codecov Report

Merging #838 (0c9eec3) into master (ec99f79) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #838   +/-   ##
=======================================
  Coverage   22.37%   22.37%           
=======================================
  Files         167      167           
  Lines       19535    19535           
=======================================
  Hits         4371     4371           
  Misses      15164    15164           

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

@agodemar agodemar merged commit 66e27e6 into JSBSim-Team:master Feb 16, 2023
bcoconni pushed a commit to bcoconni/jsbsim that referenced this pull request Apr 8, 2023
Changed angle brackets to quotes, since expat.h is in the current directory.
# 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