fortran-lang / fortls

fortls - Fortran Language Server
https://fortls.fortran-lang.org/
MIT License
257 stars 41 forks source link

Fix nested pre-processor blocks #405

Closed albertziegenhagel closed 5 months ago

albertziegenhagel commented 5 months ago

In the pre-processor code, when we check whether a #define or #include statement should be evaluated, only the top of the pre-processor stack was taken into account. This commit changes it so that all the previously taken branches need to evaluate to true.

Fixes #404.

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.24%. Comparing base (81a33a1) to head (7c24d0c).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #405 +/- ## ======================================= Coverage 88.23% 88.24% ======================================= Files 35 35 Lines 4795 4796 +1 ======================================= + Hits 4231 4232 +1 Misses 564 564 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.