Dds dmr refactor - Githubissues

OPENDAP / libdap4

A new version of libdap that contains both DAP2 and DAP4 support

GNU Lesser General Public License v2.1

13 stars 14 forks source link

Dds dmr refactor #171

Closed jgallagher59701 closed 2 years ago

jgallagher59701 commented 2 years ago

I refactored DDS and DMR to remove old code and provide better iterator access

DDS: New way of accessing the 'vars' vector makes range-based for loops possible. I removed old code that was excluded with the pre processor.
Old code n configure.ac was removed.
Now DAP4 builds by default and the only form of the DDX that is supported is the 3.2 version
Added tests for the new DDS::variables() method that returns a const & to vars.

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

60.0% Coverage
0.0% Duplication

jgallagher59701 commented 2 years ago

Excellent! Range based subsetting wooooooo. :shipit:

Well, not exactly - support for range-base C++ for loops, not range-based DAP CEs. That will be a bot more work. But I do have a branch that as a WIP.