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

Restyle cleanup: Ensure we limit the system headers included in .h files. #1970

Closed
wants to merge 4 commits into from
Closed

Restyle cleanup: Ensure we limit the system headers included in .h files. #1970

wants to merge 4 commits into from

Commits on Feb 4, 2022

  1. cleanup: Ensure we limit the system headers included in .h files. 

    Most system headers contain functions (e.g. `memcpy` in `string.h`)
which aren't needed in our own header files. For the most part, our own
headers should only include types needed to declare our own types and
functions. We now enforce this so we think twice about which headers we
really need in the .h files.
    @iphydf
    iphydf committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    8b17f57 View commit details
    Browse the repository at this point in the history

  2. Restyled by black

    @restyled-commits
    restyled-commits committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    be90dd9 View commit details
    Browse the repository at this point in the history

  3. Restyled by reorder-python-imports

    @restyled-commits
    restyled-commits committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    712f2f0 View commit details
    Browse the repository at this point in the history

  4. Restyled by yapf

    @restyled-commits
    restyled-commits committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    425d783 View commit details
    Browse the repository at this point in the history