Changeset 1424 for XIOS/dev/XIOS_DEV_CMIP6/src

Timestamp:

02/09/18 12:11:04 (6 years ago)

Author:

oabramkina

Message:

Bugfix: avoiding unnecessary cast of default values to float or short int.

File:

• XIOS/dev/XIOS_DEV_CMIP6/src/io/nc4_data_output.cpp (modified) (1 diff)

XIOS/dev/XIOS_DEV_CMIP6/src/io/nc4_data_output.cpp

@@ -2026 +2026 @@
            if (!field->default_value.isEmpty())
            {
-              double default_value = field->default_value.getValue();
-              short sdefault_value = (short)default_value;
-              float fdefault_value = (float)default_value;
-              if (type == NC_DOUBLE)
-                 SuperClassWriter::setDefaultValue(fieldid, &default_value);
-              else if (type == NC_SHORT)
-                  SuperClassWriter::setDefaultValue(fieldid, &sdefault_value);
-              else
-                 SuperClassWriter::setDefaultValue(fieldid, &fdefault_value);
+             double default_value = field->default_value.getValue();
+             if (type == NC_DOUBLE)
+             {
+               SuperClassWriter::setDefaultValue(fieldid, &default_value);
+             }
+             else if (type == NC_SHORT)
+             {
+               short sdefault_value = (short)default_value;
+               SuperClassWriter::setDefaultValue(fieldid, &sdefault_value);
+             }
+             else
+             {
+               float fdefault_value = (float)default_value;
+               SuperClassWriter::setDefaultValue(fieldid, &fdefault_value);
+             }
            }
            else