fixing problems when built without logging, took out unneeded msgs

E3SM-Project · Jun 16, 2016 · e7dd43e · e7dd43e
1 parent 77152b4
commit e7dd43e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 133 deletions.
diff --git a/src/clib/pio_internal.h b/src/clib/pio_internal.h
@@ -17,7 +17,7 @@
 #include <gptl.h>
 #endif
 
-#ifdef PIO_ENABLE_LOGGING
+#if PIO_ENABLE_LOGGING
 void pio_log(int severity, const char *fmt, ...);
 #define LOG(e) pio_log e
 #else
@@ -176,178 +176,54 @@ enum PIO_MSG{
   PIO_MSG_INQ_VARID,
   PIO_MSG_DEF_VAR,
   PIO_MSG_INQ_VAR,
-  PIO_MSG_PUT_ATT_DOUBLE,
-  PIO_MSG_PUT_ATT_INT,
   PIO_MSG_RENAME_ATT,
   PIO_MSG_DEL_ATT,
   PIO_MSG_INQ,
-  PIO_MSG_GET_ATT_TEXT,
-  PIO_MSG_GET_ATT_SHORT,
-  PIO_MSG_PUT_ATT_LONG,
   PIO_MSG_REDEF,
   PIO_MSG_SET_FILL,
   PIO_MSG_ENDDEF,
   PIO_MSG_RENAME_VAR,
-  PIO_MSG_PUT_ATT_SHORT,
-  PIO_MSG_PUT_ATT_TEXT,
   PIO_MSG_INQ_ATTNAME,
-  PIO_MSG_GET_ATT_ULONGLONG,
-  PIO_MSG_GET_ATT_USHORT,
-  PIO_MSG_PUT_ATT_ULONGLONG,
-  PIO_MSG_GET_ATT_UINT,
-  PIO_MSG_GET_ATT_LONGLONG,
-  PIO_MSG_PUT_ATT_SCHAR,
-  PIO_MSG_PUT_ATT_FLOAT,
   PIO_MSG_RENAME_DIM,
-  PIO_MSG_GET_ATT_LONG,
   PIO_MSG_INQ_DIM,
   PIO_MSG_INQ_DIMID,
-  PIO_MSG_PUT_ATT_USHORT,
-  PIO_MSG_GET_ATT_FLOAT,
   PIO_MSG_SYNC,
-  PIO_MSG_PUT_ATT_LONGLONG,
-  PIO_MSG_PUT_ATT_UINT,
-  PIO_MSG_GET_ATT_SCHAR,
   PIO_MSG_INQ_ATTID,
   PIO_MSG_DEF_DIM,
-  PIO_MSG_GET_ATT_INT,
-  PIO_MSG_GET_ATT_DOUBLE,
-  PIO_MSG_PUT_ATT_UCHAR,
-  PIO_MSG_GET_ATT_UCHAR,
-  PIO_MSG_PUT_VARS_UCHAR,
-  PIO_MSG_GET_VAR1_SCHAR,
-  PIO_MSG_GET_VARS_ULONGLONG,
   PIO_MSG_GET_VARM_UCHAR,
   PIO_MSG_GET_VARM_SCHAR,
-  PIO_MSG_GET_VARS_SHORT,
-  PIO_MSG_GET_VAR_DOUBLE,
-  PIO_MSG_GET_VARA_DOUBLE,
-  PIO_MSG_GET_VAR_INT,
-  PIO_MSG_GET_VAR_USHORT,
-  PIO_MSG_PUT_VARS_USHORT,
-  PIO_MSG_GET_VARA_TEXT,
-  PIO_MSG_PUT_VARS_ULONGLONG,
-  PIO_MSG_GET_VARA_INT,
   PIO_MSG_PUT_VARM,
-  PIO_MSG_GET_VAR1_FLOAT,
-  PIO_MSG_GET_VAR1_SHORT,
-  PIO_MSG_GET_VARS_INT,
-  PIO_MSG_PUT_VARS_UINT,
-  PIO_MSG_GET_VAR_TEXT,
   PIO_MSG_GET_VARM_DOUBLE,
   PIO_MSG_PUT_VARM_UCHAR,
-  PIO_MSG_PUT_VAR_USHORT,
-  PIO_MSG_GET_VARS_SCHAR,
-  PIO_MSG_GET_VARA_USHORT,
-  PIO_MSG_PUT_VAR1_LONGLONG,
-  PIO_MSG_PUT_VARA_UCHAR,
   PIO_MSG_PUT_VARM_SHORT,
-  PIO_MSG_PUT_VAR1_LONG,
-  PIO_MSG_PUT_VARS_LONG,
-  PIO_MSG_GET_VAR1_USHORT,
-  PIO_MSG_PUT_VAR_SHORT,
-  PIO_MSG_PUT_VARA_INT,
-  PIO_MSG_GET_VAR_FLOAT,
-  PIO_MSG_PUT_VAR1_USHORT,
-  PIO_MSG_PUT_VARA_TEXT,
   PIO_MSG_PUT_VARM_TEXT,
-  PIO_MSG_GET_VARS_UCHAR,
-  PIO_MSG_GET_VAR,
   PIO_MSG_PUT_VARM_USHORT,
-  PIO_MSG_GET_VAR1_LONGLONG,
-  PIO_MSG_GET_VARS_USHORT,
-  PIO_MSG_GET_VAR_LONG,
-  PIO_MSG_GET_VAR1_DOUBLE,
-  PIO_MSG_PUT_VAR_ULONGLONG,
-  PIO_MSG_PUT_VAR_INT,
-  PIO_MSG_GET_VARA_UINT,
-  PIO_MSG_PUT_VAR_LONGLONG,
-  PIO_MSG_GET_VARS_LONGLONG,
-  PIO_MSG_PUT_VAR_SCHAR,
-  PIO_MSG_PUT_VAR_UINT,
-  PIO_MSG_PUT_VAR,
-  PIO_MSG_PUT_VARA_USHORT,
-  PIO_MSG_GET_VAR_LONGLONG,
-  PIO_MSG_GET_VARA_SHORT,
-  PIO_MSG_PUT_VARS_SHORT,
-  PIO_MSG_PUT_VARA_UINT,
-  PIO_MSG_PUT_VARA_SCHAR,
   PIO_MSG_PUT_VARM_ULONGLONG,
-  PIO_MSG_PUT_VAR1_UCHAR,
   PIO_MSG_PUT_VARM_INT,
-  PIO_MSG_PUT_VARS_SCHAR,
-  PIO_MSG_GET_VARA_LONG,
-  PIO_MSG_PUT_VAR1,
-  PIO_MSG_GET_VAR1_INT,
-  PIO_MSG_GET_VAR1_ULONGLONG,
-  PIO_MSG_GET_VAR_UCHAR,
-  PIO_MSG_PUT_VARA_FLOAT,
-  PIO_MSG_GET_VARA_UCHAR,
-  PIO_MSG_GET_VARS_FLOAT,
-  PIO_MSG_PUT_VAR1_FLOAT,
   PIO_MSG_PUT_VARM_FLOAT,
-  PIO_MSG_PUT_VAR1_TEXT,
-  PIO_MSG_PUT_VARS_TEXT,
   PIO_MSG_PUT_VARM_LONG,
-  PIO_MSG_GET_VARS_LONG,
-  PIO_MSG_PUT_VARS_DOUBLE,
-  PIO_MSG_GET_VAR1,
-  PIO_MSG_GET_VAR_UINT,
-  PIO_MSG_PUT_VARA_LONGLONG,
-  PIO_MSG_GET_VARA,
-  PIO_MSG_PUT_VAR_DOUBLE,
-  PIO_MSG_GET_VARA_SCHAR,
-  PIO_MSG_PUT_VAR_FLOAT,
-  PIO_MSG_GET_VAR1_UINT,
-  PIO_MSG_GET_VARS_UINT,
-  PIO_MSG_PUT_VAR1_ULONGLONG,
   PIO_MSG_PUT_VARM_UINT,
-  PIO_MSG_PUT_VAR1_UINT,
-  PIO_MSG_PUT_VAR1_INT,
-  PIO_MSG_GET_VARA_FLOAT,
   PIO_MSG_GET_VARM_TEXT,
-  PIO_MSG_PUT_VARS_FLOAT,
-  PIO_MSG_GET_VAR1_TEXT,
-  PIO_MSG_PUT_VARA_SHORT,
-  PIO_MSG_PUT_VAR1_SCHAR,
-  PIO_MSG_PUT_VARA_ULONGLONG,
   PIO_MSG_PUT_VARM_DOUBLE,
   PIO_MSG_GET_VARM_INT,
-  PIO_MSG_PUT_VARA,
-  PIO_MSG_PUT_VARA_LONG,
   PIO_MSG_GET_VARM_UINT,
   PIO_MSG_GET_VARM,
-  PIO_MSG_PUT_VAR1_DOUBLE,
-  PIO_MSG_GET_VARS_DOUBLE,
-  PIO_MSG_GET_VARA_LONGLONG,
-  PIO_MSG_GET_VAR_ULONGLONG,
   PIO_MSG_PUT_VARM_SCHAR,
-  PIO_MSG_GET_VARA_ULONGLONG,
-  PIO_MSG_GET_VAR_SHORT,
   PIO_MSG_GET_VARM_FLOAT,
-  PIO_MSG_PUT_VAR_TEXT,
-  PIO_MSG_PUT_VARS_INT,
-  PIO_MSG_GET_VAR1_LONG,
   PIO_MSG_GET_VARM_LONG,
   PIO_MSG_GET_VARM_USHORT,
-  PIO_MSG_PUT_VAR1_SHORT,
-  PIO_MSG_PUT_VARS_LONGLONG,
   PIO_MSG_GET_VARM_LONGLONG,
-  PIO_MSG_GET_VARS_TEXT,
-  PIO_MSG_PUT_VARA_DOUBLE,
   PIO_MSG_PUT_VARS,
-  PIO_MSG_PUT_VAR_UCHAR,
-  PIO_MSG_GET_VAR1_UCHAR,
-  PIO_MSG_PUT_VAR_LONG,
   PIO_MSG_GET_VARS,
   PIO_MSG_GET_VARM_SHORT,
   PIO_MSG_GET_VARM_ULONGLONG,
   PIO_MSG_PUT_VARM_LONGLONG,
-  PIO_MSG_GET_VAR_SCHAR,
-  PIO_MSG_GET_ATT_UBYTE,
-  PIO_MSG_PUT_ATT_STRING,
-  PIO_MSG_GET_ATT_STRING,
-  PIO_MSG_PUT_ATT_UBYTE,
+  PIO_MSG_PUT_VAR1,
+  PIO_MSG_PUT_VARA,
+  PIO_MSG_PUT_VAR,
+  PIO_MSG_GET_VAR1,
+  PIO_MSG_GET_VARA,
+  PIO_MSG_GET_VAR,
   PIO_MSG_INQ_VAR_FILL,
   PIO_MSG_DEF_VAR_FILL,
   PIO_MSG_DEF_VAR_DEFLATE,

diff --git a/src/clib/pioc_support.c b/src/clib/pioc_support.c
@@ -2,7 +2,7 @@
  * Support functions.
  */
 #include <config.h>
-#ifdef PIO_ENABLE_LOGGING
+#if PIO_ENABLE_LOGGING
 #include <stdarg.h>
 #include <unistd.h>
 #endif /* PIO_ENABLE_LOGGING */
@@ -27,8 +27,8 @@ int PIOc_set_log_level(int level)
     printf("setting log level to %d\n", level);
     pio_log_level = level;
     MPI_Comm_rank(MPI_COMM_WORLD, &my_rank);
-    return PIO_NOERR;
 #endif /* PIO_ENABLE_LOGGING */
+    return PIO_NOERR;
 }
 
 #if PIO_ENABLE_LOGGING