summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #25 from byrnHDF/developLarry Knox2020-10-075-130/+18
|\ | | | | gitub workflow changes
| * Remove obsolete fileAllen Byrne2020-10-072-112/+0
| |
| * Merge remote-tracking branch 'origin/develop' into developAllen Byrne2020-10-0741-14551/+14431
| |\ | |/ |/|
* | Update reformatting settings, trace wrapping, and printf-format strings (#22)Quincey Koziol2020-10-0641-14551/+14431
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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
| * Update code ownersAllen Byrne2020-10-071-3/+3
| |
| * Merge branch 'develop' of https://github.com/HDFGroup/hdf5.git into developAllen Byrne2020-10-060-0/+0
| |\ | |/ |/|
* | Fix VFD h5repacktest (#20)Allen Byrne2020-10-067-137/+141
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * h5repacktest VFD fix and formatting changes * Fix memory leak in H5PL__find_plugin_in_path When encountering a directory, the current loop iteration was skipped without freeing the memory allocated for the path. * Refactor parser GCC diagnostic statements * genparser on jelly with default bison/flex Co-authored-by: Michael Kuhn <michael.kuhn@ovgu.de>
* | Merge pull request #15 from lrknox/developLarry Knox2020-10-052-10/+7
|\ \ | | | | | | Correct entries related to --enable-build-mode and --enable-profiling in
| * | Correct entries related to --enable-build-mode and --enable-profiling inLarry Knox2020-10-032-10/+7
| | | | | | | | | | | | INSTALL file, and remove obsolete SZIP paragraph from COPYING file.
* | | Merge pull request #12 from byrnHDF/developLarry Knox2020-10-0510-23/+23
|\ \ \ | | | | | | | | HDFFV-11096 fix typo in h5dump usage text
* \ \ \ Merge pull request #16 from michaelkuhn/fix-plugin-path-leakLarry Knox2020-10-051-1/+3
|\ \ \ \ | | | | | | | | | | Fix memory leak in H5PL__find_plugin_in_path
| * | | | Fix memory leak in H5PL__find_plugin_in_pathMichael Kuhn2020-10-051-1/+3
|/ / / / | | | | | | | | | | | | | | | | When encountering a directory, the current loop iteration was skipped without freeing the memory allocated for the path.
| | | * Update for minimum of two ownersAllen Byrne2020-10-061-12/+12
| | | |
| | | * Correct entries related to --enable-build-mode and --enable-profiling inLarry Knox2020-10-062-10/+7
| | | | | | | | | | | | | | | | INSTALL file, and remove obsolete SZIP paragraph from COPYING file.
| | | * github environment variable changeAllen Byrne2020-10-062-4/+4
| | | |
| | | * genparser on jelly with default bison/flexAllen Byrne2020-10-062-90/+90
| | | |
| | | * Refactor parser GCC diagnostic statementsAllen Byrne2020-10-063-34/+44
| | | |
| | | * Fix memory leak in H5PL__find_plugin_in_pathMichael Kuhn2020-10-051-1/+3
| | | | | | | | | | | | | | | | | | | | When encountering a directory, the current loop iteration was skipped without freeing the memory allocated for the path.
| | | * h5repacktest VFD fix and formatting changesAllen Byrne2020-10-053-13/+7
| | |/ | |/|
| * | HDFFV-11096 replace backticks with doublequotesAllen Byrne2020-10-0410-23/+23
| | |
| * | HDFFV-11096 fix typo in h5dump usage textAllen Byrne2020-10-039-9/+9
|/ /
* | Merge pull request #11 from byrnHDF/developLarry Knox2020-10-031-0/+1
|\ \ | |/ |/| Add mingw define
| * Add mingw defineAllen Byrne2020-10-021-0/+1
|/
* Merge pull request #2917 in HDFFV/hdf5 from ~BYRN/hdf5_adb:develop to developAllen Byrne2020-10-022-2/+9
|\ | | | | | | | | * commit '1a4c5cc22c0f8aac568f39e67d695d906cf17c87': Actions updates
| * Actions updatesAllen Byrne2020-10-022-2/+9
|/
* Merge pull request #2906 in HDFFV/hdf5 from ~BYRN/hdf5_adb:develop to developLarry Knox2020-10-021118-319927/+322123
|\ | | | | | | | | * commit 'b2d661b508a7fc7a2592c13bc6bdc175551f075d': Clang-format of source files
| * Clang-format of source filesAllen Byrne2020-09-301118-319927/+322123
|/
* Merge pull request #2905 in HDFFV/hdf5 from ~BYRN/hdf5_adb:develop to developAllen Byrne2020-09-308-35/+28
|\ | | | | | | | | | | * commit '3f15fb35222a0fb2ba2f1f88a2d78daaea633ec8': Fix jni with %Lg instead of %Lf Pre-format fixes for warnings as errors
| * Fix jni with %Lg instead of %LfAllen Byrne2020-09-294-9/+7
| | | | | | Fix warnings due to improper formed define tag in h5diff
| * Pre-format fixes for warnings as errorsAllen Byrne2020-09-295-28/+23
|/
* Merge pull request #2650 in HDFFV/hdf5 from ↵David Young2020-09-28105-1089/+758
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ~DYOUNG/werror:rebased-fprintf-experiment to develop * commit 'c0fbc5c086566d5d3c1d1ef26baa81a53d59fc08': (24 commits) Use the right format string, "%zu", for size_t. Repair more format strings. Fix a bunch of format string errors reported by Larry. Fix some HDfprintf compilation errors: use the right format strings ("zu", PRIuHSIZE), avoid casting some printf arguments, pass the right number of arguments. Test the format string "ll" before "l", "L", and "q", like the ./configure script does. This ought to fix the compilation failure in test/dt_arith.c that Allen told me about: Cast a non-void pointer to void pointer for "%p". Use PRIu32 and "zu" formats. Delete some casts from `size_t`. I'm taking a guess that this code intended to point the 2-digit wide hexadecimal octet values, not 2 character-wide pointers to the bytes. The %02p format, which is a GNU-ism, disagreed with GCC 8.3.0 and the option flags we use. %08p is not portable, it's a GNU-ism. Use %8p, instead. Squashes a GCC error. Add format string macros PRI[doxX]HID for hid_t and use PRIdHID. Use HDva_copy() and introduce a bunch of compatbility format-string constants for uppercase hexadecimal strings, `PRIX...`. Should fix the VS2010 errors that Allen mentioned: Always #define HDfprintf as fprintf in this header. I believe this will fix the Windows build error that Allen reported. Provide an HDvasprintf implementation only if it isn't #defined. This should fix the mingw compilation issue that Allen reported. Fix va_list usage in the vasprintf(3) implementation. Promote format-string warnings to errors. Use the portable `-eq` operator instead of the bash-ism `==`. Fixes the tests on NetBSD, where /bin/sh != bash. Restore a literal percent sign ("%%") that I accidentally deleted. Fix code that made GCC complain about a NULL or `unsigned char *` arguments for "%s". Take pains to provide UINT64_MAX in all conditions. Correct a couple of format strings. ...
| * Use the right format string, "%zu", for size_t.David Young2020-09-271-1/+1
| |
| * Merge remote-tracking branch 'hdffv/develop' into rebased-fprintf-experimentDavid Young2020-09-2798-8187/+9687
| |\
| * | Repair more format strings.David Young2020-09-266-48/+49
| | |
| * | Merge remote-tracking branch 'hdffv/develop' into rebased-fprintf-experimentDavid Young2020-09-257-2/+249
| |\ \
| * | | Fix a bunch of format string errors reported by Larry.David Young2020-09-252-32/+38
| | | |
| * | | Fix some HDfprintf compilation errors: use the right format stringsDavid Young2020-09-243-20/+34
| | | | | | | | | | | | | | | | | | | | ("zu", PRIuHSIZE), avoid casting some printf arguments, pass the right number of arguments.
| * | | Test the format string "ll" before "l", "L", and "q", like theDavid Young2020-09-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ./configure script does. This ought to fix the compilation failure in test/dt_arith.c that Allen told me about: /home/buildbot/bb-workers/centos8-vm01/hdf5trunk-StdShar-code-centos8/build/hdfsrc/test/dt_arith.c: In function ‘test_conv_int_1’: /home/buildbot/bb-workers/centos8-vm01/hdf5trunk-StdShar-code-centos8/build/hdfsrc/test/dt_arith.c:2500:34: error: format ‘%ld’ expects argument of type ‘long int’, but argument 3 has type ‘long long int’ [-Werror=format=] HDfprintf(stdout," %29"H5_PRINTF_LL_WIDTH"d\n", *((long long*)aligned));
| * | | Cast a non-void pointer to void pointer for "%p".David Young2020-09-031-1/+1
| | | |
| * | | Merge remote-tracking branch 'hdffv/develop' into rebased-fprintf-experimentDavid Young2020-09-03588-16919/+18831
| |\ \ \
| * | | | Use PRIu32 and "zu" formats. Delete some casts from `size_t`.David Young2020-07-312-4/+4
| | | | |
| * | | | Merge remote-tracking branch 'hdffv/develop' into rebased-fprintf-experimentDavid Young2020-07-30131-6114/+5975
| |\ \ \ \
| * | | | | I'm taking a guess that this code intended to point the 2-digit wideDavid Young2020-07-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | hexadecimal octet values, not 2 character-wide pointers to the bytes. The %02p format, which is a GNU-ism, disagreed with GCC 8.3.0 and the option flags we use.
| * | | | | %08p is not portable, it's a GNU-ism. Use %8p, instead. Squashes a GCC error.David Young2020-07-301-1/+1
| | | | | |
| * | | | | Add format string macros PRI[doxX]HID for hid_t and use PRIdHID.David Young2020-07-302-1/+7
| | | | | |
| * | | | | Use HDva_copy() and introduce a bunch of compatbility format-string constantsDavid Young2020-07-303-2/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | for uppercase hexadecimal strings, `PRIX...`. Should fix the VS2010 errors that Allen mentioned: H5system.obj : error LNK2019: unresolved external symbol va_copy referenced in function HDvasprintf [C:\autotest\hdf5trunk-StdShar-code-vs10\build\ctest\hdfbld\src\hdf5-shared.vcxproj] 2 ..\..\..\hdfsrc\test\h5test.c(2103): error C2146: syntax error : missing ')' before identifier 'PRIX64' [C:\autotest\hdf5trunk-StdShar-code-vs10\build\ctest\hdfbld\test\hdf5_test-static.vcxproj]
| * | | | | Merge branch 'rebased-fprintf-experiment' into fprintf-developDavid Young2020-07-0991-1035/+666
| |\ \ \ \ \
| | * | | | | Always #define HDfprintf as fprintf in this header. I believe this will fixDavid Young2020-07-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | the Windows build error that Allen reported.
| | * | | | | Provide an HDvasprintf implementation only if it isn't #defined. This shouldDavid Young2020-07-071-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | fix the mingw compilation issue that Allen reported.
| | * | | | | Fix va_list usage in the vasprintf(3) implementation.David Young2020-07-071-2/+7
| | | | | | |