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

dxgi: Add DXGI_OUTDUPL_FLAG enum annotation to DuplicateOutput1() #1981

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MarijnS95
Copy link
Contributor

Draft separation from #1911 (comment).

On the side, alphabetically sort some flags and use --enumMakeFlags in emitter.settings.rsp instead of --with-attribute XXX=Flags in Namespace/settings.rsp.

@@ -13,11 +13,3 @@ _D3DCOLORVALUE
<IncludeRoot>/um/dxprogrammablecapture.h
--namespace
Windows.Win32.Graphics.Dxgi
--with-attribute
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inversely, could we also have all the =AssociatedEnum in here? It seems much nicer to me to have configuration in the namespace-specific settings.rsp than in one humongous global file, if possible?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. The scraper and the emitter have different roles and supporting toolchains.

Copy link
Contributor Author

@MarijnS95 MarijnS95 Sep 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any complete docs on what configures the scraper (probably the namespace-specific settings.rsp) and what configures the emitter (probably the glboal emitter.settings.rsp... 🙂), and when to use either?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

@MarijnS95 MarijnS95 Sep 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mikebattista unfortunately that doesn't explain directly which flags are available, nor what precedence I should stick to. Searching further in the repo shows that --enumMakeFlags is a feature of ClangSharpSourceToWinmd so it makes sense that that only works via emitter settings (and likewise, --with-attribute seems to be a feature of the scraper).

The question remains: do we prefer to set the Flags attribute on the emitter via --enumMakeFlags, or do we prefer to set it earlier via --with-attribute on the scraper as done before this PR?

mikebattista
mikebattista previously approved these changes Sep 9, 2024
On the side, alphabetically sort some flags and use `--enumMakeFlags`
in `emitter.settings.rsp` instead of `--with-attribute XXX=Flags`
in `Namespace/settings.rsp`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants