NCAR / lrose-core

Core C/C++ code for LROSE.
https://www.eol.ucar.edu/content/lidar-radar-open-software-environment
Other
92 stars 51 forks source link

Fix RadxBeamBlock compilation on OSX #68

Closed marack closed 5 years ago

marack commented 5 years ago

This change removes unnecessary code that was pulled in from Rainfields 3 when copying the beam blocking algorithm into LROSE.

In particular the metadata subsystem and related dependencies are deleted. This functionality was related to generic metadata handling in Rainfields and is not needed by RadxBeamBlock.

Also removed some unnecessary template metaprogramming which automatically selected between std::memcpy and std::copy for trivial types. This was a case of premature optimization and the std::copy version is both more correct, and compiles down to identical instructions for trivial types.

Resolves #57