summaryrefslogtreecommitdiffstats
path: root/hl/src/H5DS.c
Commit message (Collapse)AuthorAgeFilesLines
* VFD SWMR: Merges from develop (#1670)Dana Robinson2022-04-211-92/+80
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Cleans up SWMR test scripts (both legacy and VFD) * Brings over format source changes for Java * Normalization of HL, Fortran, C++, Java w/ develop * Removes the STATIC flavor of FUNC_ENTER macros (#1622) * Removes the STATIC flavor of FUNC_ENTER macros * Remove H5_NO_ALIGNMENT_RESTRICTIONS (#1426) * Do not conditionally compile code that uses a pointer dereference and assignment to copy a potentially unaligned variable to aligned automatic storage, or vice versa. Instead, always use naked `memcpy(3)`s. Disassembling the generated code reveals that the `memcpy(3)`s optimize (`-O3`) to a single `mov` instruction for x86_64, which is not strict about alignment. This change reduces the size of code and scripts by 143 lines, eases our way to cross-compilation, and avoids invoking undefined behavior. * Committing clang-format changes * Per discussion, use HD and add comments. Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com> * Removes unused definitions from module headers (#1624) * Misc stuff from develop (includes some parallel things) * Brings over SWMR test quiet mode changes from develop Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
* VFD SWMR: normalization with develop (#1472)Dana Robinson2022-03-061-4/+4
| | | Much normalization with develop. Still needs tools changes wrt VFD plugins.
* VFD SWMR: Normalization with develop (#1270)Dana Robinson2021-12-081-282/+618
| | | Brings many October-November changes from develop
* Normalized high-level library with developDana Robinson2021-04-051-115/+64
|
* Updates license url and history docsDana Robinson2021-03-251-1/+1
|
* Formats the source and updates the gcc warning pragmasDana Robinson2021-03-251-936/+903
|
* Squashed commit of the token_refactoring branch:Dana Robinson2020-05-201-62/+109
|
* Small changes from the token_refactoring branch, to reduce the delta to developQuincey Koziol2020-05-201-18/+18
|
* Revert "Make a squash commit of 'Quiet some warnings by adjusting warnings ↵David Young2020-05-201-1/+1
| | | | level and fixing some code.' (commit 5c911d8baf3)"
* Revert "Oops, remove more C99 designated initializers for VS 2010 ↵David Young2020-05-201-5/+1
| | | | | | compatibility." This reverts commit f907b511d06612dafc7814a7c30f2f3d2b76d52b.
* Oops, remove more C99 designated initializers for VS 2010 compatibility.David Young2020-05-201-1/+5
|
* Initialize a variable before its first use.David Young2020-05-201-1/+1
|
* Merge all of my changes from merge-back-to-feature-vfd_swmr-attempt-1,David Young2019-12-091-8/+8
| | | | | | | | including the merge of `hdffv/hdf5/develop`, back to the branch that Vailin and I share. Now I need to put this branch on a fork with a less confusing name than vchoi_fork!
* HD prefix and whitespaceAllen Byrne2018-08-281-53/+53
|
* Changes made based on feedback from pull request #1039.Vailin Choi2018-05-141-14/+14
|
* Fix for HDFFV-10180 Performance issues with H5Oget_info.Vailin Choi2018-04-241-14/+14
|
* 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-r26575] Removed compiler warning for hl/tools and hl/src.Scot Breitenfeld2015-03-251-30/+34
| | | | | | HDF5-250 and HDF5-241 tested: h5committest
* [svn-r26417] Removed compiler warnings from HL. HDF5-237Scot Breitenfeld2015-03-101-7/+8
| | | | Tested: h5committest
* [svn-r25755] Removed spurious casts in a few HDfree() statements.Dana Robinson2014-10-311-3/+3
| | | | | | | Discovered when I temporarily #defined HDfree() to a more complicated function while investigating something. Tested on a local linux VM. This is a very minor change.
* [svn-r25614] Fixed HDFFV-8670 Scot Breitenfeld2014-09-241-9/+56
| | | | *H5DSis_scale and other HL APIs do not null terminate string (and other issues)
* [svn-r25496] Description:Quincey Koziol2014-07-301-2/+7
| | | | | | | | | 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-r24376] Updated all code to use HDmalloc, HDcalloc, and HDfree.Dana Robinson2013-10-311-47/+47
| | | | | | | | | | | | | Prep work for adding a memory free function, part of HDFFV-8551. Tested on: 32-bit LE linux (jam) * with C++, Fortran * with thread safety on These are minor changes (almost entirely renaming foo functions to HDfoo) that don't change functionality. This should not require a full commit test.
* [svn-r24020] Description:Quincey Koziol2013-08-171-7/+1
| | | | | | | | Clean up compiler warnings Tested on: Mac OSX/64 10.8.4 (amazon) w/C++ & FORTRAN (too minor to require h5committest)
* [svn-r23241] Fix for: HDFFV-7596 H5DSget_scale returns size that includes ↵Scot Breitenfeld2013-02-111-1/+3
| | | | | | | | NULL termination and H5DSget_label returns size Updated tests. Tested: jam( gnu, intel)
* [svn-r21157] Description: JIRA HDFFV-7673: fixed segfault when retrieving ↵Scot Breitenfeld2011-07-311-13/+20
| | | | | | | | | the length of a a dimension scale that does not exist. Implemented a test for the situation and added an if condition depending on if the label name exists. Tested: jam (gnu and intel)
* [svn-r21124] Issue 2763 - adding OAPL_ID as a new parameter to ↵Raymond Lu2011-07-191-5/+5
| | | | | | | | | H5Rdereference. A followup commit for r21117. I made 2 changes: 1. Quincey changed H5P_DATASET_ACCESS_DEFAULT to H5P_DEFAULT for the tests that call H5Rdereference. Inside H5R_dereference, I assigned H5P_DATASET_ACCESS_DEFAULT to OAPL_ID if it is H5P_DEFAULT. 2. I added the test for invalid OAPL_ID in trefer.c. Tested on jam, koala, heiwa.
* [svn-r21121] Description:Quincey Koziol2011-07-181-1/+1
| | | | | | | | | Switch from H5P_DATASET_ACCESS_DEFAULT to H5P_DEFAULT for calls to H5Rdereference2(). Tested on: Mac OS X/32 10.6.8 (amazon) w/debug (too minor to require h5committest)
* [svn-r21117] Issue 2763 - I added a new parameter of object access property ↵Raymond Lu2011-07-181-6/+6
| | | | | | list to the function H5Rdereference. It's called H5Rdereference2 now. H5Rdereference function has been deprecated to H5Rdereference1. I also added some test cases in trefer.c. Tested on jam, heiwa, and koala.
* [svn-r20901] Remove if() that avoided a copy of the last element to itself. ↵Allen Byrne2011-05-251-4/+2
| | | | | | | | This was to solve an issue of a difference between 1.8 and trunk. 1.8 used the wrong variable in the if() and upon reflection the if() was determined to be unnecessary and complicated the logic. 1.8 tests would have treated the if() as always true with the result being the same as though the if() was absent. This change reflects that equivalance and will avoid a potential problem in 1.8. Tested: local linux, reviewed by two developers
* [svn-r20870] Description: Fixed HDFFV-7605: Dim Scales APIs- error when ↵Scot Breitenfeld2011-05-201-1/+1
| | | | | | | | unattaching dim. sacles * Fixed Loop counter when removing object in dimension set with more then one scale. Tested: jam (intel and gnu)
* [svn-r20061] Description:Quincey Koziol2011-02-081-164/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring changes from Coverity branch to trunk: r19930: Fix memory leaks involving VL attributes in h5repack and h5diff. The buffers in copy_attr and diff_attr were not checked for the presence of a vlen before being freed, and vlen storage was never reclaimed. Added checks and calls to H5D_vlen_reclaim(). r19933: Purpose: Fix memory leak in H5L_move_cb() Description: H5L_move_cb copied the source link using H5O_msg_copy() but freed it manually using H5MM_xfree(). Since H5O_link_copy allocates the link using H5FL_MALLOC, this causes the link to be allocated from the free list but is never put back on the free list when it is freed. This prevents the link free list from shutting down properly. Modified H5L_move_cb() and H5L_move_dest_cb() to free the link properly using H5O_msg_free(). r19973: Fix resource leaks by freeing string created by HD5f2string r19974: Issue #345: Inialize buf variable to null Tested on: Mac OS X/32 10.6.6 (amazon) w/debug & production (h5committested on Coverity branch)
* [svn-r18882] Maintenance: fixed a typo in email addressElena Pourmal2010-05-221-1/+1
|
* [svn-r18880] Bug fix: Elena Pourmal2010-05-221-44/+55
| | | | | | | | Fixed a bug (1855) in H5DSattach_scale, H5DSis_attached and H5DSdetach_scale caused by using H5Tget_native_type function to determine the native type for reading REFERENCE_LIST attribute. The bug was exposed on Mac PPC. Platforms tested: juniper (Mac PPC), jam, amani, linew
* [svn-r18818] Bug fix: Elena Pourmal2010-05-151-112/+94
| | | | | | | | | | | Fixed a bug in the H5DSdetach_scale function when 0 bytes were allocated after the last reference to a dim. scale was removed from the list of references in a VL element of the DIMENSION_LIST attribute; modified the function to comply with the Spec: DIMENSION_LIST attribute is deleted now when no dimension scales left attached. Platforms tested: jam with electric fence, amani amd linew
* [svn-r18197] Description:Quincey Koziol2010-01-301-41/+41
| | | | | | | | | 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-r17566] Fix for Bug 1562. H5DSis_attached needed to account for ↵Allen Byrne2009-10-011-4/+24
| | | | | | | | | platform dependent types. Added test and test files for LE and BE dimension scales. Reworked part of test_ds file as part of debug. Added missing H5Tclose in region reference routines. H5DS.c checked for other locations which need to account for platform dependent types in dealing with ds_list_t. Added generator program for test files of LE and BE dimension scales. Tested: local linux, h5committest
* [svn-r17268] Description:Quincey Koziol2009-07-301-3/+3
| | | | | | | | | | | Bring r17266 from 1.8 branch to trunk: Clean up various compiler warnings from IRIX64 build Tested on: Mac OS X/32 (amazon) debug & production (yes, I know it's not an IRIX64 system :-) Too minor to require h5committest
* [svn-r16719] clean linux warningsPedro Vicente Nunes2009-04-091-6/+6
| | | | tested: linux
* [svn-r16706] #1538 (B2) Problems with Dim Scale APIs reported by MathworksPedro Vicente Nunes2009-04-081-16/+22
| | | | | | | ISSUE2: the scale index input/output parameter value passed to H5DSiterate_scales was not always incremented (it returns the scale index current iteration). SOLUTION FOR ISSUE2: modified the cycle in H5DSiterate_scales so that the scale index is always incremented TEST: added some test cases with calls to invalid indices and H5DSiterate_scales with return scale indices and visitor data tested: windows, linux
* [svn-r16696] #1538 (B2) Problems with Dim Scale APIs reported by MathworksPedro Vicente Nunes2009-04-081-60/+102
| | | | | | | ISSUE: When some functions are called with an invalid dimension index (for example, greater than the rank of the dataset) the function does not immediately fail, rather asserts down the code. SOLUTION: added this check for H5DSdetach_scale, H5DSset_label, H5DSget_label (the other functions have this check) . H5DSiterate_scales also added a check for an invalid dimension scale index TEST: added some test cases with calls to invalid indices DOCS: Nothing added
* [svn-r16676] #1521 (B2) H5DSdetach_scale memory leakPedro Vicente Nunes2009-04-071-3/+11
| | | | | | | | ISSUE: Purify (Windows Visual Studio 6) complains of a memory leak in H5DSdetach_scale call regarding a H5Aread call (stack is H5A_read, H5T_convert, H5T_conv_vlen, H5T_vlen_seq_mem_write, H5MM_malloc). SOLUTION: When a scale is detached from a dataset, the variable length structure length field is decreased in one entry. The associated pointer must be reallocated with the new length. DOCS AND TEST: Nothing added tested: windows with purify, linux
* [svn-r16588] Pedro Vicente Nunes2009-03-191-1574/+1671
| | | | | | 1. #1487 (B1) DS memory leaks ISSUE 1: On a "go to" error condition, previously allocated buffers were not freed. NOTE: these are "potential" memory leaks because typically the error conditions do not occur, so the potential memory leaks also do NOT occur. ISSUE 2: A function used to read dimension scales realistic data (topography of the North Atlantic, latitude and longitude) was being called without the data buffer being freed. SOLUTION FOR ISSUE 1: added "free" calls for the allocated buffers on the error sections. SOLUTION FOR ISSUE 2: added a "free" call after the read_data function, after using the buffer (on a H5Dwrite and H5Screate_simple). The read_data function reads both data and dimensions from the ASCII data files. DOCS: no docs added. Done for 1.9 and 1.8 STATUS: Closed bug tested: h5committest
* [svn-r16549] Bug #608: Memory leak in H5DSset_label. Added code to free sub ↵Allen Byrne2009-03-051-225/+258
| | | | | | | | | string ptr's belonging to buffer in H5DSset_label and H5DSget_label. Also added free of buffers in error section of both functions. Potential memory leaks may exist elsewhere, and this will not close the bug. Tested: h5committest vista 32 VS2008
* [svn-r15462] Description:Quincey Koziol2008-08-121-3/+3
| | | | | | | | Correct compiler warnings from Visual Studio. Tested on: Mac OS X/32 10.5.4 (amazon) w/FORTRAN & C++ (Too minor to require full h5committest)
* [svn-r14225] Description:Quincey Koziol2007-10-311-16/+16
| | | | | | | | | | | | | | | | | | | Change H5Oget_info -> H5Oget_info_by_name and re-add H5Oget_info in a simpler form for querying a particular object, to align with other new API routines. Tested on: FreeBSD/32 6.2 (duty) in debug mode FreeBSD/64 6.2 (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/default API=1.6.x, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Mac OS X/32 10.4.10 (amazon) in debug mode
* [svn-r14222] Description:Quincey Koziol2007-10-301-2/+2
| | | | | | | | | | | | | | | | | | Change H5Adelete2 -> H5Adelete_by_name and un-deprecate H5Adelete1 back to H5Adelete, for better consistency with new other API routiens. Tested on: FreeBSD/32 6.2 (duty) in debug mode FreeBSD/64 6.2 (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/default API=1.6.x, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Mac OS X/32 10.4.10 (amazon) in debug mode
* [svn-r14218] Description:Quincey Koziol2007-10-301-5/+5
| | | | | | | | | | | | | | | | | | | | | Changed H5Acreate2 -> H5Acreate_by_name, to be more consistent with other new API routines. Re-added simpler form of H5Acreate2, which creates attributes directly on an object. Tested on: FreeBSD/32 6.2 (duty) in debug mode FreeBSD/64 6.2 (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/default API=1.6.x, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Mac OS X/32 10.4.10 (amazon) in debug mode
* [svn-r14217] Description:Quincey Koziol2007-10-301-13/+13
| | | | | | | | | | | | | | | | | | | | | Change H5Aopen -> H5Aopen_by_name, in order to be more consistent with other new API routines. Re-add H5Aopen as a simpler routine, to open attributes on a particular object. (Much like the old H5Aopen_name routine). Tested on: FreeBSD/32 6.2 (duty) in debug mode FreeBSD/64 6.2 (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/default API=1.6.x, w/C++ & FORTRAN, in production mode Linux/64-ia64 2.6 (cobalt) w/Intel compilers, w/C++ & FORTRAN, in production mode Solaris/32 2.10 (linew) w/deprecated symbols disabled, w/C++ & FORTRAN, w/szip filter, in production mode Mac OS X/32 10.4.10 (amazon) in debug mode
* [svn-r14198] formatted some commentsPedro Vicente Nunes2007-10-101-17/+17
| | | | tested: linux