Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hdf5 merge issue 487 v112 #491

Merged
merged 41 commits into from
Mar 26, 2021
Merged

Hdf5 merge issue 487 v112 #491

merged 41 commits into from
Mar 26, 2021

Commits on Nov 23, 2020

  1. Configuration menu
    Copy the full SHA
    bcaf211 View commit details
    Browse the repository at this point in the history

Commits on Nov 26, 2020

  1. Configuration menu
    Copy the full SHA
    9b9ef60 View commit details
    Browse the repository at this point in the history
  2. First cut of the H5 public API documentation. (#80)

    * First cut of the H5 public API documentation.
    
    * Added H5Z "bonus track."
    
    * Applied Quincey's patch.
    
    * Added the missing patches from Quincey's original patch.
    
    * H5PL (complete) and basic H5VL API documentation.
    
    * Added H5I API docs.
    
    * Added H5L API docs.
    
    * First installment from Elena's H5T batch.
    
    * Second installment of Elena's H5T batch.
    
    * Final installment of Elena's H5T batch.
    gheber authored and lrknox committed Nov 26, 2020
    Configuration menu
    Copy the full SHA
    7a78fd1 View commit details
    Browse the repository at this point in the history
  3. Full set of current H5F documentation. (#105)

    * First cut of the H5 public API documentation.
    
    * Added H5Z "bonus track."
    
    * Applied Quincey's patch.
    
    * Added the missing patches from Quincey's original patch.
    
    * H5PL (complete) and basic H5VL API documentation.
    
    * Added H5I API docs.
    
    * Added H5L API docs.
    
    * First installment from Elena's H5T batch.
    
    * Second installment of Elena's H5T batch.
    
    * Final installment of Elena's H5T batch.
    
    * Migrated documentation for SWMR functions.
    
    * Catching up on MDC functions.
    
    * Integrated the H5F MDC function documentation.
    
    * Added MDC and parallel H5F functions.
    
    * Slightly updated main page.
    
    * Added doxygen/dox/H5AC_cache_config_t.dox to MANIFEST.
    gheber authored and lrknox committed Nov 26, 2020
    Configuration menu
    Copy the full SHA
    ba77da0 View commit details
    Browse the repository at this point in the history
  4. Doxygen - added (mostly) beginner functions (#112)

    * Doxygen - added (mostly) beginner functions
    
    * Removed duplicate H5Pset_szip function
    bljhdf authored and lrknox committed Nov 26, 2020
    Configuration menu
    Copy the full SHA
    3c0394b View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    3360ebd View commit details
    Browse the repository at this point in the history

Commits on Dec 11, 2020

  1. Configuration menu
    Copy the full SHA
    5be0fe8 View commit details
    Browse the repository at this point in the history

Commits on Dec 14, 2020

  1. Configuration menu
    Copy the full SHA
    a1ec23d View commit details
    Browse the repository at this point in the history

Commits on Dec 17, 2020

  1. Configuration menu
    Copy the full SHA
    0937643 View commit details
    Browse the repository at this point in the history
  2. close #195. (#196)

    * Update HDF5PluginMacros.cmake
    
    * Update HDF5PluginMacros.cmake
    hyoklee authored and lrknox committed Dec 17, 2020
    Configuration menu
    Copy the full SHA
    93f4ce1 View commit details
    Browse the repository at this point in the history

Commits on Dec 26, 2020

  1. Configuration menu
    Copy the full SHA
    d9fcbec View commit details
    Browse the repository at this point in the history
  2. Avoid aligned access for references by decoding into temporary buffer…

    … and then copying the result into the actual buffer. Update test to be more thorough with using compound datatype fields everywhere. (#206)
    qkoziol authored and lrknox committed Dec 26, 2020
    Configuration menu
    Copy the full SHA
    16ca6d1 View commit details
    Browse the repository at this point in the history

Commits on Jan 6, 2021

  1. Configuration menu
    Copy the full SHA
    acfe570 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    edf801c View commit details
    Browse the repository at this point in the history

Commits on Jan 9, 2021

  1. Configuration menu
    Copy the full SHA
    3736bd1 View commit details
    Browse the repository at this point in the history

Commits on Feb 23, 2021

  1. Configuration menu
    Copy the full SHA
    dbc345b View commit details
    Browse the repository at this point in the history
  2. Fix undefined left shifting of negative numbers (#338)

    Undefined Bahavior Sanitizer errored here about left shifting negative numbers.
    seanm authored and lrknox committed Feb 23, 2021
    Configuration menu
    Copy the full SHA
    3af66dd View commit details
    Browse the repository at this point in the history

Commits on Mar 10, 2021

  1. Configuration menu
    Copy the full SHA
    67d2ada View commit details
    Browse the repository at this point in the history
  2. Fixes various warnings noticed on Windows (#425)

    * Fixes various warnings noticed on Windows
    
    - Adds a prototype for our implementation of vasprintf
    - Return type of H5_get_utf16_str() is now non-const
    - Fixes possible uninitialized return type in Wremove_utf8
    - Better isolation of fork() code in accum.c:test_swmr_write_big()
    - Better isolation of non-zlib code in dsets.c:test_filter_delete()
    - Removed unused variable in trefer.c:test_reference_cmpnd_obj()
    
    * Fixes clang-format issues
    derobins authored and lrknox committed Mar 10, 2021
    Configuration menu
    Copy the full SHA
    febde7c View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    ac550e1 View commit details
    Browse the repository at this point in the history
  4. Applied clang-tidy readability-non-const-parameter warning fixes auto… (

    #429)
    
    * Automatically applied clang-tidy readability-avoid-const-params-in-decls fixes
    
    Removes useless const declarations.
    
    * Fixed most readability-non-const-parameter warnings
    
    These changes were made automatically by clang-tidy, but I manually reverted the changes related to the H5Z_func_t signature.
    
    * Reformat source with clang v10.0.1.
    
    Co-authored-by: Larry Knox <lrknox@hdfgroup.org>
    seanm and lrknox committed Mar 10, 2021
    Configuration menu
    Copy the full SHA
    9840892 View commit details
    Browse the repository at this point in the history
  5. Added C++11 override keyword where appropriate (#433)

    Added H5_OVERRIDE macro for compatibility with both C++11 and older.
    seanm authored and lrknox committed Mar 10, 2021
    Configuration menu
    Copy the full SHA
    e391ca0 View commit details
    Browse the repository at this point in the history
  6. Various clang tidy warning fixes (#448)

    * 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>
    seanm and lrknox committed Mar 10, 2021
    Configuration menu
    Copy the full SHA
    aaa4929 View commit details
    Browse the repository at this point in the history
  7. Removed checks/workarounds for pre-C++89 compatibility (#449)

    After 30+ years, just assume that the following exist:
    - extension-less includes
    - namespaces
    - std::
    - static_cast
    - bool
    seanm authored and lrknox committed Mar 10, 2021
    Configuration menu
    Copy the full SHA
    c7ae947 View commit details
    Browse the repository at this point in the history
  8. Fixed all clang-tidy bugprone-suspicious-string-compare warnings (#451)

    * 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>
    seanm and lrknox committed Mar 10, 2021
    Configuration menu
    Copy the full SHA
    eb95b11 View commit details
    Browse the repository at this point in the history

Commits on Mar 11, 2021

  1. Configuration menu
    Copy the full SHA
    80d2a1d View commit details
    Browse the repository at this point in the history

Commits on Mar 21, 2021

  1. Configuration menu
    Copy the full SHA
    fb6f838 View commit details
    Browse the repository at this point in the history
  2. Simplified hl parsing (#399)

    * Stop using global variables to share parse context with the lexer.
    
    The lexer uses an unconventional strategy for parsing lexical categories
    NUMBER (decimal numbers) and STRING (double-quoted strings) that involves
    sharing the parse context with the lexer using global variables. There
    are a couple of problems with that. First, the lexer is too complicated
    for the simple tokenization it performs—it's hard to tell if it is
    correct. Second, as @seanm points out, the shared global variables
    spill into the namespace shared by other libraries and application
    programs—e.g., VTK.
    
    * Regenerate source files from *.[yl].
    
    * Replace strndup, which isn't available on Windows, with a custom
    routine, `trim_quotes`, that produces a copy of its `const char *`
    argument with leading and trailing double quotes ('"') removed.
    
    While I am here, remove the unnecessary statement `BEGIN INITIAL;`,
    which I should have deleted in a previous commit.
    
    * Regenerate .c from .l.
    
    * You haven't programmed in C until you have programmed in High-Definition
    (HD) C.
    
    * \#include "H5private.h" for HD* definitions.
    
    * Regenerate *.[ch] from *.[yl].
    gnuoyd authored and lrknox committed Mar 21, 2021
    Configuration menu
    Copy the full SHA
    bfdc7df View commit details
    Browse the repository at this point in the history
  3. Removed workarounds for pre-standard inline keyword (#423)

    Fixes -Wreserved-identifier warnings due to multiple underscores in H5_HAVE___INLINE.
    seanm authored and lrknox committed Mar 21, 2021
    Configuration menu
    Copy the full SHA
    a7647a0 View commit details
    Browse the repository at this point in the history
  4. Replaces checks for fork, etc. with checks for unistd.h (#457)

    * Replaces checks for fork, etc. with H5_HAVE_UNISTD_H
    
    Code previously checked for individual POSIX API calls using
    H5_HAVE_FORK, etc. The calls we use have been standardized for
    decades and available via unistd.h.
    
    Some test messages that were missing when tests are skipped
    due to a lack of unistd.h were also added.
    
    The configure checks for individual POSIX API calls will be
    removed in a later commit.
    
    * Stupid formatter
    derobins authored and lrknox committed Mar 21, 2021
    Configuration menu
    Copy the full SHA
    d286ab4 View commit details
    Browse the repository at this point in the history
  5. develop JNI export references and java updates (#467)

    * OESS-98 convert plugin option to FetchContent, add tests
    
    * Fixes for pkcfg files because of plugin option
    
    * OESS-98 fix tools test for plugins
    
    * Keep doxygen comments under 100 chars long - format hint
    
    * Whitespace
    
    * HDFFV-11144 - Reclassify CMake messages
    
    * HDFFV-11099/11100 added help text
    
    * Reworked switch statement to compare string instead
    
    * Fix typo
    
    * Update CDash mode
    
    * Correct name of threadsafe
    
    * Correct option name
    
    * Undo accidental commit
    
    * Note LLVM 10 to 11 format default changes
    
    * Update format plugin
    
    * Undo clang-format version 11 changes
    
    * One more correction
    
    * Update supported platforms
    
    * Revert whitespace changes
    
    * Correct whitespace
    
    * Changes from PR#3
    
    * HDFFV-11213 added option to control gcc10 warnings diagnostics
    
    * HDFFV-11212 Use the new references correctly in JNI utility and tests
    
    * format source
    
    * Fix typo
    
    * Add new test file
    
    * HDFFV-11212 - update test and remove unused arg
    
    * Minor non-space formatting changes
    
    * Use H5I_INVALID_ID instead of "-1"
    
    * source formatting
    
    * add missing testfile, update jni function
    
    * Undo commit of debug code
    
    * remove mislocated file
    
    * Fix h5repack test for handling of fapls and id close
    
    * Update h5diff test files usage text
    
    * HDFFV-11212 add new ref tests for JNI export dataset
    
    * src format update
    
    * Remove blank line typo
    
    * src format typo
    
    * long double requires %Lg
    
    * Another long double foramt specifer S.B. %Lg
    
    * issue with t128bit test
    
    * Windows issue with h5dump and type.
    
    * Fix review issues
    
    * refactor function nesting and fix error checks
    
    * format fixes
    
    * Remove untested functions and javadoc quiet comments
    
    * Restore TRY block.
    
    * Change string append errors to memory exception
    
    * revert to H5_JNI_FATAL_ERROR - support functions need work
    
    * Add assertion error for h5util functions
    
    * remove duplicate function
    
    * format fix
    
    * Revert HD function error handling
    
    * Update copyright comments
    
    * GH #386 java folder copyright corrections
    
    * Whitespace
    
    * GH #359 implement and fix tools 1.6 API usage
    
    * remove excessive comments
    
    * Flip inits to correct ifdef section
    
    * rework ifdef to be simpler
    
    * format issue
    
    * Reformat ifdef inits
    
    * remove static attribute
    
    * format compliance
    
    * Update names
    
    * Revert because logic relies on float not being int
    
    * Changes noticed from creating merge of #412
    
    * Double underscore change
    
    * Correct compiler version variable used
    
    * Remove header guard underscores
    
    * Whitespace cleanup
    
    * Split format source and commit changes on repo push
    
    * remove pre-split setting
    
    * Change windows TS to use older VS.
    
    * correct window os name
    
    * HDFFV-11212 JNI export util and Javadoc
    
    * Suggested review changes
    
    * Another change found
    
    * Committing clang-format changes
    
    Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
    2 people authored and lrknox committed Mar 21, 2021
    Configuration menu
    Copy the full SHA
    bfc49df View commit details
    Browse the repository at this point in the history
  6. Fix display of long double in tools (#469)

    * OESS-98 convert plugin option to FetchContent, add tests
    
    * Fixes for pkcfg files because of plugin option
    
    * OESS-98 fix tools test for plugins
    
    * Keep doxygen comments under 100 chars long - format hint
    
    * Whitespace
    
    * HDFFV-11144 - Reclassify CMake messages
    
    * HDFFV-11099/11100 added help text
    
    * Reworked switch statement to compare string instead
    
    * Fix typo
    
    * Update CDash mode
    
    * Correct name of threadsafe
    
    * Correct option name
    
    * Undo accidental commit
    
    * Note LLVM 10 to 11 format default changes
    
    * Update format plugin
    
    * Undo clang-format version 11 changes
    
    * One more correction
    
    * Update supported platforms
    
    * Revert whitespace changes
    
    * Correct whitespace
    
    * Changes from PR#3
    
    * HDFFV-11213 added option to control gcc10 warnings diagnostics
    
    * HDFFV-11212 Use the new references correctly in JNI utility and tests
    
    * format source
    
    * Fix typo
    
    * Add new test file
    
    * HDFFV-11212 - update test and remove unused arg
    
    * Minor non-space formatting changes
    
    * Use H5I_INVALID_ID instead of "-1"
    
    * source formatting
    
    * add missing testfile, update jni function
    
    * Undo commit of debug code
    
    * remove mislocated file
    
    * Fix h5repack test for handling of fapls and id close
    
    * Update h5diff test files usage text
    
    * HDFFV-11212 add new ref tests for JNI export dataset
    
    * src format update
    
    * Remove blank line typo
    
    * src format typo
    
    * long double requires %Lg
    
    * Another long double foramt specifer S.B. %Lg
    
    * issue with t128bit test
    
    * Windows issue with h5dump and type.
    
    * Fix review issues
    
    * refactor function nesting and fix error checks
    
    * format fixes
    
    * Remove untested functions and javadoc quiet comments
    
    * Restore TRY block.
    
    * Change string append errors to memory exception
    
    * revert to H5_JNI_FATAL_ERROR - support functions need work
    
    * Add assertion error for h5util functions
    
    * remove duplicate function
    
    * format fix
    
    * Revert HD function error handling
    
    * Update copyright comments
    
    * GH #386 java folder copyright corrections
    
    * Whitespace
    
    * GH #359 implement and fix tools 1.6 API usage
    
    * remove excessive comments
    
    * Flip inits to correct ifdef section
    
    * rework ifdef to be simpler
    
    * format issue
    
    * Reformat ifdef inits
    
    * remove static attribute
    
    * format compliance
    
    * Update names
    
    * Revert because logic relies on float not being int
    
    * Changes noticed from creating merge of #412
    
    * Double underscore change
    
    * Correct compiler version variable used
    
    * Remove header guard underscores
    
    * Whitespace cleanup
    
    * Split format source and commit changes on repo push
    
    * remove pre-split setting
    
    * Change windows TS to use older VS.
    
    * correct window os name
    
    * HDFFV-11212 JNI export util and Javadoc
    
    * Suggested review changes
    
    * Another change found
    
    * Committing clang-format changes
    
    * HDFFV-11113 long double in tools
    
    * HDFFV-11113 add note
    
    Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
    2 people authored and lrknox committed Mar 21, 2021
    Configuration menu
    Copy the full SHA
    7e541fa View commit details
    Browse the repository at this point in the history
  7. Update clang config (#473)

    * 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>
    2 people authored and lrknox committed Mar 21, 2021
    Configuration menu
    Copy the full SHA
    2f058d2 View commit details
    Browse the repository at this point in the history
  8. Correct syntax error. (#474)

    a Fortran compile -> a Fortran compiler
    hyoklee authored and lrknox committed Mar 21, 2021
    Configuration menu
    Copy the full SHA
    e8d6f84 View commit details
    Browse the repository at this point in the history
  9. Cleans up a couple of MSVC warnings in testhdf5 (#475)

    * Fixes a few testhdf5 warnings raised in Visual Studio
    
    Visual Studio is grumpier about treating pointers like integers than
    gcc.
    
    * Committing clang-format changes
    
    Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
    2 people authored and lrknox committed Mar 21, 2021
    Configuration menu
    Copy the full SHA
    b1815c6 View commit details
    Browse the repository at this point in the history
  10. Fix typos and grammar errors. (#476)

    hyoklee authored and lrknox committed Mar 21, 2021
    Configuration menu
    Copy the full SHA
    d5cfc82 View commit details
    Browse the repository at this point in the history
  11. Suppresses the tcheck_version test's abort dialog on Windows (#477)

    * Suppresses the tcheck_version test's abort dialog on Windows
    
    Windows raises a modal abort/retry/ignore dialog box when CRT
    calls abort(). This change installs a report hook that suppresses
    the dialog so that the CMake tests don't time out waiting for a
    nonexistent user to click a dialog box.
    
    * Committing clang-format changes
    
    * Removes __cdecl from callback
    
    Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
    2 people authored and lrknox committed Mar 21, 2021
    Configuration menu
    Copy the full SHA
    efd1a78 View commit details
    Browse the repository at this point in the history

Commits on Mar 25, 2021

  1. Rerun genparser.

    lrknox committed Mar 25, 2021
    Configuration menu
    Copy the full SHA
    415306d View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    d64f11f View commit details
    Browse the repository at this point in the history

Commits on Mar 26, 2021

  1. Configuration menu
    Copy the full SHA
    f4e7160 View commit details
    Browse the repository at this point in the history
  2. Merge branch 'hdf5_merge_issue_487_v112' of https://github.com/lrknox…

    …/hdf5 into hdf5_merge_issue_487_v112
    lrknox committed Mar 26, 2021
    Configuration menu
    Copy the full SHA
    50f6d98 View commit details
    Browse the repository at this point in the history