summaryrefslogtreecommitdiffstats
path: root/tools/h5import/h5import.c
Commit message (Collapse)AuthorAgeFilesLines
* [svn-r27801] Minor warning fixes in the tools.Dana Robinson2015-09-161-11/+11
| | | | Tested on: jam (too minor for h5committest)
* [svn-r25197] Description:Quincey Koziol2014-05-171-7/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | Bring changes from hdf5_1_8_coverity branch back to trunk: r20878: Issue 76: Check if H5Tget_nmembers(type) fails and simply return(FALSE). Also move printf to after check. r20880: Issue 192: Create ret_val var set to -1. Add out label for failures to jump to, return ret_val at bottom. r20882: Fixes for coverity: 1) bug #1679: remove dead code in test/mf.c 2) bug #1680: remove dead code in tools/lib/h5diff_dset.c r20883: Fix coverity issue 585 Description: Changed variable "c" in processStrData in h5import.c to an int, to match fgetc return value, and removed call to feof, instead checking if c == EOF. Tested on: MacOSX/64 10.9.3 (amazon) w/C++, FORTRAN & parallel (too minor to require h5committest)
* [svn-r24804] Description:Quincey Koziol2014-03-161-9/+8
| | | | | | | | | | | | | | | | | | | | | | | | Brought changes from Coverity branch back to trunk, and cleaned up misc. other warnings & formatting issues: r20833: Fixed Coverity 667 and 668 with real integer overflow tests this time. r20834: Use HDstrncpy and HDstrncat. --gh r20835: Change to use strncpy - use base_len + 1 for line 156, use HDstrlen(path) + 1 for line 159 r20836: Fixed coverity 585 by casting output of fgetc() to a char. r20837: Changed sprintf calls to snprintf with size 1 less than the allocated buffer to address coverity issue #967. Tested on: Mac OSX/64 10.9.2 (amazon) w/C++, FORTRAN & parallel (too minor to require h5committest)
* [svn-r24066] Eliminate switch default warning by printing err message.Allen Byrne2013-08-231-9/+103
|
* [svn-r24020] Description:Quincey Koziol2013-08-171-3/+0
| | | | | | | | Clean up compiler warnings Tested on: Mac OSX/64 10.8.4 (amazon) w/C++ & FORTRAN (too minor to require h5committest)
* [svn-r22921] Description:Quincey Koziol2012-10-181-200/+169
| | | | | | | | | | | | | | | | | | | | | | | | | Review Coverity changes and bring them back to trunk. (QK & JK) r20402: Added #includes for h5tools.y or h5tools_utils.h as required to remedy implicit function declarations which caused compiler warnings and coverity issues 703-4 and 708-11. r20414: This is related to the previous checkin r20399. There were incorrect updates which caused incorrect behavior when no file was given. Also possible segfault when handling hyperslab options. Simplify the code changes. r20449: Description: Modified H5E_walk2_cb to check return value of H5I_object_verify. r20450: fixed coverity 813 Tested on: Mac OSX/64 10.8.2 (amazon) w/C++, FORTRAN, debug & threadsafe (too minor to require h5committest)
* [svn-r22589] Change strto functions to the HD prefix versions.Allen Byrne2012-07-201-5/+5
| | | | Tested: local
* [svn-r22588] HDFFV-721: h5dump supplies input for h5importAllen Byrne2012-07-191-252/+1758
| | | | | Tested: local linux with cmake jam with configure
* [svn-r22170] Removed extraneous svn executable properties from ↵Dana Robinson2012-03-281-0/+0
| | | | non-executable files.
* [svn-r22004] Reduced warnings and fixed conflicts resulting from including ↵Allen Byrne2012-02-281-4/+5
| | | | | | h5tools.h Tested: local linux
* [svn-r21985] Tools must call h5tools_init before command_line parsing or ↵Allen Byrne2012-02-241-0/+3
| | | | | | | | using tools_mesg functions. Tools library uses alais variables for stdout, stderr and tools library uses HDfprintf() functions. Tested: local linux
* [svn-r21982] Correct HD prefix in tools for exitAllen Byrne2012-02-241-2/+2
| | | | Checked for HD support.
* [svn-r21981] Correct HD prefix in tools for fprintfAllen Byrne2012-02-241-425/+425
| | | | Checked for HD support.
* [svn-r21366] JIRA HDFFV-2748 replace use of _WIN32 in source. Created new ↵Allen Byrne2011-09-061-2/+2
| | | | | | defines H5_HAVE_WIN32_API and H5_HAVE_VISUAL_STUDIO defines to use. These can be properly set during configurration. Tested: windows and local linux - reviewed internally
* [svn-r21091] Replace #ifdef WIN32 around 64 bit integers with ↵Allen Byrne2011-07-111-95/+90
| | | | | | H5_SIZEOF_LONG_LONG. This is set if 64-bit integers are supported. Tested: local linux
* [svn-r20875] Refactor h5import to close files and free allocations. Also ↵Allen Byrne2011-05-201-2206/+2111
| | | | | | corrected test dependencies so that memcheck would process tests in correct order. Tested: local linux - confirmed with valgrind
* [svn-r18759] pull r18757 from 1.8 branchAllen Byrne2010-05-111-0/+6
| | | | | | | | | Enable tools lib to be built as a dll on windows. Added two get/set functions for progname and d_status. Also add windows import/export declarations to functions. Updated error_mesg() and warn_mesg() to remove progname argument and use get functions Tested: Windows, linux
* [svn-r18197] Description:Quincey Koziol2010-01-301-10/+10
| | | | | | | | | 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-r18055] Bug fix: 1192Albert Cheng2009-12-261-1/+1
| | | | | | | | Description: Fixed exit code (sometimes return code in Main) to follow the HDF5 standards. Tested: H5committested plus serial test in Jam.
* [svn-r17980] Description:Quincey Koziol2009-12-101-5/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring Coverity changes into the trunk: (also other minor cleanups) r17955: Fix Coverity item 24. Add missing error condition to H5AC_ext_config_2_int_config. r17956: Fix Coverity item 24. Improve error checking in H5A_compact_build_table_cb. r17957: Fix Coverity item 150. Fix warning in H5A_compact_build_table_cb. r17958: Fix Coverity item 117. Fix error handling in H5B_shared_new. r17959: Fix Coverity item 209. Added an assertion for leaf->shared in H5B2_cache_leaf_dest. r17960: Fix Coverity item 208. Added an assertion for internal->shared in H5B2_cache_internal_dest. r17961: Fix Coverity item 89. Reworked the code to avoid array overrun in H5C__autoadjust__ageout__insert_new_marker. r17962: Fix for coverity Resource_leak 195,203,204,205. r17963: Fix Coverity item 44. Prevented potential NULL dereference in H5D_btree_debug. r17964: Fix Coverity issues #197, 198 & 199: memory not being released. (Also clean up other resource leaks in nearby and/or similar code). r17965: Fix Coverity issue #151: release resources on error r17966: Fix Coverity issue #187: Remove leftover code remnant from prior bugfix which was causing resource leak of open files. r17967: Fixed Coverity issues # 193 & 194. Removed unnecessary memory allocation and added comparison of length of path parameter to the size of the destination buffer in h5import.h/h5import.c. r17968: Fix Coverity item 144. Fixed memory leak on error in H5D_chunk_copy. r17969: Fix for coverity Resource_leak #196. r17970: Coverity 167-173: Initialized pointer of buffers. In error handling, closed types and free memory. 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-r17838] Description:Quincey Koziol2009-11-051-94/+94
| | | | | | | | | | | | | | | | | | | | | | | | | Further refactoring of v2 B-tree code, moving toward being able to pass a context information to a client's encode/decode callbacks. Also, clean up of other minor compiler warnings and code formatting issues. 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.1 (amazon) in debug mode Mac OS X/32 10.6.1 (amazon) w/C++ & FORTRAN, w/threadsafe, in production mode
* [svn-r17697] Bug fix: (1192)Albert Cheng2009-10-201-2/+2
| | | | | | | | Error exit code of -1 is illegal (exit code is unsigned). Changed it to EXIT_FAILURE (1). Also changed exit(0) to exit(EXIT_SUCCESS) for better coding. Tested: jam.
* [svn-r16568] #1462 By selecting a compression type, a big endian byte order ↵Pedro Vicente Nunes2009-03-111-2/+2
| | | | | | | | | | | | | | | | was being selected. When reading the compression parameter keyword, the compression type read flag was incorrectly set to read, removed this line of code in->configOptionVector[COMPRESS] = 1; Modified one configuration file to have the COMPRESSION-TYPE GZIP Keyword. Entered a bug description fix of - h5import: By selecting a compression type, a big endian byte order was being selected (PVN - 2009/11/3) tested: linux
* [svn-r15699] Bug fix #1281Pedro Vicente Nunes2008-09-251-6/+27
| | | | | | | | | | The TEXTFPE was described in the usage (read floats in scientific notation) but was not implemeneted. Solution: remove TEXTFPE from the help system and have the TEXTFPE input type do the same thing as TEXTPF currently does to support backwards compatibility (the format read for both TEXTFPE and TEXTPF is %f) added a test tested: windows, linux
* [svn-r15628] Description:Quincey Koziol2008-09-161-17/+17
| | | | | | | | | | | | | 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-r14903] Undoing change committed in r14902.Scot Breitenfeld2008-04-301-9/+46
|
* [svn-r14902] Merged fortran_1_8 branch changes r14505:14901 into the trunk. ↵Scot Breitenfeld2008-04-301-46/+9
| | | | New fortran wrappers added.
* [svn-r14613] 971 (B2) h5import bug on Windows w/binary datasets. CLOSED. ↵Pedro Vicente Nunes2008-02-191-9/+46
| | | | | | | | | | | | | | | | | | fread in windows needs a binary file to be open with "rb" instead of "r" otherwise it terminates execution if an end of file character is found on the input file. Besides that the binary file generated needs to be open with "wb" , otherwise an end of line character is read twice. DONE NOW for 1.8, already done previously for 1.6 renamed the h5import test files to have the extensions text input files = .txt binary input files = .bin configuration files = .conf hdf5 files = .h5 besides that in very test the files have the same name except extension. For example TOOLTEST txtin16.txt -c $srcdir/testfiles/txtin16.conf -o txtin16.h5 The convention for the test name is for example, for "txtin16" "txt" for text then "in16" means integer 16 size Tested: linux, solaris
* [svn-r14227] new feature: add a -V version flag for h5importPedro Vicente Nunes2007-10-311-0/+9
| | | | tested: windows, linux
* [svn-r14199] Description:Quincey Koziol2007-10-111-18/+11
| | | | | | | | | | | | | | | | | | | | | Add H5Dcreate to API versioned routines, replacing internal usage with H5Dcreate2 Fix thread-safe error stack initialization for API versioned error stack printing 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-r14115] Description:Quincey Koziol2007-08-281-2/+2
| | | | | | | | | | | | | | | | | Remove all plain calls to H5Gopen() from source, replacing them with either H5Gopen2(). Add test for H5Gopen1(). Reformatted several pieces of code, to clean them up. Tested on: FreeBSD/32 6.2 (duty) FreeBSD/64 6.2 (liberty) Linux/32 2.6 (kagiso) Linux/64 2.6 (smirom) Solaris/32 5.10 (linew) Mac OS X/32 10.4.10 (amazon)
* [svn-r14104] Description:Quincey Koziol2007-08-231-36/+27
| | | | | | | | | | | | | Pursue calls to H5Gcreate() relentlessly and ruthlessly exterminate them, leaving only a few tame specimens in text files and comments. ;-) Tested on: Mac OS X/32 10.4.10 (amazon) FreeBSD/32 6.2 (duty) FreeBSD/64 6.2 (liberty) Linux/32 2.6 (kagiso) Linux/64 2.6 (smirom) Solaris/32 5.10 (linew)
* [svn-r14071] Pedro Vicente Nunes2007-08-101-1/+1
| | | | | | typo fix in usage message tested: linux
* [svn-r14069] typo fix in usage messagePedro Vicente Nunes2007-08-101-1/+1
| | | | tested: linux
* [svn-r14068] typo fix in usage messagePedro Vicente Nunes2007-08-101-1/+1
| | | | tested: linux
* [svn-r14044] Pedro Vicente Nunes2007-08-071-1/+1
| | | | | | updated the usage message to say the STR option is implemented tested: windows, linux
* [svn-r14039] Pedro Vicente Nunes2007-08-071-2/+196
| | | | | | | | | | | | | New feature: implementation of h5import conversion of an ASCII plain-text file containing text data The string type H5T_C_S1 is used to define the data (the datum is defined here as one line of text in the text file). The size is set to variable length (H5T_VARIABLE) The space used is a 1D array with as many elements as there are lines in the ASCII file (a line is defined by the inclusion of an end of line character, ASCII number 10). A first traversal of the input text file must be made to determine the number of lines in the file and thus the dimensionality of the dataset. New test to the test script added text input files and teststr.h5 for h5dump to compare added Tested: windows, linux, solaris
* [svn-r14000] Pedro Vicente Nunes2007-07-231-2/+2
| | | | | | | bug fixes: help message had incorrections, typos tested: linux
* [svn-r13999] Pedro Vicente Nunes2007-07-231-279/+279
| | | | | | | purpose: bug 791 redirect help output to stdout rather to stderr tested: windows, linux
* [svn-r13998] Pedro Vicente Nunes2007-07-231-3/+3
| | | | | | typo fixes in error messages tested: linux
* [svn-r13766] In Visual Studio 2005 for 64-bit, _WIN32 is defined, but not ↵Scott Wegner2007-05-181-8/+8
| | | | | | | WIN32, so I've standardized all #ifdef's to use _WIN32. This should not affect any other platform. Tested: Visual Studio (32- and 64-bit) on Win XP
* [svn-r13253] Updated all C and C++ style source code files with the THG ↵Albert Cheng2007-02-071-2/+3
| | | | | | | | | copyright notice. Tested platform: Kagiso only since it is only a comment block change. If it works in one machine, it should work in all, I hope. Still need to check the parallel build on copper.
* [svn-r13064] Pedro Vicente Nunes2006-12-141-5/+5
| | | | | | | | | | | | | | | | | | | Fix several bugs 1) the parsing of subsetting was using atoi to convert the parameter to an int, which caused problems for numbers greater that int. Substitute with atof 2) the printing of indices in the subsetting case was not being done. Solution: calculate the element position at the start of the subsetting using the algorythm Given an index I(z,y,x) its position from the beginning of an array of sizes A(size_z, size_y,size_x) is given by Position of I(z,y,x) = index_z * size_y * size_x + index_y * size_x + index_x And pass that position to the function that dumps data, h5tools_dump_simple_data. 3) several index counters were declared as int, use hsize_t instead 4) modified the test generation program so that it includes test cases for subsetting of 1d, 2d, 3d, and 4d arrays and add these tests to the shell script
* [svn-r12140] Purpose:MuQun Yang2006-03-231-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | portable issue on windows Description: setvbuf on visual studio 2005 needs the size to be greater or equal to 2. <description of setvbuf from msdn library> Buffer size in bytes. Allowable range: 2 <= size <= INT_MAX (2147483647). Internally, the value supplied for size is rounded down to the nearest multiple of 2. h5import used linebuffer option of setvbuf for stderr and stdout and the size is 0 by default. This causes core dump with visual 2005 on windows 64-bit. Solution: 1. Use HDsetvbuf inside h5import.c, 2. Define HDsetvbuf inside windows H5pubconf.h for windows to work around the core dump of h5import test. This is probably a bug inside visual studio 2005. Platforms tested: h5committest( shanti refuses to be connected) windows 6.0 at XP visual 2005 at XP-64bit Misc. update:
* [svn-r11886] Purpose:Quincey Koziol2006-01-231-18/+17
| | | | | | | | | | | | | Code cleanup Description: Check in some of the code cleanups from working on the external link support. (This doesn't include any of the external link features) Platforms tested: FreeBSD 4.11 (sleipnir) Mac OSX.4 (amazon) Linux 2.4
* [svn-r11245] Purpose:Quincey Koziol2005-08-131-351/+351
| | | | | | | | | | | | | | | | | | | | Code cleanup Description: Trim trailing whitespace, which is making 'diff'ing the two branches difficult. Solution: Ran this script in each directory: foreach f (*.[ch] *.cpp) sed 's/[[:blank:]]*$//' $f > sed.out && mv sed.out $f end Platforms tested: FreeBSD 4.11 (sleipnir) Too minor to require h5committest
* [svn-r9934] Purpose:Robert E. McGrath2005-02-041-1/+17
| | | | | | | | | | | | | | | Bug Description: Refinement to bug fix #282 Solution: Altered argument handling to do a better default. Platforms tested: verbena, shanti Misc. update:
* [svn-r9927] Purpose:Robert E. McGrath2005-02-031-0/+30
| | | | | | | | | | | | | | | Bug fix #282 Description: Problem with byte order Solution: Do it right. See 1.6 checkin Platforms tested: verbena, shanti Misc. update:
* [svn-r8383] Purpose:Quincey Koziol2004-04-181-13/+1
| | | | | | | | | | | | | Code cleanup Description: Clean up lots of warnings based on those reported from the SGI compilers as well as gcc. Platforms tested: SGI O3900, IRIX64 6.5 (Cheryl's SGI machine) FreeBSD 4.9 (sleipnir) w/ & w/o parallel h5committest
* [svn-r7574] Purpose:Bill Wendling2003-10-081-28/+28
| | | | | | | | | | | | | | | | Code Improvement Description: Changed from passing a structure into a function by value into passing it in by pointer. Noticed this while compiling with a highly optimizing compiler which took >30 minutes to analyze the program (granted, this is the compiler's fault, but in general, it's better to pass large structures in by pointer and not by value). Platforms tested: Linux (specific to h5import, so only needed to test on one platform) Misc. update:
generated by cgit v0.12 at 2024-07-22 04:24:17 (GMT)