put contents of netcdf_units into NetCDFUnits namespace

src/NETCDF/dump_netcdf.cpp

@@ -44,6 +44,8 @@
 
 using namespace LAMMPS_NS;
 using namespace MathConst;
+using NetCDFUnits::Quantity;
+using NetCDFUnits::get_unit_for;
 
 static const char NC_FRAME_STR[]         = "frame";
 static const char NC_SPATIAL_STR[]       = "spatial";

src/NETCDF/dump_netcdf.h

@@ -29,7 +29,7 @@ DumpStyle(netcdf,DumpNetCDF);
 #include "dump_custom.h"
 
 namespace LAMMPS_NS {
-enum class Quantity;
+namespace NetCDFUnits {enum Quantity : int;} // forward declaration
 
 const int NC_FIELD_NAME_MAX = 100;
 const int DUMP_NC_MAX_DIMS = 100;
@@ -47,7 +47,7 @@ class DumpNetCDF : public DumpCustom {
     int field[DUMP_NC_MAX_DIMS];     // field indices corresponding to the dim.
     char name[NC_FIELD_NAME_MAX];    // field name
     int var;                         // NetCDF variable
-    Quantity quantity;               // type of the quantity
+    NetCDFUnits::Quantity quantity;  // type of the quantity
 
     bool constant;    // is this property per file (not per frame)
     int ndumped;      // number of enties written for this prop.

src/NETCDF/dump_netcdf_mpiio.cpp

@@ -44,6 +44,8 @@
 
 using namespace LAMMPS_NS;
 using namespace MathConst;
+using NetCDFUnits::Quantity;
+using NetCDFUnits::get_unit_for;
 
 static const char NC_FRAME_STR[]         = "frame";
 static const char NC_SPATIAL_STR[]       = "spatial";

src/NETCDF/dump_netcdf_mpiio.h

@@ -29,7 +29,7 @@ DumpStyle(netcdf/mpiio,DumpNetCDFMPIIO);
 #include "dump_custom.h"
 
 namespace LAMMPS_NS {
-enum class Quantity;
+namespace NetCDFUnits {enum Quantity : int;} // forward declaration
 
 const int NC_MPIIO_FIELD_NAME_MAX = 100;
 const int DUMP_NC_MPIIO_MAX_DIMS = 100;
@@ -47,7 +47,7 @@ class DumpNetCDFMPIIO : public DumpCustom {
     int field[DUMP_NC_MPIIO_MAX_DIMS];     // field indices corresponding to the dim.
     char name[NC_MPIIO_FIELD_NAME_MAX];    // field name
     int var;                               // NetCDF variable
-    Quantity quantity;                     // type of the quantity
+    NetCDFUnits::Quantity quantity;        // type of the quantity
   };
 
   typedef void (DumpNetCDFMPIIO::*funcptr_t)(void *);

src/NETCDF/netcdf_units.cpp

@@ -22,7 +22,9 @@
 
 #include "error.h"
 
-std::string LAMMPS_NS::get_unit_for(const char* unit_style, Quantity quantity, Error* error) {
+using namespace LAMMPS_NS;
+
+std::string NetCDFUnits::get_unit_for(const char* unit_style, Quantity quantity, Error* error) {
   if (!strcmp(unit_style, "lj")) {
     if (quantity == Quantity::UNKNOWN) {
       return "";

src/NETCDF/netcdf_units.h

@@ -25,8 +25,9 @@
 namespace LAMMPS_NS {
 class Error;
 
+namespace NetCDFUnits {
 // type of quantity for per-atom values (used to get the unit)
-enum class Quantity {
+enum Quantity : int {
     UNKNOWN = 0,
     TIME,
     DISTANCE,
@@ -38,7 +39,7 @@ enum class Quantity {
 // get the name of the unit for the given `quantity` in the given LAMMPS
 // `unit_style` any error will be reported through `error`
 std::string get_unit_for(const char* unit_style, Quantity quantity, Error* error);
-
+}
 }
 
 #endif