@nrnhines: as we started looking into legacy code & constructs, sometime it's hard / confusing to easily find which features or variables are relevant today. So we thought it would be best to delete the code parts that are not relevant today.

Do you think it's ok to remove the #else parts of the preprocessor directories mentioned above?

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.50%. Comparing base (0cdcc5c) to head (416069a).
⚠️ Report is 2875 commits behind head on master.

@@            Coverage Diff             @@
##           master    #2007      +/-   ##
==========================================
- Coverage   47.51%   47.50%   -0.01%     
==========================================
  Files         527      527              
  Lines      118886   118877       -9     
==========================================
- Hits        56483    56473      -10     
- Misses      62403    62404       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

✔️ a9953cb -> Azure artifacts URL

@nrnhines: as we started looking into legacy code & constructs, sometime it's hard / confusing to easily find which features or variables are relevant today. So we thought it would be best to delete the code parts that are not relevant today.

Do you think it's ok to remove the #else parts of the preprocessor directories mentioned above?

I have no objection to the removal.

✔️ 416069a -> Azure artifacts URL