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.

Sign up for GitHub

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

Jump to bottom

Put H5T_CONV_ab macros in do..while loops #3432

Merged
merged 1 commit into from
Aug 25, 2023
Merged

Put H5T_CONV_ab macros in do..while loops #3432

merged 1 commit into from
Aug 25, 2023

Commits on Aug 25, 2023

  1. Put H5T_CONV_ab macros in do..while loops 

    Ever since a recent round of macro cleanup, bin/trace and clang-format
have been bickering over what H5Tconv.c should look like and neither
produces readable code.

This change puts the top-level H5T_CONV_ab macros in do..while loops,
adds appropriate semicolons, and adds the missing H5_CLANG_DIAG_ON|OFF
and H5_GCC_CLANG_DIAG_ON|OFF macros to the list of statement macros
clang-format recognizes. H5Tconv.c is now readable and both bin/trace
and clang-format are happy.
    @derobins
    derobins committed Aug 25, 2023
    Configuration menu
    Copy the full SHA
    8eadee5 View commit details
    Browse the repository at this point in the history