summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | Quiet decleration-after-statement warnings.David Young2019-11-132-6/+5
| | | | | | | |
| * | | | | | | Move a statement under some declarations since some vintages of VisualDavid Young2019-11-131-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Studio don't like declarations after statements.
| * | | | | | | Document H5D__chunk_mem_xfree_wrapper().David Young2019-11-131-0/+5
| | | | | | | |
| * | | | | | | Undo accidental test deletion.David Young2019-11-131-1/+1
| | | | | | | |
| * | | | | | | Oops, delete a debug printf that snuck in here.David Young2019-11-131-1/+0
| | | | | | | |
| * | | | | | | Undo my changes to the HD macros, hadn't really intended those to be onDavid Young2019-11-131-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this branch....
| * | | | | | | Make errors of some more warnings. Move disabled warnings toDavid Young2019-11-131-31/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | DEVELOPER_WARNING_CFLAGS. Put just one warning option on a line, and sort some of the options.
| * | | | | | | Cast to the parameter type, H5VL_token_t *, instead of to unsigned char *.David Young2019-11-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (H5VL_token_t, too, probably should turn from a typedef for a constant-length array to an array encapsulated in a struct.)
| * | | | | | | Change hdset_reg_ref_t and H5R_ref_t from arrays of unsigned char toDavid Young2019-11-133-136/+145
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | structs containing those arrays. Encapsulating the arrays in this way makes it easier to write and think about pointers to these types, casts to/from these types, etc. An interesting side-effect that we probably should *not* rely on is that the struct-encapsulation changes the alignment so that some GCC warnings about casts that increase the alignment requirement of the operand go away. Warnings like that have to be taken seriously: I will add -Werror=cast-align to the default compiler flags so that they stop the build quickly. GCC warnings led me to some surprising casts in test/trefer.c. I found that it was possible to make many simplifications after introducing the struct-encapsulation that I described, above.
| * | | | | | | In every instance, `x` is initialized by `memcpy`, but GCC isn't smartDavid Young2019-11-131-17/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | enough to figure that out. Quiet some warnings by always initializing `x` to 0.
| * | | | | | | `same_file` is assigned but never used. Delete it.David Young2019-11-131-8/+0
| | | | | | | |
| * | | | | | | Don't assign a constant string to a pointer to non-constant character.David Young2019-11-131-1/+1
| | | | | | | |
| * | | | | | | GCC isn't smart enough to realize that `fill_c` is always initializedDavid Young2019-11-131-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | and exclusively used in `datatype == H5T_COMPOUND` branches, so just initialize it at its declaration so that GCC doesn't warn.
| * | | | | | | I'm not sure what the previous code was trying to do, casting an arrayDavid Young2019-11-131-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | of `hsize_t`, `start`, to `long long`, but I think the way that I have rewritten it, it probably produces a more useful result? As a bonus, GCC has stopped warning about it.
| * | | | | | | Quiet a warning about an unused variable. This code looks like itDavid Young2019-11-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | should be heavily restructured to avoid the use of globals like `pass`, but that's a project for another day and another person.
| * | | | | | | Straggler from previous commit: build and link the new .c files where IDavid Young2019-11-131-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | moved some previously-static variables and functions that were causing unused-variable/function warnings.
| * | | | | | | Make many warnings about H5T_copy() calls that discard const qualifiersDavid Young2019-11-132-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into a few warnings in H5T_copy() itself.
| * | | | | | | Use FUNC_ENTER_STATIC_NOERR instead of FUNC_ENTER_STATIC for a functionDavid Young2019-11-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | that does not generate its own error, so GCC does not warn that err_occurred (or whatever it is called) is unused or set and unused.
| * | | | | | | Explicitly initialize `type_flags`; the compiler does not realize thatDavid Young2019-11-131-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | it is set before use by passing it as a reference to another function.
| * | | | | | | Initialize a couple of return values to avoid used-before-set warnings.David Young2019-11-131-2/+2
| | | | | | | |
| * | | | | | | Initialize a couple of variables that the compiler does not realize areDavid Young2019-11-132-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | initialized when they're passed by reference to functions.
| * | | | | | | To quiet a warning, initialize some `struct timeval`s that the compilerDavid Young2019-11-131-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | does not realize are always set before use by `gettimeofday`.
| * | | | | | | Move some static functions and variables to .c files to avoid unusedDavid Young2019-11-134-54/+106
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | function/variable warnings.
| * | | | | | | Delete unused variable.David Young2019-11-131-1/+0
| | | | | | | |
| * | | | | | | Change a signed variable to unsigned to avoid a warning about the signDavid Young2019-11-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | being lost by a cast.
| * | | | | | | Avoid unused-variable warnings: change static const strings in a coupleDavid Young2019-11-132-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | of header-file templates to #defines.
| * | | | | | | Demote errors to warnings for a couple of unused-but-set variables thatDavid Young2019-11-132-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I cannot untangle right now.
| * | | | | | | Delete an unused variable.David Young2019-11-131-1/+0
| | | | | | | |
| * | | | | | | Mark an unused parameter H5_ATTR_UNUSED.David Young2019-11-131-1/+1
| | | | | | | |
| * | | | | | | Delete variables that are unused or set and unused.David Young2019-11-131-12/+0
| | | | | | | |
| * | | | | | | Initialize a variable before its first use.David Young2019-11-131-1/+1
| | | | | | | |
| * | | | | | | Format a pointer with %p to avoid a gripe about casting a pointer to anDavid Young2019-11-131-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | integer.
| * | | | | | | Merge remote-tracking branch 'hdf5/develop' into add-werror-and-squash-some toDavid Young2019-11-1242-326/+792
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | see if new warnings-as-errors have cropped up.
| * \ \ \ \ \ \ \ Merge remote-tracking branch 'hdf5/develop' into add-werror-and-squash-someDavid Young2019-11-0829-41/+275
| |\ \ \ \ \ \ \ \ | | | |_|_|_|_|/ / | | |/| | | | | | | | | | | | | | | to pick up the NetBSD-compatibility changes that I recently merged.
| * | | | | | | | Change some GCC warnings to errors. Fix code to quiet some warnings.David Young2019-11-0518-57/+149
| | |_|_|_|/ / / | |/| | | | | |
| | | | | | * | Remove unneeded flag overwritesAllen Byrne2019-11-251-8/+0
| | | | | | | |
| | | | | | * | Merging in latest from upstream (HDFFV/hdf5:refs/heads/develop)Allen Byrne2019-11-2511-17/+285
| | | | | | |\ \ | |_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * commit 'b8a56718e40efaf6954a760984b9d513dbff8f7d': Add documentation for H5CX_get_ohdr_flags Add object header flags to API context Use major error code of calling package Handle H5CX_state_t for lcpl Add character encoding to lcpl context Change LC property name from 'create_intermediate_group' to 'intermediate_group' Use API context to store/retrieve LCPL when creating intermediate groups
* | | | | | | | Merge pull request #2029 in HDFFV/hdf5 from ~CHOGAN/hdf5:feature/cx_lcpl to ↵Chris Hogan2019-11-2511-17/+285
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | develop * commit '4ce9c10c568ff81ebd4eb7585039cbe3c40d1718': Add documentation for H5CX_get_ohdr_flags Add object header flags to API context Use major error code of calling package Handle H5CX_state_t for lcpl Add character encoding to lcpl context Change LC property name from 'create_intermediate_group' to 'intermediate_group' Use API context to store/retrieve LCPL when creating intermediate groups
| * | | | | | | | Add documentation for H5CX_get_ohdr_flagsChris Hogan2019-11-151-0/+13
| | | | | | | | |
| * | | | | | | | Add object header flags to API contextChris Hogan2019-11-144-5/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add missing DCPL to API context state Replace a couple LCPL H5P_get calls with H5CX_get_*
| * | | | | | | | Use major error code of calling packageChris Hogan2019-11-081-2/+2
| | | | | | | | |
| * | | | | | | | Handle H5CX_state_t for lcplChris Hogan2019-11-041-0/+21
| | | | | | | | |
| * | | | | | | | Add character encoding to lcpl contextChris Hogan2019-11-043-43/+54
| | | | | | | | |
| * | | | | | | | Change LC property name from 'create_intermediate_group' to 'intermediate_group'Chris Hogan2019-10-301-9/+9
| | | | | | | | |
| * | | | | | | | Use API context to store/retrieve LCPL when creating intermediate groupsChris Hogan2019-10-2910-11/+167
| |/ / / / / / /
| | | | | | * | Merging in latest from upstream (HDFFV/hdf5:refs/heads/develop)Allen Byrne2019-11-211-1/+1
| | | | | | |\ \ | |_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | * commit '33d173d672f4affc804cf6e3c43b92693bd3ed68': Modify H5Ssel_iter_get_seq_list() to accept iterators that have reached the end of iteration (returning zero sequences). Previously it could cause an assertion failure.
* | | | | | | | Merge pull request #2043 in HDFFV/hdf5 from ~NFORTNE2/hdf5_naf:seq_list_fix ↵Neil Fortner2019-11-201-1/+1
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | to develop * commit 'c104f4cb87a1558ab6a772967887738dc9800f0d': Modify H5Ssel_iter_get_seq_list() to accept iterators that have reached the end of iteration (returning zero sequences). Previously it could cause an assertion failure.
| * | | | | | | | Modify H5Ssel_iter_get_seq_list() to accept iterators that have reachedNeil Fortner2019-11-151-1/+1
| | |_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the end of iteration (returning zero sequences). Previously it could cause an assertion failure.
| | | | | | * | Comment correctionAllen Byrne2019-11-212-2/+2
| | | | | | | |
| | | | | | * | Merging in latest from upstream (HDFFV/hdf5:refs/heads/develop)Allen Byrne2019-11-2012-40/+73
| | | | | | |\ \ | |_|_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * commit '13456df8a47fdcd8f96f170c9218e57a03066120': Minor correction Update list of tested platforms Add VS2019 support Fixed the H5_mpi_get_bigio_count function prototype