-
Notifications
You must be signed in to change notification settings - Fork 123
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
Fix tracking queue family properties #1810
Fix tracking queue family properties #1810
Conversation
CI gfxreconstruct build queued with queue ID 278563. |
CI gfxreconstruct build # 5064 running. |
CI gfxreconstruct build # 5064 passed. |
@MarkY-LunarG if I reviewed this and it looks good, are you okay with @ziga-lunarg merging it or do you want a chance to review? |
I'm ok with this. This might be something we have to do in other API entrypoints (like EnumPhysDevs). But let's start with this for now. |
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.
Just put a comment around each instance explaining why you're doing it. No ones going to see the commit and know why just by looking at the code.
3b94abb
to
a663a1b
Compare
CI gfxreconstruct build queued with queue ID 279427. |
CI gfxreconstruct build # 5081 running. |
CI gfxreconstruct build # 5081 passed. |
Ran into a case where an app did:
vkGetPhysicalDeviceQueueFamilyProperties()
to get the countvkGetPhysicalDeviceQueueFamilyProperties()
with the countvkGetPhysicalDeviceQueueFamilyProperties()
with 1And we just stored the 1.