Update reformatting settings, trace wrapping, and printf-format strings (#22)

* Add BEGIN_FUNC / CATCH / END_FUNC to macros

* Remove incorrect trailing '{'

* Update trace script to wrap H5TRACE macros at 110 characters and improve
comments for code that performs this operation.

* Clean up printf-formatting in H5public.h and correct some oversights in the code.

* Reformat with BEGIN_FUNC / END_FUNC handled correctly by clang-format

* Run clang-format on changes.

* Correct H5_SIZEOF_HSIZE_T/H5_SIZEOF_HSSIZE_T macros to match typedef.

* And make the undef value match also.

* Update comment

* Update clang-format actions to exclude generated files.

* Post-process generated H5LTparse.h file as well as H5LTparse.c file.

* Fix formatting

* Update version of github clang-format action

1 files changed, 87 insertions, 75 deletions

diff --git a/hl/src/H5LTparse.h b/hl/src/H5LTparse.h
index 4855f42..abff969 100644
--- a/hl/src/H5LTparse.h
+++ b/hl/src/H5LTparse.h
@@ -1,8 +1,9 @@
-/* A Bison parser, made by GNU Bison 3.0.4.  */
+/* A Bison parser, made by GNU Bison 3.7.2.  */
 
 /* Bison interface for Yacc-like parsers in C
 
-   Copyright (C) 1984, 1989-1990, 2000-2015 Free Software Foundation, Inc.
+   Copyright (C) 1984, 1989-1990, 2000-2015, 2018-2020 Free Software Foundation,
+   Inc.
 
    This program is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@ -30,99 +31,110 @@
    This special exception was added by the Free Software Foundation in
    version 2.2 of Bison.  */
 
+/* DO NOT RELY ON FEATURES THAT ARE NOT DOCUMENTED in the manual,
+   especially those whose name start with YY_ or yy_.  They are
+   private implementation details that can be changed or removed.  */
+
 #ifndef YY_H5LTYY_HL_SRC_H5LTPARSE_H_INCLUDED
-#define YY_H5LTYY_HL_SRC_H5LTPARSE_H_INCLUDED
+# define YY_H5LTYY_HL_SRC_H5LTPARSE_H_INCLUDED
 /* Debug traces.  */
 #ifndef YYDEBUG
-#define YYDEBUG 0
+# define YYDEBUG 0
 #endif
 #if YYDEBUG
 extern int H5LTyydebug;
 #endif
 
-/* Token type.  */
+/* Token kinds.  */
 #ifndef YYTOKENTYPE
-#define YYTOKENTYPE
-enum yytokentype {
-    H5T_STD_I8BE_TOKEN       = 258,
-    H5T_STD_I8LE_TOKEN       = 259,
-    H5T_STD_I16BE_TOKEN      = 260,
-    H5T_STD_I16LE_TOKEN      = 261,
-    H5T_STD_I32BE_TOKEN      = 262,
-    H5T_STD_I32LE_TOKEN      = 263,
-    H5T_STD_I64BE_TOKEN      = 264,
-    H5T_STD_I64LE_TOKEN      = 265,
-    H5T_STD_U8BE_TOKEN       = 266,
-    H5T_STD_U8LE_TOKEN       = 267,
-    H5T_STD_U16BE_TOKEN      = 268,
-    H5T_STD_U16LE_TOKEN      = 269,
-    H5T_STD_U32BE_TOKEN      = 270,
-    H5T_STD_U32LE_TOKEN      = 271,
-    H5T_STD_U64BE_TOKEN      = 272,
-    H5T_STD_U64LE_TOKEN      = 273,
-    H5T_NATIVE_CHAR_TOKEN    = 274,
-    H5T_NATIVE_SCHAR_TOKEN   = 275,
-    H5T_NATIVE_UCHAR_TOKEN   = 276,
-    H5T_NATIVE_SHORT_TOKEN   = 277,
-    H5T_NATIVE_USHORT_TOKEN  = 278,
-    H5T_NATIVE_INT_TOKEN     = 279,
-    H5T_NATIVE_UINT_TOKEN    = 280,
-    H5T_NATIVE_LONG_TOKEN    = 281,
-    H5T_NATIVE_ULONG_TOKEN   = 282,
-    H5T_NATIVE_LLONG_TOKEN   = 283,
-    H5T_NATIVE_ULLONG_TOKEN  = 284,
-    H5T_IEEE_F32BE_TOKEN     = 285,
-    H5T_IEEE_F32LE_TOKEN     = 286,
-    H5T_IEEE_F64BE_TOKEN     = 287,
-    H5T_IEEE_F64LE_TOKEN     = 288,
-    H5T_NATIVE_FLOAT_TOKEN   = 289,
-    H5T_NATIVE_DOUBLE_TOKEN  = 290,
-    H5T_NATIVE_LDOUBLE_TOKEN = 291,
-    H5T_STRING_TOKEN         = 292,
-    STRSIZE_TOKEN            = 293,
-    STRPAD_TOKEN             = 294,
-    CSET_TOKEN               = 295,
-    CTYPE_TOKEN              = 296,
-    H5T_VARIABLE_TOKEN       = 297,
-    H5T_STR_NULLTERM_TOKEN   = 298,
-    H5T_STR_NULLPAD_TOKEN    = 299,
-    H5T_STR_SPACEPAD_TOKEN   = 300,
-    H5T_CSET_ASCII_TOKEN     = 301,
-    H5T_CSET_UTF8_TOKEN      = 302,
-    H5T_C_S1_TOKEN           = 303,
-    H5T_FORTRAN_S1_TOKEN     = 304,
-    H5T_OPAQUE_TOKEN         = 305,
-    OPQ_SIZE_TOKEN           = 306,
-    OPQ_TAG_TOKEN            = 307,
-    H5T_COMPOUND_TOKEN       = 308,
-    H5T_ENUM_TOKEN           = 309,
-    H5T_ARRAY_TOKEN          = 310,
-    H5T_VLEN_TOKEN           = 311,
-    STRING                   = 312,
-    NUMBER                   = 313
-};
+# define YYTOKENTYPE
+  enum yytokentype
+  {
+    YYEMPTY = -2,
+    YYEOF = 0,                     /* "end of file"  */
+    YYerror = 256,                 /* error  */
+    YYUNDEF = 257,                 /* "invalid token"  */
+    H5T_STD_I8BE_TOKEN = 258,      /* H5T_STD_I8BE_TOKEN  */
+    H5T_STD_I8LE_TOKEN = 259,      /* H5T_STD_I8LE_TOKEN  */
+    H5T_STD_I16BE_TOKEN = 260,     /* H5T_STD_I16BE_TOKEN  */
+    H5T_STD_I16LE_TOKEN = 261,     /* H5T_STD_I16LE_TOKEN  */
+    H5T_STD_I32BE_TOKEN = 262,     /* H5T_STD_I32BE_TOKEN  */
+    H5T_STD_I32LE_TOKEN = 263,     /* H5T_STD_I32LE_TOKEN  */
+    H5T_STD_I64BE_TOKEN = 264,     /* H5T_STD_I64BE_TOKEN  */
+    H5T_STD_I64LE_TOKEN = 265,     /* H5T_STD_I64LE_TOKEN  */
+    H5T_STD_U8BE_TOKEN = 266,      /* H5T_STD_U8BE_TOKEN  */
+    H5T_STD_U8LE_TOKEN = 267,      /* H5T_STD_U8LE_TOKEN  */
+    H5T_STD_U16BE_TOKEN = 268,     /* H5T_STD_U16BE_TOKEN  */
+    H5T_STD_U16LE_TOKEN = 269,     /* H5T_STD_U16LE_TOKEN  */
+    H5T_STD_U32BE_TOKEN = 270,     /* H5T_STD_U32BE_TOKEN  */
+    H5T_STD_U32LE_TOKEN = 271,     /* H5T_STD_U32LE_TOKEN  */
+    H5T_STD_U64BE_TOKEN = 272,     /* H5T_STD_U64BE_TOKEN  */
+    H5T_STD_U64LE_TOKEN = 273,     /* H5T_STD_U64LE_TOKEN  */
+    H5T_NATIVE_CHAR_TOKEN = 274,   /* H5T_NATIVE_CHAR_TOKEN  */
+    H5T_NATIVE_SCHAR_TOKEN = 275,  /* H5T_NATIVE_SCHAR_TOKEN  */
+    H5T_NATIVE_UCHAR_TOKEN = 276,  /* H5T_NATIVE_UCHAR_TOKEN  */
+    H5T_NATIVE_SHORT_TOKEN = 277,  /* H5T_NATIVE_SHORT_TOKEN  */
+    H5T_NATIVE_USHORT_TOKEN = 278, /* H5T_NATIVE_USHORT_TOKEN  */
+    H5T_NATIVE_INT_TOKEN = 279,    /* H5T_NATIVE_INT_TOKEN  */
+    H5T_NATIVE_UINT_TOKEN = 280,   /* H5T_NATIVE_UINT_TOKEN  */
+    H5T_NATIVE_LONG_TOKEN = 281,   /* H5T_NATIVE_LONG_TOKEN  */
+    H5T_NATIVE_ULONG_TOKEN = 282,  /* H5T_NATIVE_ULONG_TOKEN  */
+    H5T_NATIVE_LLONG_TOKEN = 283,  /* H5T_NATIVE_LLONG_TOKEN  */
+    H5T_NATIVE_ULLONG_TOKEN = 284, /* H5T_NATIVE_ULLONG_TOKEN  */
+    H5T_IEEE_F32BE_TOKEN = 285,    /* H5T_IEEE_F32BE_TOKEN  */
+    H5T_IEEE_F32LE_TOKEN = 286,    /* H5T_IEEE_F32LE_TOKEN  */
+    H5T_IEEE_F64BE_TOKEN = 287,    /* H5T_IEEE_F64BE_TOKEN  */
+    H5T_IEEE_F64LE_TOKEN = 288,    /* H5T_IEEE_F64LE_TOKEN  */
+    H5T_NATIVE_FLOAT_TOKEN = 289,  /* H5T_NATIVE_FLOAT_TOKEN  */
+    H5T_NATIVE_DOUBLE_TOKEN = 290, /* H5T_NATIVE_DOUBLE_TOKEN  */
+    H5T_NATIVE_LDOUBLE_TOKEN = 291, /* H5T_NATIVE_LDOUBLE_TOKEN  */
+    H5T_STRING_TOKEN = 292,        /* H5T_STRING_TOKEN  */
+    STRSIZE_TOKEN = 293,           /* STRSIZE_TOKEN  */
+    STRPAD_TOKEN = 294,            /* STRPAD_TOKEN  */
+    CSET_TOKEN = 295,              /* CSET_TOKEN  */
+    CTYPE_TOKEN = 296,             /* CTYPE_TOKEN  */
+    H5T_VARIABLE_TOKEN = 297,      /* H5T_VARIABLE_TOKEN  */
+    H5T_STR_NULLTERM_TOKEN = 298,  /* H5T_STR_NULLTERM_TOKEN  */
+    H5T_STR_NULLPAD_TOKEN = 299,   /* H5T_STR_NULLPAD_TOKEN  */
+    H5T_STR_SPACEPAD_TOKEN = 300,  /* H5T_STR_SPACEPAD_TOKEN  */
+    H5T_CSET_ASCII_TOKEN = 301,    /* H5T_CSET_ASCII_TOKEN  */
+    H5T_CSET_UTF8_TOKEN = 302,     /* H5T_CSET_UTF8_TOKEN  */
+    H5T_C_S1_TOKEN = 303,          /* H5T_C_S1_TOKEN  */
+    H5T_FORTRAN_S1_TOKEN = 304,    /* H5T_FORTRAN_S1_TOKEN  */
+    H5T_OPAQUE_TOKEN = 305,        /* H5T_OPAQUE_TOKEN  */
+    OPQ_SIZE_TOKEN = 306,          /* OPQ_SIZE_TOKEN  */
+    OPQ_TAG_TOKEN = 307,           /* OPQ_TAG_TOKEN  */
+    H5T_COMPOUND_TOKEN = 308,      /* H5T_COMPOUND_TOKEN  */
+    H5T_ENUM_TOKEN = 309,          /* H5T_ENUM_TOKEN  */
+    H5T_ARRAY_TOKEN = 310,         /* H5T_ARRAY_TOKEN  */
+    H5T_VLEN_TOKEN = 311,          /* H5T_VLEN_TOKEN  */
+    STRING = 312,                  /* STRING  */
+    NUMBER = 313                   /* NUMBER  */
+  };
+  typedef enum yytokentype yytoken_kind_t;
 #endif
 
 /* Value type.  */
-#if !defined YYSTYPE && !defined YYSTYPE_IS_DECLARED
+#if ! defined YYSTYPE && ! defined YYSTYPE_IS_DECLARED
+union YYSTYPE
+{
+#line 72 "hl/src/H5LTparse.y"
 
-union YYSTYPE {
-#line 72 "hl/src/H5LTparse.y" /* yacc.c:1909  */
+    int     ival;         /*for integer token*/
+    char    *sval;        /*for name string*/
+    hid_t   hid;          /*for hid_t token*/
 
-    int   ival; /*for integer token*/
-    char *sval; /*for name string*/
-    hid_t hid;  /*for hid_t token*/
+#line 128 "hl/src/H5LTparse.h"
 
-#line 119 "hl/src/H5LTparse.h" /* yacc.c:1909  */
 };
-
 typedef union YYSTYPE YYSTYPE;
-#define YYSTYPE_IS_TRIVIAL  1
-#define YYSTYPE_IS_DECLARED 1
+# define YYSTYPE_IS_TRIVIAL 1
+# define YYSTYPE_IS_DECLARED 1
 #endif
 
+
 extern YYSTYPE H5LTyylval;
 
-int H5LTyyparse(void);
+hid_t H5LTyyparse (void);
 
 #endif /* !YY_H5LTYY_HL_SRC_H5LTPARSE_H_INCLUDED  */