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

Use BEZ_ prefix for exported C ABI names #167

Merged
merged 10 commits into from Jan 10, 2020
Merged

Use BEZ_ prefix for exported C ABI names #167

merged 10 commits into from Jan 10, 2020

Commits on Jan 10, 2020

  1. Added BEZ_ prefix to all ABI names.

    Done via:
    
    ```
    git grep -l "name='" -- '*.f90' | xargs sed -i '' s/"name='"/"name='BEZ_"/g
    git grep -l "name='" -- '*.f90' | xargs sed -i    s/"name='"/"name='BEZ_"/g
    ```
    
    This was an automated change, will be followed up by hand-written
    changes to all references to these ABI procedures.
    dhermes committed Jan 10, 2020
    Configuration menu
    Copy the full SHA
    62ccd01 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    c6de06a View commit details
    Browse the repository at this point in the history
  3. Referring to BEZ_ prefixed ABI names in header files.

    (Still need to run `clang-format` on these, it is not yet installed
    on my laptop and I am currently offline.)
    dhermes committed Jan 10, 2020
    Configuration menu
    Copy the full SHA
    8c8c456 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    e30c7bd View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    28854c4 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    66f3356 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    678ca3b View commit details
    Browse the repository at this point in the history
  8. Verifying docs/abi builds and the examples run.

    - Updated some Sphinx references with `BEZ_` prefix
    - Fixed an oversight in `example_locate_point_curve`
    - Fixed an old reference to `example_curve_specialize.c` (now is
      a reference to `example_specialize_curve.c`)
    - Change `-I src/fortran/include/` line to use `...` to indicate
      that path is not in the same directory as the `example*.c`
      files
    dhermes committed Jan 10, 2020
    Configuration menu
    Copy the full SHA
    15bdcc2 View commit details
    Browse the repository at this point in the history
  9. Configuration menu
    Copy the full SHA
    15d16f7 View commit details
    Browse the repository at this point in the history
  10. Disabling missing-return-type-doc Pylint message on Sphinx aliases.

    I introduced this in dc58f0a,
    but commited it with `[ci skip]`. (I should never use `ci skip`,
    I've learned that lesson before.)
    dhermes committed Jan 10, 2020
    Configuration menu
    Copy the full SHA
    2cce61b View commit details
    Browse the repository at this point in the history