-
Notifications
You must be signed in to change notification settings - Fork 120
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
base: main
Are you sure you want to change the base?
Conversation
@@ -13,11 +13,3 @@ _D3DCOLORVALUE | |||
<IncludeRoot>/um/dxprogrammablecapture.h | |||
--namespace | |||
Windows.Win32.Graphics.Dxgi | |||
--with-attribute |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
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?
On the side, alphabetically sort some flags and use `--enumMakeFlags` in `emitter.settings.rsp` instead of `--with-attribute XXX=Flags` in `Namespace/settings.rsp`.
26da534
to
c4cc88f
Compare
Draft separation from #1911 (comment).
On the side, alphabetically sort some flags and use
--enumMakeFlags
inemitter.settings.rsp
instead of--with-attribute XXX=Flags
inNamespace/settings.rsp
.