| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* commit 'a8892bb42d6f6e4fbc30fae0eb2b957f81c938b9': (45 commits)
Oops, remove more C99 designated initializers for VS 2010 compatibility.
Add an #include to get a function declaration.
Don't use C99 designated initializers, they're not compatible with Visual Studio 2010.
Quiet some more maybe-uninitialized warnings---each is a false positive, *sigh*. This is more code that may not compile with VS2010, *sigh sigh*.
Always warn on maybe-uninitialized. -Wincompatible-pointer-types was not available until GCC 5, so enable it only if that's the GCC version we're using.
Only promote maybe-uninitialized warnings to errors on GCC 8. Even on GCC 8, there may be false positives at low optimization levels? I need to check.
Only use -Werror=cast-function-type with GCC 8 and later.
Put all of the -W options back into the order I found them in so that it's easier to compare old and new config/gnu-flags.
Add new source files to CMakeLists.txt.
Mention the -Werror= flags in libhdf5.settings.in.
free -> HDfree
Promote decleration-after-statement warnings to errors.
Quiet decleration-after-statement warnings.
Move a statement under some declarations since some vintages of Visual Studio don't like declarations after statements.
Document H5D__chunk_mem_xfree_wrapper().
Undo accidental test deletion.
Oops, delete a debug printf that snuck in here.
Undo my changes to the HD macros, hadn't really intended those to be on this branch....
Make errors of some more warnings. Move disabled warnings to 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 *.
...
|
| |\
| | |
| | |
| | |
| | |
| | |
| | | |
~DYOUNG/vchoi_fork:add-werror-and-squash-some to develop
* commit 'f907b511d06612dafc7814a7c30f2f3d2b76d52b':
Oops, remove more C99 designated initializers for VS 2010 compatibility.
|
| | | |
|
| |\ \
| | |/
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
~DYOUNG/vchoi_fork:add-werror-and-squash-some to develop
* commit '62208b056a09c01855fbac7f75146be58ad6bfe5': (44 commits)
Add an #include to get a function declaration.
Don't use C99 designated initializers, they're not compatible with Visual Studio 2010.
Quiet some more maybe-uninitialized warnings---each is a false positive, *sigh*. This is more code that may not compile with VS2010, *sigh sigh*.
Always warn on maybe-uninitialized. -Wincompatible-pointer-types was not available until GCC 5, so enable it only if that's the GCC version we're using.
Only promote maybe-uninitialized warnings to errors on GCC 8. Even on GCC 8, there may be false positives at low optimization levels? I need to check.
Only use -Werror=cast-function-type with GCC 8 and later.
Put all of the -W options back into the order I found them in so that it's easier to compare old and new config/gnu-flags.
Add new source files to CMakeLists.txt.
Mention the -Werror= flags in libhdf5.settings.in.
free -> HDfree
Promote decleration-after-statement warnings to errors.
Quiet decleration-after-statement warnings.
Move a statement under some declarations since some vintages of Visual Studio don't like declarations after statements.
Document H5D__chunk_mem_xfree_wrapper().
Undo accidental test deletion.
Oops, delete a debug printf that snuck in here.
Undo my changes to the HD macros, hadn't really intended those to be on this branch....
Make errors of some more warnings. Move disabled warnings to 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 *.
Change hdset_reg_ref_t and H5R_ref_t from arrays of unsigned char to 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.
...
|
| | | |
|
| | |\
| | |/
| |/| |
|
| | |
| | |
| | |
| | | |
Visual Studio 2010.
|
| | |
| | |
| | |
| | |
| | | |
*sigh*. This is more code that may not compile with VS2010, *sigh
sigh*.
|
| | |
| | |
| | |
| | | |
available until GCC 5, so enable it only if that's the GCC version we're using.
|
| | |
| | |
| | |
| | |
| | | |
GCC 8, there may be false positives at low optimization levels? I need
to check.
|
| | | |
|
| | |
| | |
| | |
| | | |
easier to compare old and new config/gnu-flags.
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
Studio don't like declarations after statements.
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
this branch....
|
| | |
| | |
| | |
| | |
| | | |
DEVELOPER_WARNING_CFLAGS. Put just one warning option on a line, and
sort some of the options.
|
| | |
| | |
| | |
| | |
| | | |
(H5VL_token_t, too, probably should turn from a typedef for a
constant-length array to an array encapsulated in a struct.)
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
| | |
| | |
| | |
| | |
| | | |
enough to figure that out. Quiet some warnings by always initializing
`x` to 0.
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
and exclusively used in `datatype == H5T_COMPOUND` branches, so just
initialize it at its declaration so that GCC doesn't warn.
|
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
| | |
| | |
| | |
| | |
| | | |
should be heavily restructured to avoid the use of globals like `pass`,
but that's a project for another day and another person.
|
| | |
| | |
| | |
| | |
| | | |
moved some previously-static variables and functions that were causing
unused-variable/function warnings.
|
| | |
| | |
| | |
| | | |
into a few warnings in H5T_copy() itself.
|
| | |
| | |
| | |
| | |
| | | |
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.
|
| | |
| | |
| | |
| | | |
it is set before use by passing it as a reference to another function.
|
| | | |
|
| | |
| | |
| | |
| | | |
initialized when they're passed by reference to functions.
|
| | |
| | |
| | |
| | | |
does not realize are always set before use by `gettimeofday`.
|
| | |
| | |
| | |
| | | |
function/variable warnings.
|
| | | |
|
| | |
| | |
| | |
| | | |
being lost by a cast.
|
| | |
| | |
| | |
| | | |
of header-file templates to #defines.
|
| | |
| | |
| | |
| | | |
I cannot untangle right now.
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
integer.
|
| | |\
| | | |
| | | |
| | | | |
see if new warnings-as-errors have cropped up.
|
| | |\ \
| | | | |
| | | | |
| | | | | |
to pick up the NetBSD-compatibility changes that I recently merged.
|