Changeset 8627 for trunk/psLib/src/sys/psError.h

Timestamp:

Aug 25, 2006, 6:34:28 PM (20 years ago)

Author:

jhoblitt

Message:

add gcc format attributes to:

psAbort()
psErrorStackPrint()
p_psError()
p_psWarning()
psLogMsg()
p_psTrace()

add PS_ASSERT_LONG_LARGER_THAN_OR_EQUAL
add PS_ASSERT_S64_WITHIN_RANGE
fix PS_ASSERT_LONG_WITHIN_RANGE
fix a wide range of format related issues:

• missing format field specifiers
• missing format args
• incorrect format field specifiers
• constants declared with the wrong type (float vs. int)
• PS_ASSERT* for the wrong type
• attemps to print structs with *printf()
• unportable format specifiers, eg. long vs. long long

File:

• trunk/psLib/src/sys/psError.h (modified) (5 diffs)

trunk/psLib/src/sys/psError.h

@@ -12 +12 @@
  *  @author Eric Van Alst, MHPCC
  *
- *  @version $Revision: 1.27 $ $Name: not supported by cvs2svn $
- *  @date $Date: 2006-08-08 23:11:25 $
+ *  @version $Revision: 1.28 $ $Name: not supported by cvs2svn $
+ *  @date $Date: 2006-08-26 04:34:28 $
  *
  *  Copyright 2004-2005 Maui High Performance Computing Center, University of Hawaii
@@ -92 +92 @@
  *
  */
+#ifdef __GNUC__
 void psErrorStackPrint(
     FILE* fd,                          ///< destination file descriptor
-    const char* format,                   ///< printf-style format of header line
-    ...                                ///< any parameters required in fmt
-);
+    const char* format,                ///< printf-style format of header line
+    ...                                ///< any parameters required in fmt
+) __attribute__((format(printf, 2, 3)));
+#else // __GNUC__
+void psErrorStackPrint(
+    FILE* fd,                          ///< destination file descriptor
+    const char* format,                ///< printf-style format of header line
+    ...                                ///< any parameters required in fmt
+);
+#endif // __GNUC__
+
 
 #ifndef SWIG
@@ -155 +164 @@
  *  @return psErrorcode    the given error code
  */
+#ifdef __GNUC__
 psErrorCode p_psError(
     const char* filename,              ///< file name
@@ -163 +173 @@
     const char* format,                ///< printf-style format of header line
     ...                                ///< any parameters required in fmt
-);
+) __attribute__((format(printf, 6, 7)));
+#else // __GNUC__
+psErrorCode p_psError(
+    const char* filename,              ///< file name
+    unsigned int lineno,               ///< line number in file
+    const char* func,                  ///< function name
+    psErrorCode code,                  ///< Error class code
+    bool new,                          ///< true if error originates at this location
+    const char* format,                ///< printf-style format of header line
+    ...                                ///< any parameters required in fmt
+);
+#endif // __GNUC__
 
 /** Logs a warning message.
@@ -172 +193 @@
  *
 */
+#ifdef __GNUC__
 void p_psWarning(
     const char* file,                  ///< file name
     int lineno,                        ///< line number in file
     const char* func,                  ///< function name
-    const char* fmt,                   ///< printf-style format of header line
-    ...                                ///< any parameters required in fmt
-);
+    const char* format,                ///< printf-style format of header line
+    ...                                ///< any parameters required in fmt
+) __attribute__((format(printf, 4, 5)));
+#else // __GNUC__
+void p_psWarning(
+    const char* file,                  ///< file name
+    int lineno,                        ///< line number in file
+    const char* func,                  ///< function name
+    const char* format,                ///< printf-style format of header line
+    ...                                ///< any parameters required in fmt
+);
+#endif // __GNUC__