summaryrefslogtreecommitdiffstats
path: root/test/external.c
Commit message (Collapse)AuthorAgeFilesLines
* Update clang config (#473)Quincey Koziol2021-03-171-10/+40
| | | | | | | * Update clang config to put H5E_BEGIN_TRY / H5E_END_TRY on separate lines, empty C++ methods on separate lines, understand that ALL_MEMBERS / UNIQUE_MEMBERS are foreach macros, and properly skip the 'config' directory in the find command without emiting a warning * Committing clang-format changes Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* Fixed all clang-tidy bugprone-suspicious-string-compare warnings (#451)Sean McBride2021-03-101-1/+1
| | | | | | | | | * Fixed all clang-tidy bugprone-suspicious-string-compare warnings This change was generated entirely by clang-tidy itself. * Reformat code with clang v10.0.1. Co-authored-by: Larry Knox <lrknox@hdfgroup.org>
* Various clang tidy warning fixes (#448)Sean McBride2021-03-101-3/+3
| | | | | | | | | | | | | | | | | | | | | | | * Fixed clang-tidy bugprone-reserved-identifier warnings * Fixed clang-tidy bugprone-assert-side-effect warnings * Fixed clang-tidy bugprone-copy-constructor-init warning * Fixed clang-tidy readability-redundant-preprocessor warning For error_test.c the removed code was already dead, because it was in the else of an `#if H5_USE_16_API` block. Based on H5Location.h, I think p_get_ref_obj_type was meant to be in `#ifndef DOXYGEN_SHOULD_SKIP_THIS` and an `#endif` was missing. Similarly, in the header, getObjTypeByIdx is only in H5_NO_DEPRECATED_SYMBOLS, not DOXYGEN_SHOULD_SKIP_THIS. * Fixed clang-tidy readability-redundant-string-init warnings * Fixed some clang-tidy performance-type-promotion-in-math-fn warnings * Fixed clang-tidy performance-unnecessary-value-param warnings * Reformat source with clang v10.0.1. Co-authored-by: Larry Knox <lrknox@hdfgroup.org>
* Update license url (#332)Larry Knox2021-02-171-1/+1
| | | | | | * Modify temporary rpath for testing in java example scripts. * Update URL in source file Copyright headers for web copy of COPYING file - src and test directories.
* HD prefix updates in src/ and test/Dana Robinson2020-10-241-2/+2
| | | | | | | Adds missing HD prefixes to API calls in src and test. Adds some extra processing to bin/checkposix to keep the noise levels down when running the script (not comprehensive).
* Clang-format of source filesAllen Byrne2020-09-301-445/+490
|
* Pre-format fixes for warnings as errorsAllen Byrne2020-09-291-24/+18
|
* Removes staff email addresses from the repositoryDana Robinson2020-08-071-1/+1
| | | | Removes redundant C library headers from hl library
* Trim trailing whitespaceQuincey Koziol2020-04-201-1/+1
|
* Move function-scope macro definition to file-scope.Jacob Smith2019-09-301-5/+6
|
* Refactor external file addition loop to static function.Jacob Smith2019-09-291-18/+57
|
* fix declaring variable in for statementJacob Smith2019-09-281-9/+10
|
* Merge branch 'develop' of ↵Jacob Smith2019-09-231-13/+13
|\ | | | | | | https://bitbucket.hdfgroup.org/scm/~jake.smith/hdf5 into bugfix/repack_external_storage
| * Add HD prefix to testsAllen Byrne2019-08-151-7/+7
| |
| * Fix conflicting types for reset_raw_data_files() parameter in ↵Larry Knox2019-08-011-6/+6
| | | | | | | | | | | | external_common.c changed from int to hbool_t. Change 0 and 1 to FALSE and TRUE in calls to reset_raw_data_files().
* | Change print calls to HD-wrapped.Jacob Smith2019-09-231-3/+2
| | | | | | | | Minor formatting tweaks (inc. regressions and in-file style-matching).
* | Merge branch 'develop' of ↵Jacob Smith2019-07-221-0/+1
|\ \ | |/ | | | | https://bitbucket.hdfgroup.org/scm/~jake.smith/hdf5 into bugfix/repack_external_storage
| * Add support for GCC9, update warnhist script, and clean up warnings.Quincey Koziol2019-07-031-1/+2
| |
* | Minor formatting adjustments.Jacob Smith2019-07-221-74/+47
| |
* | Some minor reformatting.Jacob Smith2019-05-241-193/+241
|/ | | | Replace tabs with spaces.
* Changing file names to avoid possible conflicts during parallel test.Songyu Lu2019-04-221-6/+6
|
* Left out this file in previous commit.Songyu Lu2019-04-091-11/+10
|
* This commit basically has the following changes:Songyu Lu2019-04-041-230/+1
| | | | | | 1. restored the datatype, dataspace, and LCPL of the dataset for VOL connector back to the properties. 2. splitted external.c and vds.c because they called HDsetenv in the program, instead using shell scripts to set the environment variables. 3. changed H5CX_get_vds_prefix and H5CX_get_ext_file_prefix to use H5P_peek instead of H5P_get.
* Rework of the POSIX file open permissions and macros to clean upDana Robinson2017-04-281-4/+4
| | | | | | HDopen() calls. Also fixed a minor const warning in the core VFD.
* Changed the Windows POSIX open() file permissions to be correctDana Robinson2017-04-281-4/+4
| | | | according to MSDN. Partial fix for HDFFV-9630.
* Merge pull request #426 in HDFFV/hdf5 from ~LRKNOX/hdf5_lrk:hdf5_1_10 to ↵Larry Knox2017-04-251-6/+4
| | | | | | | | | | hdf5_1_10 * commit '54957d37f5aa73912763dbb6e308555e863c43f4': Commit copyright header change for src/H5PLpkg.c which was added after running script to make changes. Add new files in release_docs to MANIFEST. Cimmit changes to Makefile.in(s) and H5PL.c that resulted from running autogen.sh. Merge pull request #407 in HDFFV/hdf5 from ~LRKNOX/hdf5_lrk:hdf5_1_10_1 to hdf5_1_10_1 Change copyright headers to replace url referring to file to be removed and replace it with new url for COPYING file.
* [svn-r29698] Switched true and false to TRUE and FALSE in a function inDana Robinson2016-04-141-2/+2
| | | | | | | test/external.c. Tested on: 64-bit Ubuntu 15.10 w/ gcc 5.2.1 autotools serial (external test only)
* [svn-r29421] Fix for HDFFV-9516.Dana Robinson2016-03-151-7/+110
| | | | | | | | Corrects a problem where H5Dget_access_plist() returns a dapl that does not contain all dapl fields. Tested on: 64-bit Ubuntu 15.10 w/ gcc 5.2.1 autotools serial
* [svn-r29182] Re-commit of HDFFV-8740 fix, which adds the ability to configureDana Robinson2016-02-221-171/+732
| | | | | | | | | | | | external dataset storage path behavior. This check-in fixes a bug in the original check-in where the external path stored in the file struct was not copied on reopen causing subsequent dataset operations to fail. Tested on: 64-bit Ubuntu 15.10 (Linux 4.2.0 x86_64) gcc 5.2.1 autotools serial w/ fortran and C++ autotools parallel (MPICH 3.1.4) w/ fortran
* [svn-r29119] Converted HDoff_t to off_t in test/external.c.Dana Robinson2016-02-151-116/+114
| | | | | | | Fixes Windows errors. Tested on: 64-bit Ubuntu 15.10 w/ gcc 5.2.1 64-bit Windows 10 w/ VS2015
* [svn-r29084] Revert of:Dana Robinson2016-02-111-649/+90
| | | | | | | | | | | | | | | r29069 and 72 (gheap cleanup) r29041 and 43 (HDFFV-8740 external storage) These cause problems in the daily tests. They will be re-introduced when their respective problems have been addressed. Tested on: 64-bit Ubuntu 15.10 (Linux 4.2.0 x86_64) gcc 5.2.1 autotools serial (check-vfd) w/ Fortran and C++ autotools parallel w/ Fortran CMake serial
* [svn-r29043] Fixes to make relative external storage work on Windows. Part ↵Dana Robinson2016-02-041-3/+3
| | | | | | | | | of HDFFV-8740. - Fixed typo in Wsetenv(). - Changed HDoff_t back to off_t in test/external.c. The function signature is off_t so the types must match or you can get conversion errors. Tested on 64-bit Windows 10 w/ VS 2015
* [svn-r29041] Patch from Steffen Kiess that allows a relative path to be ↵Dana Robinson2016-02-041-88/+647
| | | | | | | | | | | specified for external dataset files set via H5Pset_external(). Resolves HDFFV-8740 Tested on: 64-bit Ubuntu 15.10 (Linux 4.2.0 x86_64) gcc 5.2.1 autotools serial
* [svn-r29035] Removed external links test from external storage test.Dana Robinson2016-02-021-104/+0
| | | | | | | | This test is duplicated more comprehensively in the links.c file so there is no need to copy it over. Tested on: 64-bit Ubuntu 15.10 (Linux 4.2.0 x86_64) gcc 5.2.1 serial autotools
* [svn-r29034] Update the external test to use the default and latest file format.Dana Robinson2016-02-021-28/+60
| | | | | Tested on: 64-bit Ubuntu 15.10 (Linux 4.2.0 x86_64) gcc 5.2.1 serial autotools
* [svn-r29030] Cleanup of test/externals before merging user patch.Dana Robinson2016-02-021-541/+593
| | | | | Tested on: 64-bit Ubuntu 15.10 (Linux 4.2.0 x86_64) gcc 5.2.1 serial autotools
* [svn-r25496] Description:Quincey Koziol2014-07-301-1/+1
| | | | | | | | | Merge 64-bit ID changes from branch to trunk. (Plus a few minor cleanups that aren't on the branch) Tested on: Mac OSX/64 10.9.4 (amazon) w/C++ & FORTRAN (h5committested on branch already for a week)
* [svn-r22287] Description:Quincey Koziol2012-04-161-1/+5
| | | | | | | | Clean up more FUNC_ENTER/FUNC_LEAVE macros and move H5D & H5T code toward the final design (as exemplified by the H5EA & H5FA code). Tested on: Mac OSX/64 10.7.3 (amazon) w/debug & parallel
* [svn-r20529] Purpose: Add testing for bug 1864Neil Fortner2011-04-151-0/+4
| | | | | | | | | Description: Added internal tesitng routines to traverse a file and verify that symbol table information is *always* cached, whenever possible. Added this check to the end of many tests to check all the test files, right before the call to h5_cleanup. Tested: jam, amani, heiwa (h5committest)
* [svn-r19813] 1. Correct mistake for previous checkin rev #19781:Vailin Choi2010-11-171-1/+1
| | | | | | | file name listed for DISTCLEANFILES should be testlinks_env.sh 2. Add comments to test_4() in test/external.c about the link name "/ link". 3. Fix memory leak as reported by valgrind in src/H5Lexternal.c: free memory pointed to by tmp_env_prefix for HDF5_EXT_PREFIX case.
* [svn-r18419] Description:Quincey Koziol2010-03-181-0/+1
| | | | | | | | | Clean up srcdir querying code more, extracting it into single header file, to avoid compiler warnings. Tested on: Mac OS X/32 10.6.2 (amazon) w/debug & prod Linux/32 2.6 (jam)
* [svn-r18413] Description:Quincey Koziol2010-03-171-10/+1
| | | | | | | | | | | | | | Unify srcdir handling for test executables and allow them to use the srcdir setting from configure time without requiring the 'srcdir' environment variable be set (although you still can, to override the built in setting). Attempted to get this right for Windows builds also. Also add dependency between src/H5Tinit.c and src/libhdf5.settings, so that the test/testcheck_version.sh script works correctly. Tested on: Linux/32 2.6 (jam) Mac OS X/32 10.6.2 (amazon)
* [svn-r18197] Description:Quincey Koziol2010-01-301-2/+2
| | | | | | | | | Trim trailing whitespace from source code files with this command: find . \( -name "*.[ch]" -or -name "*.cpp" -or -name "*.f90" \) -print |xargs -n 1 sed -i "" 's/[[:blank:]]*$//' Tested on: None - eyeballed only
* [svn-r17953] Description:Quincey Koziol2009-12-031-20/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring Coverity changes into the trunk: r17877: Error 266: Uninitialized memspace set to -1. Changed malloc and free to HDmalloc and HDfree. Removed unused dtype var. r17878: Error 265: Uninitialized mem_space set to -1. Changed malloc and free to HDmalloc and HDfree. Error 267: Uninitialized smspace set to -1. Changed malloc and free to HDmalloc and HDfree. r17879: Error 242: Uninitialized aid set to -1. Error 243: Uninitialized sid set to -1. Uninitialized tid set to -1 for consistency r17880: Error 242: reinitialized aid to -1 after close to be ready for reuse. Error 243: reinitialized sid to -1 after close to be ready for reuse. reinitialized tid to -1 for consistency after close to be ready for reuse. r17881: use valgrind to check there is a memory leak. The fix is to free ptrstr in line 5838 at xml_dump_group() of h5dump.c after it is used. run the valgrind after the fix, no memory leak for that part of the code. r17882: Fix Coverity items 256 and 269. r17883: Error 222-230: initialized hid_t vars to -1 after close to be ready for reuse. Also added H5Tclose for tid in gent_bigdims r17884: Bug fix (Coverity run2, view 23, dead code) (this is pair-program done by Albert, Elena and Larry). Confirmed and fixed the deadcode in hyperslab read branch of function diff_datasetid. (Discovered other bad code that diff_datasetid() should be recoded. Bug 1693 is entered for this.) r17906: Fix Coverity item 260. r17907: 262: Initialized hid_t's dtype, dtype_tmp and file to -1. Initialized H5T_t * dt to NULL. r17908: Fix Coverity item 261. r17909: Fix Coverity item 248. r17910: Revise fix for Coverity item 248. r17911: Resolved coverity issues #s 263, 162, 163, 164. All issues in dsets.c. Initialized fid and did hid_t's. filter_corrupt function was returning in the middle of an if statement, bypassing free calls. Updated error handling to free buffers and hid_t's appropriately. r17912: (done by Larry and Albert) Cleanup Coverity view warnings (#231-241) about using uninitialized variables. Initialized all of them. r17913: Resolved issue 251 and 264. Initialized tid2 to -1 and initialized buffers that were freed in case of an error. r17914: Resolved coverity issues 66, 220, and 221: 66: Negative Return assignment ignored 220,221: Initialized hid_t's. r17915: Fix Coverity item 247. r17916: Fix Coverity item 246. r17917: Fix Coverity item 245. r17918: Fix Coverity item 244. r17919: Coverity Issue #84: Moved asserts in H5T_cmp to the top of the function, and converted them to HDassert. Coverity complaining about using potentially NULL pointer without checking it. Want to see if Coverity will accept Assertions as acceptable checking before using the value. Tested on: FreeBSD/32 6.3 (duty) in debug mode FreeBSD/64 6.3 (liberty) w/C++ & FORTRAN, in debug mode Linux/32 2.6 (jam) w/PGI compilers, w/default API=1.8.x, w/C++ & FORTRAN, w/threadsafe, in debug mode Linux/64-amd64 2.6 (smirom) w/Intel compilers, w/default API=1.6.x, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.4 (tg-login3) w/parallel, w/FORTRAN, in debug mode Linux/64-amd64 2.6 (abe) w/parallel, w/FORTRAN, in production mode Mac OS X/32 10.6.2 (amazon) in debug mode Mac OS X/32 10.6.2 (amazon) w/C++ & FORTRAN, w/threadsafe, in production mode
* [svn-r16144] Removed the check of H5Dset_extent for the space to be ↵Pedro Vicente Nunes2008-12-011-0/+11
| | | | | | chunked, since external datasets can be extended Tested: windows, linux
* [svn-r16142] Added a check to H5Dset_extent for the space to be chunked, ↵Pedro Vicente Nunes2008-12-011-11/+0
| | | | | | | | exit with error otherwise A call to H5Dset_extent in the external test was removed Tested: windows, linux
* [svn-r15868] Description:Quincey Koziol2008-10-151-94/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | Correct a minor error in file free space allocation which was affecting the 'multi' VFD and preventing some tests from fully working with it. Wholesale revisitation of all the places where tests were disabled with various VFDs and remove or correct all these so that _only_ the tests which _really_ can't work with particular VFDs are skipped during a 'make check-vfd' test. Tested on: Mac OS X/32 10.5.5 (amazon) in debug mode Mac OS X/32 10.5.5 (amazon) w/C++ & FORTRAN, w/threadsafe, in production mode FreeBSD/32 6.3 (duty) in debug mode FreeBSD/64 6.3 (liberty) w/C++ & FORTRAN, in debug mode Linux/32 2.6 (kagiso) w/PGI compilers, w/C++ & FORTRAN, w/threadsafe, in debug mode Linux/64-amd64 2.6 (smirom) w/Intel compilers w/default API=1.6.x, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.4 (tg-login3) w/parallel, w/FORTRAN, in production mode Linux/64-amd64 2.6 (abe) w/parallel, w/FORTRAN, in production mode
* [svn-r15628] Description:Quincey Koziol2008-09-161-2/+2
| | | | | | | | | | | | | Remove trailing whitespace from C/C++ source files, with the following script: foreach f (*.[ch] *.cpp) sed 's/[[:blank:]]*$//' $f > sed.out && mv sed.out $f end Tested on: Mac OS X/32 10.5.5 (amazon) No need for h5committest, just whitespace changes...
* [svn-r15150] Purpose: Separate Windows function macro definitions to win32defs.hScott Wegner2008-06-051-15/+15
| | | | | | | | | | | | | | Description: In library code, we try not to use system calls directly, but instead use the HD{function} macro instead. This way, we can map special versions of the call on particular systems. Previously, it was all done in H5private.h. However, in an effort to clean up platform-specific definitions, we move all of the Windows macros into a separate file, win32defs.h. This way, we can use the non-Posix versions that Visual Studio sends warnings about. Some macros are set specifically in the platform-specific header files. Then, any macros left unset will be set by the "default" implementation in H5private.h. This checkin also cleans up various source files to use the HD* macros when possible. Tested: VS2005 on WinXP VS.NET on WinXP h5committest (kagiso, linew, smirom)
* [svn-r14283] Description:Quincey Koziol2007-11-241-2/+1
| | | | | | | | Open external link file with read-only access, to avoid permissions issue in certain circumstances. Tested on: FreeBSD/32 6.2 (duty) w/read-only testfile