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

Improvements to HRESULT #666

Merged
merged 6 commits into from
Sep 7, 2022
Merged

Improvements to HRESULT #666

merged 6 commits into from
Sep 7, 2022

Commits on Aug 25, 2022

  1. Improvements to HRESULT

    elachlan committed Aug 25, 2022
    Configuration menu
    Copy the full SHA
    cc78d53 View commit details
    Browse the repository at this point in the history

Commits on Sep 6, 2022

  1. Fix HRESULT_FROM_WIN32 definition to handle negative Win32 errors

    Also ensure invariant formatting of the HRESULT
    AArnott committed Sep 6, 2022
    Configuration menu
    Copy the full SHA
    45dd702 View commit details
    Browse the repository at this point in the history
  2. Remove HRESULT_FROM_WIN32 from HRESULT type

    As a member of the HRESULT struct, it caused the otherwise small HRESULT struct to also generate the very large `WIN32_ERROR` enum, even if the user wouldn't have used this function.
    So instead, we start a new concept where 'macros' such as this can be defined as a template, and then requested for generation explicitly. The generator is smart enough to detect what types the macro uses and generate them as well, as proven by the new test.
    AArnott committed Sep 6, 2022
    Configuration menu
    Copy the full SHA
    f53f248 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    7d21740 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    18aed91 View commit details
    Browse the repository at this point in the history

Commits on Sep 7, 2022

  1. Fix visibility of macros

    AArnott committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    5f2707b View commit details
    Browse the repository at this point in the history