- 17 Oct, 2017 5 commits
-
-
Morris Jette authored
-
Brian Christiansen authored
The problem was the tests set the same variable, opt_arg, for for options, --test_suite and --test_suite_reg. But with the addition of de24ce72 the variable was being set with the different numbers out of order because the variable was being set from the environment. So the fix is to have the options set their own variable. Bug 4176
-
Tim Wickberg authored
-
Brian Christiansen authored
to avoid name collisions with c++ list types. gdb started showing the wrong type when the full lib was created.
-
Brian Christiansen authored
to set -O0 since --enable-developer and --enable-debug set optimizations.
-
- 16 Oct, 2017 28 commits
-
-
Danny Auble authored
-
Danny Auble authored
-
Danny Auble authored
-
Danny Auble authored
-
Danny Auble authored
-
Danny Auble authored
What appears to have happened is we changed only the non-internal setup. So technically this isn't needed, but does help with handling confusion later.
-
Brian Christiansen authored
-
Danny Auble authored
-
Danny Auble authored
-
Danny Auble authored
-
Morris Jette authored
-
Morris Jette authored
Coverity CID 44977
-
Morris Jette authored
-
Morris Jette authored
_alloc_mask() can return NULL, which was not handled. Coverity CID 44859
-
Morris Jette authored
-
Tim Wickberg authored
Kept separate for clarity.
-
Tim Wickberg authored
Bug 4014.
-
Tim Wickberg authored
-
Danny Auble authored
-
Morris Jette authored
Note: This check is needed since the maximum file path length really is file system dependent and not something local to the Linux distro Coverity CID 45200
-
Morris Jette authored
Coverity CID 45343
-
Morris Jette authored
Really should not happen, but this adds checks Coverity CID 171449
-
Morris Jette authored
Coverity CID 45167
-
Morris Jette authored
-
Morris Jette authored
Coverity CID 44823
-
Morris Jette authored
-
Morris Jette authored
-
Morris Jette authored
Coverity CID 45160
-
- 14 Oct, 2017 7 commits
-
-
Felip Moll authored
Setting -Werror tricks the test into failing on the "error: In the GNU C Library, "major" is defined by <sys/sysmacros.h>. For historical compatibility, it is currently defined by <sys/types.h> as well, but we plan to remove this soon. To use "major", include <sys/sysmacros.h> directly. If you did not intend to use a system-defined macro "major", you should undefine it after including <sys/types.h>." error. Since the normal Slurm build uses -Werror, this warning on including both headers will then cause the build itself to fail. Bug 3982.
-
Tim Wickberg authored
deadline_time_limit is ensuring this is always set, but the optimizer appears to not be able to work out the flow here.
-
Tim Wickberg authored
Move test above the accounting limit check. Change the test to verify first_job_ptr instead of pack_job_id to avoid warning about a possible dereference of an uninitialized variable lower down. (Note that both pack_job_id and first_job_ptr are set at the same time, so this test is functionally equivalent but able to convince the compiler that things will be okay.)
-
Tim Wickberg authored
-
Tim Wickberg authored
This one's a bit confusing, and should be a false positive, but initializing the saveptr argument to strtok_r quiets things down.
-
Tim Wickberg authored
Treat default case as an error. Visible with GCC 6.3 and -01.
-
Morris Jette authored
-